Skip to content

Commit

Permalink
wahoo: ueventd: Don't modify permissions of /dev/hw_random
Browse files Browse the repository at this point in the history
This addresses an underlying problem since /dev/hw_random is used. The
issue is that EntropyMixer, which runs in system_server, needs to read
from /dev/hw_random, but thanks to our qualcomm ueventd, EntropyMixer
can never accress this node, resulting in warnings like this:

W EntropyMixer: Failed to add HW RNG output to entropy pool
W EntropyMixer: java.io.FileNotFoundException: /dev/hw_random (Permission denied)

AOSP ueventd handles this well, so just remove this line to get things
back to stage. This issue was never exposed because Google derps the
logging here, which is already fixed on Pie.

https://android.googlesource.com/platform/frameworks/base/+/c78a463e875d91eae2cf256042fd99d130217c38

Change-Id: I42c9aaa8f57306536e51c0a17febad57a5a77581
  • Loading branch information
aviraxp authored and Eamo5 committed Aug 5, 2020
1 parent caa07a9 commit 7f08377
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion ueventd.hardware.rc
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@
/dev/jpeg0 0660 system camera
/dev/adsprpc-smd 0664 system system

/dev/hw_random 0600 root root
/dev/sdsprpc-smd 0660 system system

/dev/wcd-dsp-glink 0660 system audio
Expand Down

0 comments on commit 7f08377

Please sign in to comment.