Thread (10 messages) 10 messages, 3 authors, 2015-02-05

Re: [PATCH net-next 3/4] ethtool: add RX_ALLOC_ORDER to tunable

From: Govindarajulu Varadarajan <hidden>
Date: 2015-02-05 17:29:20

On Tue, 3 Feb 2015, Govindarajulu Varadarajan wrote:
On Mon, 2 Feb 2015, David Miller wrote:
quoted
From: Govindarajulu Varadarajan <redacted>
Date: Sat, 31 Jan 2015 17:58:09 +0530
quoted
Signed-off-by: Govindarajulu Varadarajan <redacted>
This is terrible.

You haven't explained what this means.

And to tell you the truth, from what I can tell this tunable is
very specific to how you have implemented RX frags in the enic
driver in this series and won't necessarily translate to how
other drivers manage RX buffers.
Yes I agree that this is too specific to driver. From my knowledge mlx4 also
uses similar frag allocation from large order page. Other that these two
drivers, this facility may not make sense to other drivers.

On systems with huge memory we can go higher than order-3, and if user sees 
that
order-3 are failing, he should be able to reduce the order.

Since this is very driver specific, are you fine if I move this to device 
sysfs?
I hope there are no issues with the patch 1 & 2. Should I drop the
'changing order' patches and resend patch 1 & 2 alone?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help