All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Christoph Hellwig <hch@lst.de>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] virtio-blk issues
Date: Wed, 18 Mar 2009 10:41:38 +0200	[thread overview]
Message-ID: <49C0B3C2.8060604@redhat.com> (raw)
In-Reply-To: <20090318080841.GA13638@lst.de>

Christoph Hellwig wrote:
> On Mon, Mar 16, 2009 at 03:22:24PM +0200, Avi Kivity wrote:
>   
>> The kvm bios supports 32 slots; I don't know if upstream bochs picked up 
>> these changes.
>>     
>
> I tried using the kvm pc-bios with upstream qemu, but it fails pretty
> early on when the kernel tries to initialize the timer interrupt.
>
>   

Interrupt routing is different between kvm and qemu due to HPET sending 
interrupts to PIC IRQ0 and IOAPIC INTI2; the kvm up to 2.6.29 only 
supports 1:1 mapping between PIC and IOAPIC pins 0-15.

> What's the protocol for syncing all those bios changes?
>   

I merge them occasionally, but in this case, syncing the changes would 
cause one or the other to break.

What is needed is to sync the qemu changes, and additionally add code to 
detect the non-existence of non-1:1 mapping and revert back to the old 
behavior in that case (and disable HPET as well).

>> In any case I'd like to see a virtio-blk controller which can support an 
>> arbitrary number of disks, much like a SCSI controller.  A pci slot or 
>> function per disk is wasteful.
>>     
>
> I'll see what I can do.
>   

Did I mention my wishlist includes indirect virtio requests, where a 
virtio ring entry points to an sglist instead of the buffer?  Right now 
a 1MB request will consume 256 ring entries, which kills concurrency 
with many disks on a single ring.

-- 
error compiling committee.c: too many arguments to function

      reply	other threads:[~2009-03-18  8:41 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-03 12:54 [Qemu-devel] virtio-blk issues Christoph Hellwig
2009-03-03 14:22 ` Paul Brook
2009-03-03 15:23   ` Christoph Hellwig
2009-03-03 17:22   ` Anthony Liguori
2009-03-16 13:22   ` Avi Kivity
2009-03-18  8:08     ` Christoph Hellwig
2009-03-18  8:41       ` Avi Kivity [this message]

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=49C0B3C2.8060604@redhat.com \
    --to=avi@redhat.com \
    --cc=hch@lst.de \
    --cc=qemu-devel@nongnu.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 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.