| commit | 1ba4e981dbb75985b320c2d97c7c2567b9ebcf51 | [log] [tgz] |
|---|---|---|
| author | Jeff Vander Stoep <jeffv@google.com> | Fri Oct 21 14:29:22 2016 -0700 |
| committer | Jeff Vander Stoep <jeffv@google.com> | Fri Oct 21 14:29:22 2016 -0700 |
| tree | fc2740de9ee96e209fa4db4ca215aabfc70fac4b | |
| parent | 27d2d49f4872c26831d3e016765c62def3d2fb01 [diff] |
libion: open /dev/ion read-only /dev/ion driver does not recognize the write operation [1] so open it read-only. [1] http://lxr.free-electrons.com/source/drivers/staging/android/ion/ion.c?v=4.5#L1369 Test: Angler builds and boots Bug: 32120194 Change-Id: I9b26d43ea32cb24426404668701df1f1648d336c
diff --git a/libion/ion.c b/libion/ion.c index 4565efb..2db8845 100644 --- a/libion/ion.c +++ b/libion/ion.c
@@ -34,7 +34,7 @@ int ion_open() { - int fd = open("/dev/ion", O_RDWR); + int fd = open("/dev/ion", O_RDONLY); if (fd < 0) ALOGE("open /dev/ion failed!\n"); return fd;
diff --git a/libion/tests/device_test.cpp b/libion/tests/device_test.cpp index 0be52bf..eb3f7b6 100644 --- a/libion/tests/device_test.cpp +++ b/libion/tests/device_test.cpp
@@ -46,7 +46,7 @@ void Device::SetUp() { IonAllHeapsTest::SetUp(); - m_deviceFd = open("/dev/ion-test", O_RDWR); + m_deviceFd = open("/dev/ion-test", O_RDONLY); ASSERT_GE(m_deviceFd, 0); }