netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andi Kleen <ak@muc.de>
To: Francois Romieu <romieu@fr.zoreil.com>
Cc: Stephen Hemminger <shemminger@osdl.org>, netdev@oss.sgi.com
Subject: Re: [PATCH 1/5] r8169: auto detect 32-bit slot
Date: 10 Mar 2005 00:20:24 +0100
Date: Thu, 10 Mar 2005 00:20:24 +0100	[thread overview]
Message-ID: <20050309232024.GC63395@muc.de> (raw)
In-Reply-To: <20050309230809.GC9502@electric-eye.fr.zoreil.com>

On Thu, Mar 10, 2005 at 12:08:09AM +0100, Francois Romieu wrote:
> Stephen Hemminger <shemminger@osdl.org> :
> [...]
> > I tested on old Celeron with Netgear card.  I could try Amd and/or Xeon
> > if you want.
> 
> If the systems are 64 bit, one would need to know how the driver behaves
> when DAC is forced and:

The concept of "64bit systems" is useless here anyways. Even on a 32bit
system with highmem like i386 you can get >32bit addresses to deal
with. One usecase for that would be sendfile() out of user space.
This is widely used these days.

> 1) the Netgear card is in a 32 bit slot, dma to highmem is enabled and used
> 2) the Netgear card is in a 32 bit slot, dma to highmem is enabled and unused
> 3) the Netgear card is in a 32 bit slot, dma to highmem is disabled
> 4) the Netgear card is in a 64 bit slot, dma to highmem is enabled and used
> 5) the Netgear card is in a 64 bit slot, dma to highmem is enabled and unused
> 6) the Netgear card is in a 64 bit slot, dma to highmem is disabled

Again are you sure it depends on 64bit slots? Normally 64bit slots
only offer more bandwidth, but the address protocol is the same
as 32bit and both support DAC in the same way.

> 
> + the content of the Config2 register for 1-6.
> 
> If the amd64 hosts a built-in 8169, add the behavior with highmem disabled or
> enabled + variable amount of memory.

You really should forget about amd64 or not, all that counts is
sizeof(dma_addr_t) 

-Andi

  parent reply	other threads:[~2005-03-09 23:20 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-09 19:29 [PATCH 1/5] r8169: auto detect 32-bit slot Stephen Hemminger
2005-03-09 19:47 ` Jeff Garzik
2005-03-09 20:16 ` Andi Kleen
2005-03-09 21:40   ` Francois Romieu
2005-03-09 21:55     ` Stephen Hemminger
2005-03-09 23:08       ` Francois Romieu
2005-03-09 23:16         ` Stephen Hemminger
2005-03-09 23:20         ` Andi Kleen [this message]
2005-03-10  0:45           ` Francois Romieu
2005-03-09 22:08     ` Jon Mason
2005-03-09 22:39       ` Francois Romieu
2005-03-10  4:14         ` Jon Mason
2005-03-10 11:17           ` Francois Romieu

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=20050309232024.GC63395@muc.de \
    --to=ak@muc.de \
    --cc=netdev@oss.sgi.com \
    --cc=romieu@fr.zoreil.com \
    --cc=shemminger@osdl.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).