From: Michael Ellerman <michael@ellerman.id.au>
To: Anton Blanchard <anton@samba.org>
Cc: paulus@samba.org, linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH] powerpc: pseries: Round up MSI-X requests
Date: Mon, 04 Jun 2012 16:43:35 +1000 [thread overview]
Message-ID: <1338792215.15716.5.camel@concordia> (raw)
In-Reply-To: <20120604091545.03ce738b@kryten>
On Mon, 2012-06-04 at 09:15 +1000, Anton Blanchard wrote:
> The pseries firmware currently refuses any non power of two MSI-X
> request. Unfortunately most network drivers end up asking for that
> because they want a power of two for RX queues and one or two extra
> for everything else.
>
> This patch rounds up the firmware request to the next power of two
> if the quota allows it.
There is some chance this will result in breakage because the driver
asks for N - and assumes that is what was allocated - and the device is
configured for > N.
But that's a hypothetical, and we know the current approach sucks
because it will result in many drivers falling back to a single
interrupt.
I think this is the least-worst approach in light of the FW limitations,
and we can always add quirks in here if we really have to.
Paul is the pseries maintainer so he gets to ACK or NAK it, but from an
MSI point of view it gets my +1.
cheers
next prev parent reply other threads:[~2012-06-04 6:43 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-03 23:15 [PATCH] powerpc: pseries: Round up MSI-X requests Anton Blanchard
2012-06-04 6:43 ` Michael Ellerman [this message]
2012-06-04 6:54 ` Benjamin Herrenschmidt
2012-06-05 2:47 ` Anton Blanchard
2012-06-13 5:18 ` Michael Ellerman
2012-06-13 5:25 ` 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=1338792215.15716.5.camel@concordia \
--to=michael@ellerman.id.au \
--cc=anton@samba.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=paulus@samba.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).