All of lore.kernel.org
 help / color / mirror / Atom feed
* problems starting kvm-qemu with 5+ virtual functions
@ 2012-07-11 16:52 Chris Friesen
  2012-07-11 19:34 ` Alex Williamson
  0 siblings, 1 reply; 5+ messages in thread
From: Chris Friesen @ 2012-07-11 16:52 UTC (permalink / raw)
  To: kvm

Hi,

We're running into a problem where we can't start up a single instance 
of kvm-qemu with 5 or more virtual functions (for the ethernet card) 
being passed to the guest.  It's an Intel I350 NIC if it matters.

I noticed a discussion in a thread titled "[RFC PATCH 0/2] Expose 
available KVM free memory slot count to help avoid aborts" that seemed 
to point to the hardcoded definition of KVM_MEMORY_SLOTS as being a 
possible problem, and proposed some patches to allow dynamic sizing of 
the number of memory slots.  It looks like that patch never went through 
to mainline though.

Is this likely the source of my problem?  Why didn't it ever get 
submitted to mainline.

What's the proper method of supporting 8 VFs per VM, with possibly 
several VMs?

Thanks,
Chris

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: problems starting kvm-qemu with 5+ virtual functions
  2012-07-11 16:52 problems starting kvm-qemu with 5+ virtual functions Chris Friesen
@ 2012-07-11 19:34 ` Alex Williamson
  2012-07-11 19:56   ` Chris Friesen
  0 siblings, 1 reply; 5+ messages in thread
From: Alex Williamson @ 2012-07-11 19:34 UTC (permalink / raw)
  To: Chris Friesen; +Cc: kvm

On Wed, 2012-07-11 at 10:52 -0600, Chris Friesen wrote:
> Hi,
> 
> We're running into a problem where we can't start up a single instance 
> of kvm-qemu with 5 or more virtual functions (for the ethernet card) 
> being passed to the guest.  It's an Intel I350 NIC if it matters.
> 
> I noticed a discussion in a thread titled "[RFC PATCH 0/2] Expose 
> available KVM free memory slot count to help avoid aborts" that seemed 
> to point to the hardcoded definition of KVM_MEMORY_SLOTS as being a 
> possible problem, and proposed some patches to allow dynamic sizing of 
> the number of memory slots.  It looks like that patch never went through 
> to mainline though.
> 
> Is this likely the source of my problem?

Yes, that's the problem.

> Why didn't it ever get submitted to mainline.

There were issues with extending the number of available memory slots.
By itself, those patches just prevent an abort, but don't really let you
add more devices.

> What's the proper method of supporting 8 VFs per VM, with possibly 
> several VMs?

The limiting factor to increasing memory slots was searching the array.
That's since been fixed by caching mmio page table entries.  AIUI, Avi
is now willing to accept a patch to bump memory slots to a much higher
value.  The number of available slots can be queried by qemu using
KVM_CAP_NR_MEMSLOTS, then something like the RFC you point at could be
used to allow device assignment to take advantage of that.  Thanks,

Alex



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: problems starting kvm-qemu with 5+ virtual functions
  2012-07-11 19:34 ` Alex Williamson
@ 2012-07-11 19:56   ` Chris Friesen
  2012-07-11 20:06     ` Alex Williamson
  0 siblings, 1 reply; 5+ messages in thread
From: Chris Friesen @ 2012-07-11 19:56 UTC (permalink / raw)
  To: Alex Williamson; +Cc: kvm

On 07/11/2012 01:34 PM, Alex Williamson wrote:
>
> The limiting factor to increasing memory slots was searching the array.
> That's since been fixed by caching mmio page table entries.

Thanks for the confirmation of my suspicions.

Do you know roughly when this went in?  A commit ID would be great.

Chris

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: problems starting kvm-qemu with 5+ virtual functions
  2012-07-11 19:56   ` Chris Friesen
@ 2012-07-11 20:06     ` Alex Williamson
  2012-07-11 21:08       ` Chris Friesen
  0 siblings, 1 reply; 5+ messages in thread
From: Alex Williamson @ 2012-07-11 20:06 UTC (permalink / raw)
  To: Chris Friesen; +Cc: kvm

On Wed, 2012-07-11 at 13:56 -0600, Chris Friesen wrote:
> On 07/11/2012 01:34 PM, Alex Williamson wrote:
> >
> > The limiting factor to increasing memory slots was searching the array.
> > That's since been fixed by caching mmio page table entries.
> 
> Thanks for the confirmation of my suspicions.
> 
> Do you know roughly when this went in?  A commit ID would be great.

Almost exactly a year ago:

ce88decffd17bf9f373cc233c961ad2054965667



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: problems starting kvm-qemu with 5+ virtual functions
  2012-07-11 20:06     ` Alex Williamson
@ 2012-07-11 21:08       ` Chris Friesen
  0 siblings, 0 replies; 5+ messages in thread
From: Chris Friesen @ 2012-07-11 21:08 UTC (permalink / raw)
  To: Alex Williamson; +Cc: kvm

On 07/11/2012 02:06 PM, Alex Williamson wrote:
> On Wed, 2012-07-11 at 13:56 -0600, Chris Friesen wrote:
>> On 07/11/2012 01:34 PM, Alex Williamson wrote:
>>> The limiting factor to increasing memory slots was searching the array.
>>> That's since been fixed by caching mmio page table entries.
>> Thanks for the confirmation of my suspicions.
>>
>> Do you know roughly when this went in?  A commit ID would be great.
> Almost exactly a year ago:
>
> ce88decffd17bf9f373cc233c961ad2054965667
>

Thanks, that's useful.

Chris


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2012-07-11 21:08 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-07-11 16:52 problems starting kvm-qemu with 5+ virtual functions Chris Friesen
2012-07-11 19:34 ` Alex Williamson
2012-07-11 19:56   ` Chris Friesen
2012-07-11 20:06     ` Alex Williamson
2012-07-11 21:08       ` Chris Friesen

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.