All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Miller <davem@davemloft.net>
To: herbert@gondor.apana.org.au
Cc: emil.s.tantilov@intel.com, alexander.h.duyck@intel.com,
	jesse.brandeburg@intel.com, jeffrey.t.kirsher@intel.com,
	netdev@vger.kernel.org
Subject: Re: Kernel panic during stress with igb in the upstream kernel
Date: Sun, 15 Mar 2009 20:13:53 -0700 (PDT)	[thread overview]
Message-ID: <20090315.201353.57364397.davem@davemloft.net> (raw)
In-Reply-To: <20090313115245.GA31949@gondor.apana.org.au>

From: Herbert Xu <herbert@gondor.apana.org.au>
Date: Fri, 13 Mar 2009 19:52:45 +0800

> Sorry, it seems that I've managed to get the net/net-next patches
> mixed up.  Does this help?

Actually I introduced this problem when I did the net-2.6
--> net-next-2.6 merge when your netpoll fix went it. :-/

> GRO: Move netpoll checks to correct location
> 
> As my netpoll fix for net doesn't really work for net-next, we
> need this update to move the checks into the right place.  As it
> stands we may pass freed skbs to netpoll_receive_skb.
> 
> This patch also introduces a netpoll_rx_on function to avoid GRO
> completely if we're invoked through netpoll.  This might seem
> paranoid but as netpoll may have an external receive hook it's
> better to be safe than sorry.  I don't think we need this for
> 2.6.29 though since there's nothing immediately broken by it.
> 
> This patch also moves the GRO_* return values to netdevice.h since
> VLAN needs them too (I tried to avoid this originally but alas
> this seems to be the easiest way out).  This fixes a bug in VLAN
> where it continued to use the old return value 2 instead of the
> correct GRO_DROP.
> 
> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

Applied, thanks!

  parent reply	other threads:[~2009-03-16  3:14 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <EA929A9653AAE14F841771FB1DE5A1365F69CA4280@rrsmsx501.amr.corp.intel.com>
2009-03-13 11:52 ` FW: Kernel panic during stress with igb in the upstream kernel Herbert Xu
2009-03-13 21:53   ` Tantilov, Emil S
2009-03-13 21:54     ` David Miller
2009-03-13 23:59       ` Herbert Xu
2009-03-16  3:13   ` David Miller [this message]
2009-03-16  3:18     ` David Miller
2009-03-16 13:22       ` Herbert Xu
2009-03-16 17:50         ` David Miller
2009-03-17  9:37         ` Jarek Poplawski
2009-03-17 10:02           ` Herbert Xu
2009-03-17 20:11             ` David Miller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090315.201353.57364397.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=alexander.h.duyck@intel.com \
    --cc=emil.s.tantilov@intel.com \
    --cc=herbert@gondor.apana.org.au \
    --cc=jeffrey.t.kirsher@intel.com \
    --cc=jesse.brandeburg@intel.com \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.