| commit | 57d858215f982c542bd8f4463c190a1e2986b273 | [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 | c171b9ea7674feab9aaf0cef903cf0093ee1d6f9 [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); }