netdev - RE: [PATCH] r8169: default to 64-bit DMA on systems without memory below 4 GB

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 for Android: free password hash cracker in your pocket

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

Message-ID: <80377ECBC5453840BA8C7155328B5377013C3DF6@RTITMBSV03.realtek.com.tw>
Date:	Fri, 13 May 2016 13:10:38 +0000
From:	Hau <hau@...ltek.com>
To:	Francois Romieu <romieu@...zoreil.com>,
	Ard Biesheuvel <ard.biesheuvel@...aro.org>
CC:	David Miller <davem@...emloft.net>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	Ricardo Salveti <ricardo.salveti@...aro.org>,
	"Leo Duran" <leo.duran@....com>,
	G Gregory <graeme.gregory@...aro.org>,
	nic_swsd <nic_swsd@...ltek.com>
Subject: RE: [PATCH] r8169: default to 64-bit DMA on systems without memory below 4 GB

> -----Original Message-----
> From: Francois Romieu [mailto:romieu@...zoreil.com]
> Sent: Thursday, May 12, 2016 10:09 PM
> To: Ard Biesheuvel <ard.biesheuvel@...aro.org>
> Cc: David Miller <davem@...emloft.net>; netdev@...r.kernel.org; Ricardo
> Salveti <ricardo.salveti@...aro.org>; Leo Duran <leo.duran@....com>; G
> Gregory <graeme.gregory@...aro.org>; nic_swsd <nic_swsd@...ltek.com>;
> Hau <hau@...ltek.com>
> Subject: Re: [PATCH] r8169: default to 64-bit DMA on systems without memory
> below 4 GB
> 
> > On 12 May 2016 at 01:58, David Miller <davem@...emloft.net> wrote:
> > > From: Ard Biesheuvel <ard.biesheuvel@...aro.org>
> > > Date: Wed, 11 May 2016 09:47:49 +0200
> [...]
> > > I think we should just seriously consider changing the default, it's
> > > a really outdated reasoning behind the current default setting.
> > > Maybe relevant a decade ago, but probably not now.
> > >
> > > And if the card is completely disfunctional in said configuration,
> > > the default is definitely wrong.
> >
> > The card is indeed completely disfunctional. So we could try to
> > resurrect 353176888386 ("r8169: enable 64-bit DMA by default for PCI
> > Express devices"), and instead of backing it out again if regressions
> > are reported, blacklist the particular chips. This is a much better
> > approach, since then we can also print some kind of diagnostic on
> > those arm64 systems why such a blacklisted NIC is not supported.
> 
> I doubt there will be much *reporting* from broken systems that include plain
> old PCI realtek chipsets (r8169.c::RTL_CFG_0). Changing the default for those is
> imnsho asking for troubles without clear benefit (experimental evidence
> suggests that smsc etherpower II grows older more easily than plain pci
> 8169 :o/ ).
> 
> I'd rather leave these alone and change the default for the PCI Express chipsets.
> Btw, while it does not seem to hurt, they should not need any CPlusCmd Dual
> Access Cycle tweak either. Realtek may establish it (Lin ?)

You don't have to set CPlusCmd Dual Access Cycle for the PCI Express chipsets.

------Please consider the environment before printing this e-mail.