[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20130305002857.GA7656@concordia>
Date: Tue, 5 Mar 2013 11:28:57 +1100
From: Michael Ellerman <michael@...erman.id.au>
To: Mike Qiu <qiudayu@...ux.vnet.ibm.com>
Cc: linuxppc-dev@...ts.ozlabs.org, tglx@...utronix.de,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/3] Enable multiple MSI feature in pSeries
On Mon, Mar 04, 2013 at 11:14:53AM +0800, Mike Qiu wrote:
> δΊ 2013/3/1 11:54, Michael Ellerman ει:
> >On Fri, Mar 01, 2013 at 11:08:45AM +0800, Mike wrote:
> >>Hi all
> >>
> >>Any comments? or any questions about my patchset?
> >You were going to get some performance numbers that show a definite
> >benefit for using more than one MSI.
> Yes, but my patch just enable the kernel to support this feature, whether
> to use it depens on the device driver.
Sure, but we don't add code just for fun, so unless there's a good
reason to add the feature - like better performance - we won't bother.
> And this feature has been merged to the kernel for X86 for a long time.
> See commit: 5ca72c4f7c412c2002363218901eba5516c476b1
> 51906e779f2b13b38f8153774c4c7163d412ffd9
That commit was merged in 3.9-rc1, ie. a few days ago, so no it has not been
in x86 for a long time.
That code removes the need for your first patch, which is a good start.
Please send a new version using irq_set_msi_desc_off().
cheers
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists