linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Matthew Wilcox <matthew@wil.cx>
To: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: grundler@parisc-linux.org, mingo@elte.hu, tglx@linutronix.de,
	jgarzik@pobox.com, linux-ide@vger.kernel.org,
	suresh.b.siddha@intel.com, benh@kernel.crashing.org,
	jbarnes@virtuousgeek.org, rdunlap@xenotime.net,
	mtk.manpages@gmail.com
Subject: Multiple MSI, take 4
Date: Fri, 11 Jul 2008 15:16:00 -0600	[thread overview]
Message-ID: <20080711211559.GV14894@parisc-linux.org> (raw)


Here we go with take 4.  Changes:

 - Check the requested number of interrupts against the maximum number
   the device claims to support.  Thanks to Hidetoshi Seto for pointing
   out this oversight.
 - Implemented Eric's suggestion of using a single IRQ and storing the
   data with it.
 - As a result, don't try to support the mode in the AHCI driver where
   the excess ports all share the last interrupt.  It could be done, but
   it would be rather messy and I don't have hardware that supports that
   mode anyway.

I'm fairly comfortable with the subchannel notion we're introducing
here.  It's more flexible than MSI and doesn't impose a penalty on
architectures which don't implement it.  It makes some things more
complex, but it makes other things simpler, so I think it's a wash from
a cleanliness standpoint.

Git tree still at 
git://git.kernel.org/pub/scm/linux/kernel/git/willy/misc.git multiple-msi
(and this push specifically at multiple-msi-20080711)

Patches to follow.

-- 
Intel are signing my paycheques ... these opinions are still mine
"Bill, look, we understand that you're interested in selling us this
operating system, but compare it to ours.  We can't possibly take such
a retrograde step."

             reply	other threads:[~2008-07-11 21:16 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-11 21:16 Matthew Wilcox [this message]
2008-07-11 21:16 ` [PATCH 1/6] Allow a small amount of data to be stored with the interrupt number Matthew Wilcox
2008-07-11 21:16 ` [PATCH 2/6] PCI MSI: Replace 'type' with 'is_msix' Matthew Wilcox
2008-07-11 21:16 ` [PATCH 3/6] PCI MSI: Add support for multiple messages Matthew Wilcox
2008-07-11 21:16 ` [PATCH 4/6] Rewrite MSI-HOWTO Matthew Wilcox
2008-07-11 21:16 ` [PATCH 5/6] AHCI: Request multiple MSIs Matthew Wilcox
2008-07-11 21:16 ` [PATCH 6/6] x86-64: Support for " Matthew Wilcox
2008-07-12  5:54   ` Ingo Molnar
2008-07-12 16:30     ` Matthew Wilcox
2008-07-12 18:44       ` Yinghai Lu
2008-07-12  7:37 ` Multiple MSI, take 4 Benjamin Herrenschmidt

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=20080711211559.GV14894@parisc-linux.org \
    --to=matthew@wil.cx \
    --cc=benh@kernel.crashing.org \
    --cc=grundler@parisc-linux.org \
    --cc=jbarnes@virtuousgeek.org \
    --cc=jgarzik@pobox.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=mtk.manpages@gmail.com \
    --cc=rdunlap@xenotime.net \
    --cc=suresh.b.siddha@intel.com \
    --cc=tglx@linutronix.de \
    /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).