All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jgarzik@mandrakesoft.com>
To: "David S. Miller" <davem@redhat.com>
Cc: ink@jurassic.park.msu.ru, andrew.grover@intel.com,
	mochel@osdl.org, Greg@kroah.com, linux-kernel@vger.kernel.org
Subject: Re: pci segments/domains
Date: Fri, 17 May 2002 10:33:26 -0400	[thread overview]
Message-ID: <3CE514B6.6070302@mandrakesoft.com> (raw)
In-Reply-To: <3CE4098E.2070808@mandrakesoft.com>	<20020517144755.A16767@jurassic.park.msu.ru>	<3CE512A7.70202@mandrakesoft.com> <20020517.071633.67125480.davem@redhat.com>

David S. Miller wrote:

>   From: Jeff Garzik <jgarzik@mandrakesoft.com>
>   Date: Fri, 17 May 2002 10:24:39 -0400
>
>   I know -- that's what I mean by being more explicit.  sysdata would 
>   become a pointer to struct pci_domain.
>   
>No thanks, I want to say what the layout is for
>this object.  What pci_domain will end up doing is
>making for one more dereference to "arch private"
>state and that stinks for performance :-)
>


See my previous message from the other day... this would be defined in 
each arch's asm/pci.h, which eliminates this problem you describe.  Each 
arch maintainer would indeed decide how to define it, though over time 
I'm sure it would grow commonly-named struct members.

My main want is cosmetic -- call a spade a spade, so to speak. 
 s/sysdata/pci_domain/  But doing so opens the door to increased 
flexibility.  Later steps can add common members needed by pci-to-pci 
IOMMU tricks which are common to most platforms.

    Jeff




  reply	other threads:[~2002-05-17 14:35 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-05-14  2:07 pci segments/domains Grover, Andrew
2002-05-14  2:08 ` David S. Miller
2002-05-16 19:33 ` Jeff Garzik
2002-05-17  1:31   ` David S. Miller
2002-05-16 23:10     ` Pavel Machek
2002-05-17 10:47   ` Ivan Kokshaysky
2002-05-17 10:40     ` David S. Miller
2002-05-17 11:11       ` Ivan Kokshaysky
2002-05-17 11:04         ` David S. Miller
2002-05-17 11:39           ` Ivan Kokshaysky
2002-05-17 11:41             ` David S. Miller
2002-05-17 12:26               ` Ivan Kokshaysky
2002-05-17 12:15                 ` David S. Miller
2002-05-17 14:24     ` Jeff Garzik
2002-05-17 14:16       ` David S. Miller
2002-05-17 14:33         ` Jeff Garzik [this message]
2002-05-17 14:26           ` David S. Miller
2002-05-17 14:42             ` Jeff Garzik
2002-05-17 16:24               ` Ivan Kokshaysky
2002-05-17 16:53                 ` David S. 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=3CE514B6.6070302@mandrakesoft.com \
    --to=jgarzik@mandrakesoft.com \
    --cc=Greg@kroah.com \
    --cc=andrew.grover@intel.com \
    --cc=davem@redhat.com \
    --cc=ink@jurassic.park.msu.ru \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mochel@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 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.