lkml.org 
[lkml]   [2013]   [May]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v3 net-next 3/4] ixgbe: Add support for ndo_ll_poll
    From
    Date
    On Tue, 2013-05-21 at 11:21 +0300, Or Gerlitz wrote:
    > On Tue, May 21, 2013 at 10:06 AM, Eilon Greenstein <eilong@broadcom.com> wrote:
    > > Or - at least for the bnx2x, it is easy to add support for this new ndo.
    >
    > Do you understand what's the equivalent of that mysterious module
    > param for your driver/HW - or you just copied and pasted that black
    > magic code?
    >

    The module parameter is not the interesting part of this patch. It is
    clear that unloading the module while this sort of traffic is running is
    not safe and the alternative of adding a reference count or something
    similar sounds too costly (after all, this patch is about performance).
    I just played with it to get a feel of the latency improvement - I did
    not try unloading the module during traffic so I do not care about the
    module parameter part right now. I agree that the unload should be
    looked into - but the general concept is great and it is a very nice
    improvement.





    \
     
     \ /
      Last update: 2013-05-21 10:41    [W:5.727 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site