Messages in this thread | | | Date | Thu, 13 Nov 2014 11:46:04 -0600 | From | Suman Anna <> | Subject | Re: [PATCHv2] rpmsg: compute number of buffers to allocate from vrings |
| |
Hi Ohad,
On 09/16/2014 01:33 PM, Suman Anna wrote: > The buffers to be used for communication are allocated during the > rpmsg virtio driver's probe, and the total number of buffers is > currently hard-coded to 512. The vring configuration can vary from > one platform to another or between different remote processors. The > setup of the receive buffers will throw a WARN_ON if the associated > vrings are configured with less than 256 buffers (in each direction). > So, adjust this hard-coded value to rely on the number of buffers the > virtqueue vring is setup with, but also limit to use 256 buffers at > most in each direction to avoid wacky resource tables consuming up > unreasonable memory. > > NOTE: The number of buffers is already assumed to be symmetrical > in each direction, and that logic is not unchanged. > > Signed-off-by: Suman Anna <s-anna@ti.com> > --- > v2 changes: > - add upper limit on buffers and update comments > - revise patch description
Any comments on this one, if not can you pick this up for 3.19?
regards Suman
| |