[RTEMS Project] #1518: Bdbuf IO request cleanup

[RTEMS trac]

#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