NBS (--nbuffers=0) causes software clients to crash with std::exception::what: Failed to mmap buffer 13, "Permission denied")
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mir |
Fix Released
|
High
|
Kevin DuBois | ||
mir (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
with both mesa-x11 and mesa-kms:
$ bin/mir_demo_server --nbuffers=0 --test-client bin/mir_
...
[2016-02-26 17:30:14.600378] <ERROR> MirBufferStreamAPI: Caught exception at client library boundary (in mir_buffer_
Dynamic exception type: boost::
std::exception:
13, "Permission denied"
[2016-02-26 17:30:14.600724] <ERROR> MirBufferStreamAPI: Caught exception at client library boundary (in mir_buffer_
Dynamic exception type: boost::
std::exception:
13, "Permission denied"
[2016-02-26 17:30:14.601024] <ERROR> MirBufferStreamAPI: Caught exception at client library boundary (in mir_buffer_
Dynamic exception type: boost::
std::exception:
13, "Permission denied"
Related branches
- Alan Griffiths: Approve
- Mir CI Bot: Approve (continuous-integration)
- PS Jenkins bot (community): Needs Fixing (continuous-integration)
- Alexandros Frantzis (community): Approve
- Cemil Azizoglu (community): Approve
-
Diff: 587 lines (+173/-52)17 files modifiedcmake/MirCommon.cmake (+14/-5)
include/test/mir_test_framework/placement_applying_shell.h (+12/-0)
src/client/buffer_stream.cpp (+25/-9)
src/client/buffer_vault.cpp (+15/-3)
src/client/mir_connection.cpp (+7/-1)
src/client/no_tls_future-inl.h (+5/-1)
src/client/rpc/mir_protobuf_rpc_channel.cpp (+21/-5)
src/server/compositor/stream.cpp (+1/-1)
tests/acceptance-tests/CMakeLists.txt (+1/-0)
tests/acceptance-tests/test_client_cookie.cpp (+8/-5)
tests/acceptance-tests/test_client_input.cpp (+4/-9)
tests/acceptance-tests/test_client_scaling.cpp (+1/-6)
tests/acceptance-tests/test_latency.cpp (+20/-5)
tests/acceptance-tests/test_nested_mir.cpp (+1/-1)
tests/mir_test_framework/placement_applying_shell.cpp (+18/-0)
tests/unit-tests/client/test_buffer_vault.cpp (+18/-1)
tests/unit-tests/client/test_client_buffer_stream.cpp (+2/-0)
Changed in mir: | |
status: | Invalid → In Progress |
assignee: | nobody → Kevin DuBois (kdub) |
milestone: | none → 0.21.0 |
importance: | Undecided → High |
tags: | added: nbs |
summary: |
- --nbuffers=0 doesn't work with existing clients + NBS (--nbuffers=0) doesn't work with existing clients |
summary: |
- NBS (--nbuffers=0) doesn't work with existing software clients + NBS (--nbuffers=0) doesn't work with existing software clients (they + crash with std::exception::what: Failed to mmap buffer 13, "Permission + denied") |
summary: |
- NBS (--nbuffers=0) doesn't work with existing software clients (they - crash with std::exception::what: Failed to mmap buffer 13, "Permission - denied") + NBS (--nbuffers=0) causes software clients to crash with + std::exception::what: Failed to mmap buffer 13, "Permission denied") |
Changed in mir: | |
status: | Fix Committed → Fix Released |
It's because there are bits yet to land, particularly in lp:~kdub/mir/cover-nbs-with-acceptance-tests. Merge it and it'll work.