Julian Calaby <[email protected]> writes: > On Tue, Jan 16, 2018 at 12:18 AM, <[email protected]> wrote: >> On 2018-01-09 10:07, Kalle Valo wrote: >>> >>> Maya Erez <[email protected]> wrote: >>> >>>> Add module parameter for configuring the headroom size >>>> in the skb allocation. >>>> >>>> Signed-off-by: Lazar Alexei <[email protected]> >>>> Signed-off-by: Maya Erez <[email protected]> >>>> Signed-off-by: Kalle Valo <[email protected]> >>> >>> >>> Why? >> >> >> Some platforms have specific requirements on packet alignment. >> I will upload an update of the commit text in the next set of 11ad patches. > > Is this something the platform can tell you or something you can store > in an array by platform name? (and use the maximum size if we don't > know) > > Alternatively, would it waste too much RAM to just set this to the > maximum size and have done? > > I feel that allowing the user to set it will be problematic.
I feel the same. I don't think this is something which should be controlled with a module parameter, instead the driver should choose it automatically. -- Kalle Valo
