linux-kernel - RE: [E1000-devel] [PATCH] e1000e: disable rxhash when try to enable jumbo frame also rxhash and rxcsum have enabled

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <804857E1F29AAC47BF68C404FC60A18428445A22@ORSMSX102.amr.corp.intel.com>
Date:	Fri, 8 Jun 2012 01:14:38 +0000
From:	"Allan, Bruce W" <bruce.w.allan@...el.com>
To:	David Miller <davem@...emloft.net>,
	"joe.jin@...cle.com" <joe.jin@...cle.com>
CC:	"e1000-devel@...ts.sourceforge.net" 
	<e1000-devel@...ts.sourceforge.net>,
	"guru.anbalagane@...cle.com" <guru.anbalagane@...cle.com>,
	"Brandeburg, Jesse" <jesse.brandeburg@...el.com>,
	"adnan.misherfi@...cle.com" <adnan.misherfi@...cle.com>,
	"Ronciak, John" <john.ronciak@...el.com>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [E1000-devel] [PATCH] e1000e: disable rxhash when try to enable
 jumbo frame also rxhash and rxcsum have enabled

> -----Original Message-----
> From: David Miller [mailto:davem@...emloft.net]
> Sent: Thursday, June 07, 2012 6:12 PM
> To: joe.jin@...cle.com
> Cc: e1000-devel@...ts.sourceforge.net; guru.anbalagane@...cle.com;
> Allan, Bruce W; Brandeburg, Jesse; adnan.misherfi@...cle.com; Ronciak,
> John; netdev@...r.kernel.org; linux-kernel@...r.kernel.org
> Subject: Re: [E1000-devel] [PATCH] e1000e: disable rxhash when try to
> enable jumbo frame also rxhash and rxcsum have enabled
> 
> From: Joe Jin <joe.jin@...cle.com>
> Date: Fri, 08 Jun 2012 09:06:40 +0800
> 
> > Upstream commit 70495a5 check if both rxhash and rxcsum enabled when
> enabling
> > jumbo frames and disallowed all of them enabled at the same time.
> >
> > Since jumbo frame widely be used in real world, so when try to
> enable jumbo
> > frames but rxhash and rxcsum have enabled, change the default
> behavior to
> > disable receive hashing.
> >
> > Signed-off-by: Joe Jin <joe.jin@...cle.com>
> > Signed-off-by: Guru Anbalagane <guru.anbalagane@...cle.com>
> > Acked-by: Adnan Misherfi <adnan.misherfi@...cle.com>
> 
> If I were the Intel developers I would not apply this patch, it sets
> a very bad precedence.
> 
> The tool tells you that the combination you're attempting to use is
> invalid, and the kernel log message tells you exactly why.
> 
> The driver should never automatically change configuration settings
> not actually requested by the user.

I've been working on another patch that removes the issue with disabling
one feature over another altogether, so this patch would be moot.  It will
be pushed to our internal testing organization in the next day or two.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/