-
After 15+ years, we've made a big change: Android Forums is now Early Bird Club. Learn more here.
Questions
-
Question
What app notifcation icon is this? 2 heart lines
- By Qwedalis333
- Ask a question
- 1 Replies
Attachments
What is the correct thread-safe way to access shared resource?
- By uavengineer
- Android Development
- 3 Replies
We are developing on Samsung Tab Active 4 Pro using Android Studio, kotlin and java. We are getting constant at random times app deadlocks. I am not new to developing with threads and synchronization methods however I am very new to Android development. I assume I am doing something wrong and will share what I am doing. But to set up the situation, we are using OSMDroid for the map viewing and are using layers of the map for draw icons that represent aircraft flying over the map. The main android app is done in kotlin and the UDP over ethernet is written in java files. Aircraft telemetry is received at about 1 hz over ethernet inside protobuf packets. The protobuf is received and parsed in a Listener java class using a thread and implemented in the run() method. In the udp handler where we receive and parse protobuf we call a kotlin object "class" (unfortunate use of terms with kotlin) that has a method UpdateEntity(). Here is a snippet of this code that handles a proto message type called Entity. EntityClass is the java wrapper created. This part is working fine, no issues here, just showing for context.
Datasource.INSTANCE.UpdateEntity(unit); call accesses the share list. Here is that method in total. Note Datasource is that singleton-like object construct that has all public "static" methods. (I'm a C++ guy mostly
New to kotlin)
My understanding of kotlin and android is that you need to use a coroutine if you want to use a mutex. I used a mutex to protect the shared resource, the entityUnitList of type mutableStateListOf<EntityUnit>
The Listener is started up on the main, UI thread however it uses its own thread to receive udp packets. The call to Datasource::UpdateEntity() is done in that thread, not the UI thread. Datasource is used by the UI thread as well and it needs access to the entityUnitLst to read from. The UI thread does not write to any of the elements in the list nor does it add or subtract elements from that list.
So the problem is the app will lock (not crash and disappear) and be non-responsive or frozen. This happens at various times but typically within a minute or two. My best guess is a thead race condition.
There was someone else developing the UI portions of this app and they said the UI only updates when a data element of it changes. So in UpdateEntity() these two lines do cause a UI refresh.
Here is another place accessing that list and is in the UI thread
There is more to that function but that function is called from here in a global method tha tis used in the map drawing code.
I am thinking we have a mutex locking unlocking issue where it stays locked and the UI blocks with that runBlocking{ } block waiting for the unlock. The Listener gets in and gets out when updating the entity pretty quick. The only link to the UI thread is Datasource::UpdateEntity(). When we comment out UpdateEntity() we never deaklock or freeze the app. We can even pre-load entityUnitList with entity class elements and interact with them again with no app freeze. Its only when we enable getting movement updates over ethernet (using USB/Ethernet adapter plugged into the tablet) when we freeze the app eventually.
So I think using runBlocking is wrong so what is the correct way to share that entityUnitList over multiple threads safely? We do not have to use a mutex and we could consider other list types that are better with thread synchronization.
-Steve
EDIT wow this post is long, I apologize! Just wanted to give enough details.
Code:
if (msg.getMessageTypeEnum() == MessageWrapperClass.MessageWrapper.MessageTypeEnum.MESSAGE_TYPE_ENTITY) {
EntityClass.Entity entity = msg.getMessageEntity();
Log.d(TAG, "Listener:run (thread): Entity received\"" + entity.toString());
// 0 deg points north, vertically in map
double heading = Math.atan2(
entity.getKinematics().getEastSpeed(),
entity.getKinematics().getNorthSpeed()) * 180.0 / Math.PI;
// Put to [0,360] range
if (heading < 0) {
heading += 360.0;
}
Position position = new Position(
heading,
entity.getKinematics().getPosition().getLatitudeDeg(),
entity.getKinematics().getPosition().getLongitudeDeg(),
entity.getKinematics().getPosition().getAltitudeHaeMeters(),
0.0);
HostileEntity unit = new HostileEntity(
Utils.TrimUUID(entity.getEntityID().getValue().toString()),
//todo: we need callsign here
"red", //entity.getEntityID().getValue().toString(),
position, false);
Datasource.INSTANCE.UpdateEntity(unit); // <---------- This accesses the shared list
}Datasource.INSTANCE.UpdateEntity(unit); call accesses the share list. Here is that method in total. Note Datasource is that singleton-like object construct that has all public "static" methods. (I'm a C++ guy mostly
New to kotlin)
Java:
/**
* [Datasource] holds the data that is utilized by the UI
*/
object Datasource {
var currentStatus = ""
var currentMode = Modes.Nav
var Settings:SettingsClass = SettingsClass()
//This is the list of automations
var AutomationList = mutableListOf<AutomationClass>(
WTP(),
EW()
)
var TabPages = mutableListOf<String>()
var TabPageContents = mutableListOf<TabPage>()
var KillPairs = mutableListOf<KillPair>()
var SelectedTab:Int = 0
var newTab:String = ""
var locklineschanged = false
val entityUnitListMutex = Mutex(false)
val killPairsMutex = Mutex(false)
/*
This is the list of units. Any unit added to this list will get added to the map.
*/
val entityUnitList = mutableStateListOf<EntityUnit>()
fun UpdateEntity(entity:EntityUnit)
{
if (!entity.isPositionValid()) {
Log.e(ContentValues.TAG, "Datasource::UpdateEntity " + "Invalid entity passed:")
return
}
entity.lifeTime_ms = System.currentTimeMillis() // update its life time
try {
// NOTE - FindEntity() call must remain OUTSIDE of runblocking and mutex because
// FindEntity also has mutex and we do not want to nest mutex calls. Deadlock may occur
val existingEntity = FindEntity(entity.id)
runBlocking {
entityUnitListMutex.withLock {
if (existingEntity != null) {
entity.status = existingEntity.status
entity.missionCount = existingEntity.missionCount
val index = entityUnitList.indexOf(existingEntity)
entityUnitList[index] = entity
} else {
entityUnitList.add(entity)
}
}// end mutex
}
}
catch(e:Exception)
{
Log.e(ContentValues.TAG, "Datasource::UpdateEntity " + e.toString())
}
}
/*
Find the entity in the list of active entities
*/
fun FindEntity(entityName: String): EntityUnit? {
try {
var returnval: EntityUnit? = null
runBlocking {
entityUnitListMutex.withLock {
for (i in entityUnitList.indices) {
if (entityUnitList[i].id == entityName) {
returnval = entityUnitList[i]
break
}
}
}// end mutex
}
return returnval
} catch (e: Exception) {
Log.e(ContentValues.TAG, "Datasource::FindEntity " + e.toString())
return null
}
}My understanding of kotlin and android is that you need to use a coroutine if you want to use a mutex. I used a mutex to protect the shared resource, the entityUnitList of type mutableStateListOf<EntityUnit>
The Listener is started up on the main, UI thread however it uses its own thread to receive udp packets. The call to Datasource::UpdateEntity() is done in that thread, not the UI thread. Datasource is used by the UI thread as well and it needs access to the entityUnitLst to read from. The UI thread does not write to any of the elements in the list nor does it add or subtract elements from that list.
So the problem is the app will lock (not crash and disappear) and be non-responsive or frozen. This happens at various times but typically within a minute or two. My best guess is a thead race condition.
There was someone else developing the UI portions of this app and they said the UI only updates when a data element of it changes. So in UpdateEntity() these two lines do cause a UI refresh.
Code:
entityUnitList[index] = entity
...
entityUnitList.add(entity)Here is another place accessing that list and is in the UI thread
Code:
fun PutStuffOnMap(
uiState: FOXUiState, mapView: MapView, onEntityClicked: (entityClicked: String) -> Unit
) {
try {
val items = ArrayList<OverlayItem>()
runBlocking {
Datasource.entityUnitListMutex.withLock {
for (i in entityUnitList.indices) {
val entity = entityUnitList[i]
PlotEntity(mapView, entity)?.let { items.add(it) }
}
}// end mutex
}
. . .There is more to that function but that function is called from here in a global method tha tis used in the map drawing code.
Java:
@Composable
fun MapRow(
uiState: FOXUiState,
onEntityClicked: (entityClicked: String) -> Unit,
modifier: Modifier = Modifier){
Row(modifier = modifier) {
AndroidView(
factory = { context ->
MapView(context).apply {
SetUpMap( uiState, this, onEntityClicked)
}
},
update = {
mapView ->
PutStuffOnMap( uiState, mapView, onEntityClicked)
}
)
}
}I am thinking we have a mutex locking unlocking issue where it stays locked and the UI blocks with that runBlocking{ } block waiting for the unlock. The Listener gets in and gets out when updating the entity pretty quick. The only link to the UI thread is Datasource::UpdateEntity(). When we comment out UpdateEntity() we never deaklock or freeze the app. We can even pre-load entityUnitList with entity class elements and interact with them again with no app freeze. Its only when we enable getting movement updates over ethernet (using USB/Ethernet adapter plugged into the tablet) when we freeze the app eventually.
So I think using runBlocking is wrong so what is the correct way to share that entityUnitList over multiple threads safely? We do not have to use a mutex and we could consider other list types that are better with thread synchronization.
-Steve
EDIT wow this post is long, I apologize! Just wanted to give enough details.
-
Question
Make Lock Screen Music Player Full Screen?
- By BMenendez
- Ask a question
- 2 Replies
Hi all,
Just switched from iOS to Android (wanted to try the ZFold4) and one of my biggest dislikes that I can't get past is the lockscreen music player. Is there anyway to display that full screen (like iPhones do)? I have YouTube Music but would certainly use whatever app would just make it fullscreen lol. Appreciate your help all!!!
Just switched from iOS to Android (wanted to try the ZFold4) and one of my biggest dislikes that I can't get past is the lockscreen music player. Is there anyway to display that full screen (like iPhones do)? I have YouTube Music but would certainly use whatever app would just make it fullscreen lol. Appreciate your help all!!!
-
Question
How to stop Play Store from updating itself in Android 5.1
- By Akis1919
- Ask a question
- 2 Replies
-
Question
Help Instant data usage stats for different apps individually
- By zaa sala
- Ask a question
- 1 Replies
-
Question
HELP!! can anyone tell me what app this is ???
- By Strawbrea
- Ask a question
- 4 Replies
Attachments
-
Question
Help How can I reinstall Android on my car's tablet display?
- By TzviM
- Ask a question
- 3 Replies
-
Question
Offline Notes App?
- Ask a question
- 4 Replies
Hi all,
I've been looking all over the Android App Store for a Notes App that doesn't need to connect to the Internet. On my very old Huawei G510-0200 it came with a Notes App that works completely independent from the Internet and this is great. However, it seems like everything wants to connect to the Internet...
My Huawei phone link: HUAWEI G510-0200 Specification
Thanks for any help!!!
I've been looking all over the Android App Store for a Notes App that doesn't need to connect to the Internet. On my very old Huawei G510-0200 it came with a Notes App that works completely independent from the Internet and this is great. However, it seems like everything wants to connect to the Internet...
My Huawei phone link: HUAWEI G510-0200 Specification
Thanks for any help!!!
BEST TECH IN 2023
We've been tracking upcoming products and ranking the best tech since 2007. Thanks for trusting our opinion: we get rewarded through affiliate links that earn us a commission and we invite you to learn more about us.
Smartphones
Best Android Phones
See All- Google Pixel 8 Pro Check Price
- Samsung Galaxy S23 Ultra Check Price
- Samsung Galaxy Z Fold5 Check Price
- Google Pixel 8 Check Price
- Samsung Galaxy S23 Check Price
Upcoming
See All
Best iPhones
See All- Apple iPhone 15 Pro Max Check Price
- Apple iPhone 15 Pro Check Price
- Apple iPhone 15 Plus Check Price
- Apple iPhone 15 Check Price
- Apple iPhone SE (2022) Check Price
Upcoming
See AllTablets
Best Tablets
See All- Samsung Galaxy Tab S9 Ultra Check Price
- Apple iPad Pro (2022) Check Price
- Apple iPad Air (2022) Check Price
- Apple iPad Mini (2021) Check Price
- Microsoft Surface Pro 9 Check Price
Upcoming
See AllLaptops
Best Laptops
See All- Apple Macbook Pro Check Price
- Apple Macbook Air (2023) Check Price
- Dell XPS 13 Check Price
- Acer Chromebook Spin 714 Check Price
- Dell Alienware m18 (2022) Check Price
Upcoming
See AllTelevisions
Best TVs
See All- Samsung The Frame TV Check Price
- Samsung Neo QLED 4K QN90C Check Price
- LG G3 OLED Check Price
- LG A2 OLED Check Price
- ROKU Plus Series Check Price
- Samsung S90C OLED Check Price
- SunBriteTV Veranda 3 Check Price
Upcoming
See AllGame Consoles
Best Game Consoles
See All- Nintendo Switch OLED Check Price
- Microsoft XBOX Series X Check Price
- Sony Playstation 5 Check Price
- Microsoft XBOX Series S Check Price
- Nintendo Switch Lite Check Price
Upcoming
See AllWearables
Best Wearables
See All- Oura Ring 3 Check Price
- Apple Watch Series 9 Check Price
- Google Pixel Watch 2 Check Price
- Samsung Galaxy Watch 6 Classic Check Price
- Fitbit Inspire 3 Check Price
- Amazfit Amazfit Band 7 Check Price
- Apple Watch SE Check Price
- Apple Watch Ultra 2 Check Price
