#1518: Bdbuf IO request cleanup -----------------------------+----------------------- Reporter: sebastian.huber | Owner: chrisj Type: defect | Status: assigned Priority: normal | Milestone: 4.11 Component: cpukit | Version: 4.10 Severity: normal | Resolution: Keywords: | -----------------------------+----------------------- Changes (by joel.sherrill):
* owner: joel.sherrill => chrisj * status: new => assigned Old description: > Reported by Chris Johns in Bug 1433: > > We need to remove the SG alloc calls from the cache read code. The code > currently allocates a block of RAM on the stack with an array of pointers > to > the BD. I consider the array of size of 1 at the end of the struct and > the way > it is allocated as fragile code. The BD already contain a link node which > is > not used at the driver layer. It would be much better to use a chain to > hold a > list of BDs to fill. New description: Reported by Chris Johns in Bug 1433: We need to remove the SG alloc calls from the cache read code. The code currently allocates a block of RAM on the stack with an array of pointers to the BD. I consider the array of size of 1 at the end of the struct and the way it is allocated as fragile code. The BD already contain a link node which is not used at the driver layer. It would be much better to use a chain to hold a list of BDs to fill. -- Comment: This is Chris' area of the code. -- Ticket URL: <http://devel.rtems.org/ticket/1518#comment:4> RTEMS Project <http://www.rtems.org/> RTEMS Project _______________________________________________ bugs mailing list bugs@rtems.org http://lists.rtems.org/mailman/listinfo/bugs