MatteCarra / AccA

Acc app allows to edit acc config file and add a tile to start and stop acc deamon
GNU General Public License v3.0
1.19k stars 85 forks source link

latest app and majisk module causing reboot #207

Open nutpantz opened 2 years ago

nutpantz commented 2 years ago

Running the latest app and majisk module add of today. Charge limiting to 80% working, limit charge to start when below 70% not working. Test charging switches causes reboot. Sometimes putting device on wireless charging causing reboot. It seemed to work before i updated but i was having issues with it stop charging at all after a few days and i had to reboot. Now it's much worse. Using galaxy s7 edge with lineage os Android 11 last updated 12-14-2021 with micro g

Uninstalling the majisk module and usingthe app and only using the app it seems to work again.

Update. No it was just a reboot that had it temporary working. It stopped not long after.b also stopping the service from the app also caused a reboot

VR-25 commented 2 years ago

Troublesome charging switches trigger unexpected reboots on some systems. To quickly identify these, plug the charger and run acc -x -t. After the reboot, /sdcard/acc-*.log reveals the problematic switch. It can then be removed from ctrl-files.sh or blacklisted.

nutpantz commented 2 years ago

Acc -x -t in a su terminal just shows the help info

Tried a different Android 11 rom and still have charging issues but no rebooting so far. Even thought the lock screen says I'm charging wirelessly it's not charging. Battery charge light is not on. After running the app it started charging. Battery optimisation is off for the app. had no issues with Android 8 or 9 just 11

VR-25 commented 2 years ago
  1. Flash the latest acc snapshot from here;
  2. Plug the charger;
  3. Paste and run pkill -9 accd; acc -x -t;
  4. Share /sdcard/acc-*.log after the reboot.
nutpantz commented 2 years ago

Failed to flash. Error 255

On January 29, 2022 1:38:51 AM PST, VR-25 @.***> wrote:

Flash the latest acc snapshot from here. Paste and run pkill -9 accd; acc -x -t. Share /sdcard/acc-*.log.

-- Reply to this email directly or view it on GitHub: https://github.com/MatteCarra/AccA/issues/207#issuecomment-1024876891 You are receiving this because you authored the thread.

Message ID: @.***>

VR-25 commented 2 years ago

In recovery mode, you need to mount system first. Did you try flashing via Magisk?

VR-25 commented 2 years ago

acc_2022-01-29_18.16.59.zip See if this build also triggers a reboot.

nutpantz commented 2 years ago

Flashing from magisk works. Recovery with system mounted still did not. Info attached. However phone now thinks it's charging when it's hooked up to nothing. Battery charging light is on and lock screen says it's charging. This is new behavior since the flash

On January 29, 2022 9:13:24 AM PST, VR-25 @.***> wrote:

In recovery mode, you need to mount system first. Did you try flashing via Magisk?

-- Reply to this email directly or view it on GitHub: https://github.com/MatteCarra/AccA/issues/207#issuecomment-1024950091 You are receiving this because you authored the thread.

Message ID: @.***>

nutpantz commented 2 years ago

01-29 is also causing a reboot when hooked up to the charger and checking the settings in the app causes the app the freeze.

What's the best way to uninstall everything to start fresh?

VR-25 commented 2 years ago

That version has several charging control files blacklisted. Do you mind going through steps 3 and 4 again to see what else should be blacklisted?

acc -U removes everything.

nutpantz commented 2 years ago

1-29 That caused a reboot during the test Here is the log. I love this app and oil do what i can help to make it better. Just ask.

On January 29, 2022 11:07:59 AM PST, VR-25 @.***> wrote:

That version has several charging control files blacklisted. Do you mind going through steps 3 and 4 again to see what else should be blacklisted?

acc -U removes everything.

-- Reply to this email directly or view it on GitHub: https://github.com/MatteCarra/AccA/issues/207#issuecomment-1024969220 You are receiving this because you authored the thread.

Message ID: @.***>

VR-25 commented 2 years ago

You forgot the logs. There's a new zip to share, but I would like to see your logs first.

nutpantz commented 2 years ago

Deleted Reason: pasting the contents of a massive log file here is a very impolite way of sharing logs. Besides, it causes browser performance issues.

VR-25 commented 2 years ago

What's the output of acc -ss:?

nutpantz commented 2 years ago

:/ $ su :/ # acc -ss:

battery/batt_slate_mode 0 1 battery/hmt_ta_charge 1 0 battery/store_mode 0 1 max77854-fuelgauge/charge_enabled 1 0 ac/current_max 660000 0 battery/batt_tune_chg_limit_cur 1100 0 battery/batt_tune_fast_charge_current 2100 0 battery/batt_tune_intput_charge_current 1800 0 max77854-charger/current_max 660 0 usb/current_max 660000 0 wireless/current_max 660000 0 battery/batt_tune_float_voltage 43500 voltage_now battery/fg_full_voltage 43500 voltage_now max77854-charger/voltage_max 43500 voltage_now 0 mA 250 mA 350 mA 500 mA 3700 mV

:/ #

VR-25 commented 2 years ago

acc_2022-01-30_20.38.01.zip

Plug the charger, install the new build, run pkill -9 accd; acc -t and share /data/adb/vr25/acc-data/logs/acc-t.log after the reboot.

nutpantz commented 2 years ago

Flashing that in magiskb caused a reboot and running the command caused a reboot. The requested log file was not in that directory, just a install and init log

On January 30, 2022 11:38:50 AM PST, VR-25 @.***> wrote:

acc_2022-01-30_20.38.01.zip

Plug the charger, install the new build, run pkill -9 accd; acc -t and share /data/adb/vr25/acc-data/logs/acc-t.log after the reboot.

-- Reply to this email directly or view it on GitHub: https://github.com/MatteCarra/AccA/issues/207#issuecomment-1025215122 You are receiving this because you authored the thread.

Message ID: @.***>

VR-25 commented 2 years ago
  1. Plug the charger and run pkill -9 accd; acc -s s="battery/batt_slate_mode 0 1 --"
  2. Reinstall the new build
  3. Run acc -t and you should have the log file after the reboot

On Sun, Jan 30, 2022, 20:52 nutpantz @.***> wrote:

Flashing that in magiskb caused a reboot and running the command caused a reboot. The requested log file was not in that directory, just a install and init log

On January 30, 2022 11:38:50 AM PST, VR-25 @.***> wrote:

acc_2022-01-30_20.38.01.zip

Plug the charger, install the new build, run pkill -9 accd; acc -t and share /data/adb/vr25/acc-data/logs/acc-t.log after the reboot.

-- Reply to this email directly or view it on GitHub: https://github.com/MatteCarra/AccA/issues/207#issuecomment-1025215122 You are receiving this because you authored the thread.

Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/MatteCarra/AccA/issues/207#issuecomment-1025217739, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG42GSASFKZFKXRI772EBD3UYWJH5ANCNFSM5MK2HJLQ . You are receiving this because you commented.Message ID: @.***>

nutpantz commented 2 years ago

No log file created. No reboot during flashing but it did reboot when i ran the command

Tried twice

VR-25 commented 2 years ago

acc_2022-01-31_19.07.13.zip This build will auto detect and blacklist the charging control files that are causing reboots. Run pkill -9 accd before installing.

Once you've ran acc -t a few times, and no more reboots occurred, share /data/adb/vr-25/acc-data/logs/write.log.

nutpantz commented 2 years ago

battery/batt_tune_chg_limit_cur

battery/batt_slate_mode battery/charge_control_limit battery/hmt_ta_charge battery/store_mode max77854-fuelgauge/charge_enabled

No issues flashing or running the commands. However the phone thinks it's charging all the time after being removed from the wireless charger. Seems to work fine after a reboot Lots of reboots on charger or changing settings in app

On January 31, 2022 10:44:14 AM PST, VR-25 @.***> wrote:

acc_2022-01-31_19.07.13.zip This build will auto detect and blacklist the charging control files that are causing reboots. Run pkill -9 accd before installing.

Once you run acc -t a few times, without reboots, share /data/adb/vr-25/acc-data/logs/write.log.

-- Reply to this email directly or view it on GitHub: https://github.com/MatteCarra/AccA/issues/207#issuecomment-1026092832 You are receiving this because you authored the thread.

Message ID: @.***>

VR-25 commented 2 years ago

#battery/batt_tune_chg_limit_cur is the deal. The leading hashtag means it was behind the reboots and got blacklisted. It's surprising, though, that this current control file would trigger a kernel panic. The kernel has major power management bugs.

Just to be sure, which issues are you experiencing now? Share the output of acc -t and the log archive generated by acc -le.

nutpantz commented 2 years ago

:/ $ su :/ # acc -t

(!) Charger must be plugged to continue... (i) This may take a minute or so...

(i) [battery/batt_slate_mode 0 1] works

(i) [battery/charge_control_limit 0 1] works

(!) [battery/hmt_ta_charge 1 0] won't work

(!) [battery/store_mode 0 1] won't work

(!) [max77854-fuelgauge/charge_enabled 1 0] won't work

:/ #

nutpantz commented 2 years ago

acc-logs-hero2ltebmc.tar.gz

nutpantz commented 2 years ago

I'm having Reboots sometimes when placing the phone on a wireless charger or taking it off Reboots when changing settings in the app Sometimes the charging % limits from the app don't work Sometimes the phone stops charging long before the % limit. Usually a reboot gets the phone charging to the right limit for at least 12 hours or so. Reboots from charging or changing settings is mostly random. Everything is with the fresh install of 2 different Android 11 lineage os roms

I'll look to see if there is a alternate kernel available.

VR-25 commented 2 years ago

acc_2022-02-01_15.36.09.zip I implemented workarounds for better compatibility with your system.

nutpantz commented 2 years ago

I switched to Android 10 which has a few issues but nothing big enough for me to follow up on. Android 11 on my mega 6.3 and acca works fine it seems. So it must be a device and rom issue. I'll follow up if there is a android 12 to test