2015-02-25, 02:26:45 -0800, Eric Dumazet wrote:
> On Wed, 2015-02-25 at 11:05 +0100, Sabrina Dubroca wrote:
> > There is a race condition between e1000_change_mtu's cleanups and
> > netpoll, when we change the MTU across jumbo size:
> ...
> >
> > Fixes: edbbb3ca1077 ("e1000: implement jumbo receive with partial
> > descriptors")
> > Signed-off-by: Sabrina Dubroca <s...@queasysnail.net>
> > ---
> > drivers/net/ethernet/intel/e1000/e1000_main.c | 14 +++++++++++++-
> > 1 file changed, 13 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/ethernet/intel/e1000/e1000_main.c
> > b/drivers/net/ethernet/intel/e1000/e1000_main.c
> > index 7f997d36948f..5af866150671 100644
> > --- a/drivers/net/ethernet/intel/e1000/e1000_main.c
> > +++ b/drivers/net/ethernet/intel/e1000/e1000_main.c
> > @@ -144,6 +144,9 @@ static bool e1000_clean_rx_irq(struct e1000_adapter
> > *adapter,
> > static bool e1000_clean_jumbo_rx_irq(struct e1000_adapter *adapter,
> > struct e1000_rx_ring *rx_ring,
> > int *work_done, int work_to_do);
> > +static void e1000_alloc_dummy_rx_buffers(struct e1000_adapter *adapter,
> > + struct e1000_rx_ring *rx_ring,
> > + int cleaned_count);
>
> Why are you using a forward declaration ?
>
> Simply put here the body of this (dummy) function.
I wanted to match the style for the other allocators. I'll send a v2.
Thanks,
--
Sabrina
------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel® Ethernet, visit
http://communities.intel.com/community/wired
