#
9c9a810c |
|
14-Feb-2020 |
Jérôme Duval <jerome.duval@gmail.com> |
Media Kit: BBufferCache: if not reclaimed, only mark the buffer for deletion hrev53379 clears the buffer cache for disconnected clients, and also delete buffers. This is too early (see #15263, media_addon_server crash), and should only happen after the buffer is recycled. This can be resolved by abusing the fFlags field of BBuffer to mark the buffer for deletion, and mark the buffer to be reclaimed. Some BBuffers don't reside in the SharedBufferList, so we have to mark them as to be reclaimed. For those in the SharedBufferList, call a new RemoveBuffer(), which can check whether the buffer is still to be reclaimed. For reclaimed BBuffers, delete them right away, others can be marked for deletion. fixes #15606 #15263, possibly #15433 Change-Id: I66e94138e7e10a40d4c48e2ac042f816c79f5aab Reviewed-on: https://review.haiku-os.org/c/haiku/+/2245 Reviewed-by: Adrien Destugues <pulkomandy@gmail.com> Reviewed-by: X512 <danger_mail@list.ru> Reviewed-by: Ryan Leavengood <leavengood@gmail.com>
|
#
964cab46 |
|
10-Aug-2019 |
Augustin Cavalier <waddlesplash@gmail.com> |
media: Set B_CLONEABLE_AREA where applicable. Cloning cloned buffers seems especially strange, but apparently it is actually done. The buffer management code really needs to be cleaned up...
|
#
b84955d4 |
|
23-Oct-2018 |
Barrett17 <b.vitruvio@gmail.com> |
media: Remove debug.h, finalize MediaDebug.h * Funny rust from 0 AD.
|
#
33179216 |
|
11-Jun-2015 |
Dario Casalinuovo <b.vitruvio@gmail.com> |
BBuffer: initialize variables and cleanup * Restore the BSmallBuffer exception in constructor, it's not actually implemented or used but it should still be keep safe. * Initialize some variables in BBuffer and BBufferGroup.
|
#
1cc20d82 |
|
14-Apr-2015 |
Dario Casalinuovo <b.vitruvio@gmail.com> |
BBufferGroup: Check for duplicated buffer id * This exclude some possible cheating cases. It impose to have only a buffer_id per thread, so instances will not be duplicated if not needed.
|
#
332cc6bc |
|
30-Nov-2012 |
Axel Dörfler <axeld@pinc-software.de> |
64 bit fixes for libmedia.so, ActivityMonitor, and Sudoku. * Alex, I feel your pain :-)
|
#
2f9ed888 |
|
05-Dec-2009 |
Axel Dörfler <axeld@pinc-software.de> |
* The SharedBufferList is now only cloned once in a team, no longer once for each buffer, and once for each buffer group. * Also, SharedBufferList::Get() now gets the area to clone from itself, if necessary, the caller no longer has to provide it. git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@34502 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
9dec2310 |
|
05-Dec-2009 |
Axel Dörfler <axeld@pinc-software.de> |
Sorry for this large commit in advance; it's not really possible to divide this into smaller parts: media_addon_server: * Removed (broken) use of (broken and inefficient) home-brewn Map, and List classes. This also fixes a crash on shutdown when used with the malloc_debug implementation. It's using stl::vector, and stl::map now instead. _shared_buffer_list: * Renamed _shared_buffer_list to SharedBufferList, and put it into the BPrivate namespace. Also, made a class out of it. * Separated shared buffer list creation from cloning. * Enlarged maximum number of buffers to something that is not that evil, but actually uses the space it has (ie. is a useful multiple of shared_buffer_info that fills a multiple of B_PAGE_SIZE as much as possible). * No longer drops into the debugger if the * The list that is currently used is very inefficient for the features it provides though (no change there). _buffer_id_cache: * Renamed to BufferCache, and put it into the private namespace * It now deletes its buffers on deletion; since the BBufferConsumer will be gone, too, at this point, there is little chance that there are still buffers in use. * Also, it's now using std::map instead of the (see above) Map class. BBuffer: * Got rid of the fBufferID member. Misc.: * Got rid of the global "team" variable; the media kit is now using the private app kit's current_team() now. * Added a lot of missing error checks (mostly memory allocations). * Renamed fields like "flavorid" to flavor_id, renamed "dfi_*" fields to something more detailed. * Moved ServerInterface.h from src/servers/media/ to headers/private/media. * Notifications.h was not self contained. * Added missing licenses. * Lots of cleanups, and coding style fixes. What this doesn't fix: * Bug #4954 which started all this (this comes next, though) * Deinitialization is broken, as the PortPool is uninitialized too early, and still used afterwards. * The strange add-on monitoring code in the media_addon_server git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@34500 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
21b29fb2 |
|
11-Oct-2005 |
Jérôme Duval <korli@users.berlios.de> |
changed B_ANY_KERNEL_ADDRESS to B_ANY_ADDRESS, as our kernel doesn't allow user applications to create kernel areas added some error logs git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@14355 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
951c57f3 |
|
07-Sep-2003 |
beveloper <beveloper@nowhere.fake> |
used real buffer id (the buffer handling needs to be redone) git-svn-id: file:///srv/svn/repos/haiku/trunk/current@4546 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
5ac4fbd7 |
|
19-Jun-2003 |
beveloper <beveloper@nowhere.fake> |
clean up of debugging functions FATAL is now replaced by ERROR and active at DEBUG=1 or above TRACE is active at 2 or above PRINT(level, ...) can be used for DEBUG= level or abode INFO is removed git-svn-id: file:///srv/svn/repos/haiku/trunk/current@3578 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
9bedd42c |
|
29-May-2003 |
beveloper <beveloper@nowhere.fake> |
many fixes to improve timing git-svn-id: file:///srv/svn/repos/haiku/trunk/current@3386 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
76669a29 |
|
08-Dec-2002 |
beveloper <beveloper@nowhere.fake> |
changed use of port communication, debugging functions, map, list and stack git-svn-id: file:///srv/svn/repos/haiku/trunk/current@2196 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
353b9f6b |
|
02-Dec-2002 |
beveloper <beveloper@nowhere.fake> |
changed map and list templates to be more useable, however, they will be rewritten soon. Changed debugging macros and use of them, too. Also replaced the linked lists in the BufferManager (which were complicated, but working ok) with template based ones. git-svn-id: file:///srv/svn/repos/haiku/trunk/current@2133 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
41ba915d |
|
23-Nov-2002 |
beveloper <beveloper@nowhere.fake> |
various small changes git-svn-id: file:///srv/svn/repos/haiku/trunk/current@2071 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
52a38012 |
|
08-Jul-2002 |
ejakowatz <ejakowatz@nowhere.fake> |
It is accomplished ... git-svn-id: file:///srv/svn/repos/haiku/trunk/current@10 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
331792160ab973084132a7fff6a611ca6f66245d |
|
11-Jun-2015 |
Dario Casalinuovo <b.vitruvio@gmail.com> |
BBuffer: initialize variables and cleanup * Restore the BSmallBuffer exception in constructor, it's not actually implemented or used but it should still be keep safe. * Initialize some variables in BBuffer and BBufferGroup.
|
#
1cc20d8231bde733e30716f0980f19d8bf6b1d24 |
|
14-Apr-2015 |
Dario Casalinuovo <b.vitruvio@gmail.com> |
BBufferGroup: Check for duplicated buffer id * This exclude some possible cheating cases. It impose to have only a buffer_id per thread, so instances will not be duplicated if not needed.
|
#
332cc6bc3b696473169bb9e43340b69a0a3fc4f1 |
|
30-Nov-2012 |
Axel Dörfler <axeld@pinc-software.de> |
64 bit fixes for libmedia.so, ActivityMonitor, and Sudoku. * Alex, I feel your pain :-)
|
#
2f9ed888a24130579b97509580f351ce2620c9b8 |
|
05-Dec-2009 |
Axel Dörfler <axeld@pinc-software.de> |
* The SharedBufferList is now only cloned once in a team, no longer once for each buffer, and once for each buffer group. * Also, SharedBufferList::Get() now gets the area to clone from itself, if necessary, the caller no longer has to provide it. git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@34502 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
9dec23104287e2fd592cc14148b43fb28a6e1d8e |
|
05-Dec-2009 |
Axel Dörfler <axeld@pinc-software.de> |
Sorry for this large commit in advance; it's not really possible to divide this into smaller parts: media_addon_server: * Removed (broken) use of (broken and inefficient) home-brewn Map, and List classes. This also fixes a crash on shutdown when used with the malloc_debug implementation. It's using stl::vector, and stl::map now instead. _shared_buffer_list: * Renamed _shared_buffer_list to SharedBufferList, and put it into the BPrivate namespace. Also, made a class out of it. * Separated shared buffer list creation from cloning. * Enlarged maximum number of buffers to something that is not that evil, but actually uses the space it has (ie. is a useful multiple of shared_buffer_info that fills a multiple of B_PAGE_SIZE as much as possible). * No longer drops into the debugger if the * The list that is currently used is very inefficient for the features it provides though (no change there). _buffer_id_cache: * Renamed to BufferCache, and put it into the private namespace * It now deletes its buffers on deletion; since the BBufferConsumer will be gone, too, at this point, there is little chance that there are still buffers in use. * Also, it's now using std::map instead of the (see above) Map class. BBuffer: * Got rid of the fBufferID member. Misc.: * Got rid of the global "team" variable; the media kit is now using the private app kit's current_team() now. * Added a lot of missing error checks (mostly memory allocations). * Renamed fields like "flavorid" to flavor_id, renamed "dfi_*" fields to something more detailed. * Moved ServerInterface.h from src/servers/media/ to headers/private/media. * Notifications.h was not self contained. * Added missing licenses. * Lots of cleanups, and coding style fixes. What this doesn't fix: * Bug #4954 which started all this (this comes next, though) * Deinitialization is broken, as the PortPool is uninitialized too early, and still used afterwards. * The strange add-on monitoring code in the media_addon_server git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@34500 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
21b29fb2efe1caa2b84382dc7f80171b7dbd82ed |
|
11-Oct-2005 |
Jérôme Duval <korli@users.berlios.de> |
changed B_ANY_KERNEL_ADDRESS to B_ANY_ADDRESS, as our kernel doesn't allow user applications to create kernel areas added some error logs git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@14355 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
951c57f390204454d43dfabb1df82414226618dc |
|
07-Sep-2003 |
beveloper <beveloper@nowhere.fake> |
used real buffer id (the buffer handling needs to be redone) git-svn-id: file:///srv/svn/repos/haiku/trunk/current@4546 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
5ac4fbd70dc5b5387cc80965de796deb820d4f05 |
|
19-Jun-2003 |
beveloper <beveloper@nowhere.fake> |
clean up of debugging functions FATAL is now replaced by ERROR and active at DEBUG=1 or above TRACE is active at 2 or above PRINT(level, ...) can be used for DEBUG= level or abode INFO is removed git-svn-id: file:///srv/svn/repos/haiku/trunk/current@3578 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
9bedd42cd1e0fdb44e39be7ef4f3c76786d59b50 |
|
29-May-2003 |
beveloper <beveloper@nowhere.fake> |
many fixes to improve timing git-svn-id: file:///srv/svn/repos/haiku/trunk/current@3386 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
76669a292a7a9cdf48535ff3fe91db5de31ed35e |
|
08-Dec-2002 |
beveloper <beveloper@nowhere.fake> |
changed use of port communication, debugging functions, map, list and stack git-svn-id: file:///srv/svn/repos/haiku/trunk/current@2196 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
353b9f6bcedb3fa9950f3f65c960e1deb700f84a |
|
02-Dec-2002 |
beveloper <beveloper@nowhere.fake> |
changed map and list templates to be more useable, however, they will be rewritten soon. Changed debugging macros and use of them, too. Also replaced the linked lists in the BufferManager (which were complicated, but working ok) with template based ones. git-svn-id: file:///srv/svn/repos/haiku/trunk/current@2133 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
41ba915d4209230c44208bc7480ea76532b984c9 |
|
23-Nov-2002 |
beveloper <beveloper@nowhere.fake> |
various small changes git-svn-id: file:///srv/svn/repos/haiku/trunk/current@2071 a95241bf-73f2-0310-859d-f6bbb57e9c96
|
#
52a380120846174213ccce9c4aab0dda17c72083 |
|
08-Jul-2002 |
ejakowatz <ejakowatz@nowhere.fake> |
It is accomplished ... git-svn-id: file:///srv/svn/repos/haiku/trunk/current@10 a95241bf-73f2-0310-859d-f6bbb57e9c96
|