From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47964) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WudlV-0006hp-5z for qemu-devel@nongnu.org; Wed, 11 Jun 2014 04:20:20 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WudlI-0007zc-H1 for qemu-devel@nongnu.org; Wed, 11 Jun 2014 04:20:13 -0400 Message-ID: <5398112E.1050307@suse.de> Date: Wed, 11 Jun 2014 10:19:58 +0200 From: Alexander Graf MIME-Version: 1.0 References: <1402473920-4763-1-git-send-email-aik@ozlabs.ru> <53980E15.2070003@suse.de> <539810D8.1080406@ozlabs.ru> In-Reply-To: <539810D8.1080406@ozlabs.ru> Content-Type: text/plain; charset=KOI8-R; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] spapr_pci: Advertise MSI quota List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexey Kardashevskiy , qemu-devel@nongnu.org Cc: qemu-ppc@nongnu.org, Badari Pulavarty On 11.06.14 10:18, Alexey Kardashevskiy wrote: > On 06/11/2014 06:06 PM, Alexander Graf wrote: >> On 11.06.14 10:05, Alexey Kardashevskiy wrote: >>> From: Badari Pulavarty >>> >>> Hotplug of multiple disks fails due to MSI vector quota check. >>> Number of MSI vectors default to 8 allowing only 4 devices. >>> This happens on RHEL6.5 guest. RHEL7 and SLES11 guests fallback >>> to INTX. >>> >>> One way to workaround the issue is to increase total MSIs, >>> so that MSI quota check allows us to hotplug multiple disks. >>> >>> Signed-off-by: Badari Pulavarty >>> Signed-off-by: Alexey Kardashevskiy >>> --- >>> hw/ppc/spapr_pci.c | 2 ++ >>> 1 file changed, 2 insertions(+) >>> >>> diff --git a/hw/ppc/spapr_pci.c b/hw/ppc/spapr_pci.c >>> index ddfd8bb..ebd92fd 100644 >>> --- a/hw/ppc/spapr_pci.c >>> +++ b/hw/ppc/spapr_pci.c >>> @@ -831,6 +831,7 @@ int spapr_populate_pci_dt(sPAPRPHBState *phb, >>> int bus_off, i, j; >>> char nodename[256]; >>> uint32_t bus_range[] = { cpu_to_be32(0), cpu_to_be32(0xff) }; >>> + uint16_t nmsi = 64; >> Why 64? > Why not? :) > We need some number here instead of default 8, that's pretty much it. 256? How is 64 any better than 8? How many MSIs can we support? Can't we just put that number in? If it's more than 64k, just put in 64k? Alex