From: Joseph Salisbury <joseph.salisbury@canonical.com>
To: Pravin Shelar <pshelar@nicira.com>
Cc: davem@davemloft.net, ben.howard@canonical.com,
edumazet@google.com, xiyou.wangcong@gmail.com,
alexander.h.duyck@intel.com, jiri@resnulli.us,
netdev@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>,
kys@microsoft.com, haiyangz@microsoft.com
Subject: Re: [Regression][v3.10][v3.11] net: Kill link between CSUM and SG features.
Date: Tue, 16 Jul 2013 15:53:46 -0400 [thread overview]
Message-ID: <51E5A4CA.20902@canonical.com> (raw)
In-Reply-To: <CALnjE+oCeo=T+F3BvTBtcV7oDMPLJAEz1p1sjRGxmE8Wq8gWAg@mail.gmail.com>
On 07/16/2013 02:29 PM, Pravin Shelar wrote:
> On Tue, Jul 16, 2013 at 11:09 AM, Joseph Salisbury
> <joseph.salisbury@canonical.com> wrote:
>> Hi Pravin,
>>
>> A bug was opened against the Ubuntu kernel[0]. After a kernel bisect,
>> it was found that reverting the following commit resolved this bug:
>>
>> commit ec5f061564238892005257c83565a0b58ec79295
>> Author: Pravin B Shelar <pshelar@nicira.com>
>> Date: Thu Mar 7 09:28:01 2013 +0000
>>
>> net: Kill link between CSUM and SG features.
>>
>>
>> The regression was introduced as of v3.10-rc1.
>>
>> I see that you are the author of this patch, so I wanted to run this by
>> you. I was thinking of requesting a revert for v3.11, but I wanted to
>> get your feedback first.
>>
>>
> This issue was reported before and turning off sg solves problem for
> HyperV driver. Drivers (intel and bnx2x) that I tested works fine with
> this patch. Have you seen the issue on any other NIC?
Thanks for the feedback. We have only seen it on Intel hardware. Can
you send a pointer to the patch? If so, we can provide some additional
testing. Do you plan on submitting the patch for inclusion in 3.11?
>
>
>> Thanks,
>>
>> Joe
>>
>> [0] http://pad.lv/1200243
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe netdev" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2013-07-16 19:54 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-16 18:09 [Regression][v3.10][v3.11] net: Kill link between CSUM and SG features Joseph Salisbury
2013-07-16 18:29 ` Pravin Shelar
2013-07-16 19:53 ` Joseph Salisbury [this message]
2013-07-16 20:50 ` Pravin Shelar
2013-07-17 15:30 ` Joseph Salisbury
2013-07-17 17:45 ` Haiyang Zhang
2013-07-17 18:22 ` Joseph Salisbury
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=51E5A4CA.20902@canonical.com \
--to=joseph.salisbury@canonical.com \
--cc=alexander.h.duyck@intel.com \
--cc=ben.howard@canonical.com \
--cc=davem@davemloft.net \
--cc=edumazet@google.com \
--cc=haiyangz@microsoft.com \
--cc=jiri@resnulli.us \
--cc=kys@microsoft.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=pshelar@nicira.com \
--cc=xiyou.wangcong@gmail.com \
/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.