public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
To: Arjan van de Ven <arjan-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>
Cc: kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
Subject: Re: Booting from PV disk driver (Was: Re:	[PATCH	10/13] KVM: Wire up hypercall handler ..)
Date: Sun, 25 Feb 2007 08:34:48 +0200	[thread overview]
Message-ID: <45E12E08.8040402@qumranet.com> (raw)
In-Reply-To: <1172385067.3265.1.camel-NIQFrBLA1CpScpXdPBN83iCwEArCW2h5@public.gmane.org>

Arjan van de Ven wrote:
> On Sun, 2007-02-25 at 07:17 +0200, Avi Kivity wrote:
>   
>> Arjan van de Ven wrote:
>>     
>>>> The easiest thing I can think of is to have the PV disk driver show up 
>>>> as an actual PCI device and to use a PCI option rom to hijack the 
>>>> appropriate interrupt.
>>>>     
>>>>         
>>> yeah that's a good plan. Being a PCI device also helps with the OS
>>> userspace knowing which module to load; even if the device itself never
>>> gets accessed. 
>>>
>>>   
>>>       
>> A problem with pci, at least if you use the pci interrupts, is that your 
>> interrupt might be shared with another device.  I guess a solution is 
>> not to use pci interrupts, but then we need some mechanism to allocate 
>> interrupts.
>>     
>
> shared interrupts aren't a big deal in Linux. at all.
> In fact, sharing all PV interrupts to one number is a performance
> enhancement ;)
>   

If you share a PV interrupt with a non-PV interrupt, then for each PV 
interrupt you have to check whether the non-PV interrupt fired.  That 
involves at least one expensive vmexit.

I agree that sharing PV interrupts is not very expensive (though I don't 
see why you call it an optimization - if you share 100 interrupts on one 
line you need to check 100 interrupt sources every time the interrupt 
fires).

-- 
Do not meddle in the internals of kernels, for they are subtle and quick to panic.


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

  parent reply	other threads:[~2007-02-25  6:34 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-19 10:20 [PATCH 0/13] kvm updates for 2.6.21 Avi Kivity
     [not found] ` <45D979D3.2020907-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-19 10:21   ` [PATCH 1/13] KVM: mmu: add missing dirty page tracking cases Avi Kivity
2007-02-19 10:22   ` [PATCH 2/13] KVM: Move virtualization deactivation from CPU_DEAD state to CPU_DOWN_PREPARE Avi Kivity
2007-02-19 10:23   ` [PATCH 3/13] KVM: Cosmetics Avi Kivity
2007-02-19 10:24   ` [PATCH 4/13] KVM: vmx: hack set_cr0_no_modeswitch() to actually do modeswitch Avi Kivity
2007-02-19 10:25   ` [PATCH 5/13] KVM: Use ARRAY_SIZE macro instead of manual calculation Avi Kivity
2007-02-19 10:26   ` [PATCH 6/13] KVM: Use page_private()/set_page_private() apis Avi Kivity
2007-02-19 10:27   ` [PATCH 7/13] KVM: add MSR based hypercall API Avi Kivity
2007-02-19 10:28   ` [PATCH 8/13] KVM: Add host hypercall support for vmx Avi Kivity
2007-02-19 10:29   ` [PATCH 9/13] KVM: Add hypercall host support for svm Avi Kivity
2007-02-19 10:30   ` [PATCH 10/13] KVM: Wire up hypercall handlers to a central arch-independent location Avi Kivity
     [not found]     ` <20070219103052.4D23725016B-LjA0eNSCdXrQnzwC+xcbyw@public.gmane.org>
2007-02-21 10:37       ` Pavel Machek
     [not found]         ` <20070221103733.GI3945-+ZI9xUNit7I@public.gmane.org>
2007-02-22 10:14           ` Avi Kivity
     [not found]             ` <45DD6CF0.3010509-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-22 10:17               ` Dor Laor
     [not found]                 ` <64F9B87B6B770947A9F8391472E032160A91BAF3-yEcIvxbTEBqsx+V+t5oei8rau4O3wl8o3fe8/T/H7NteoWH0uzbU5w@public.gmane.org>
2007-02-22 10:22                   ` Joerg Roedel
     [not found]                     ` <20070222102228.GC12801-5C7GfCeVMHo@public.gmane.org>
2007-02-22 10:37                       ` Avi Kivity
2007-02-22 10:34                   ` Arjan van de Ven
     [not found]                     ` <1172140490.3531.236.camel-NIQFrBLA1CpScpXdPBN83iCwEArCW2h5@public.gmane.org>
2007-02-22 10:40                       ` Avi Kivity
     [not found]                         ` <45DD7330.1030001-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-22 11:01                           ` Arjan van de Ven
     [not found]                             ` <1172142081.3531.243.camel-NIQFrBLA1CpScpXdPBN83iCwEArCW2h5@public.gmane.org>
2007-02-22 13:04                               ` Avi Kivity
2007-02-22 13:12                                 ` [kvm-devel] " Arjan van de Ven
     [not found]                                   ` <1172149924.3531.260.camel-NIQFrBLA1CpScpXdPBN83iCwEArCW2h5@public.gmane.org>
2007-02-22 13:29                                     ` Avi Kivity
     [not found]                                       ` <45DD9A9D.4060500-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-22 14:09                                         ` Arjan van de Ven
     [not found]                                           ` <1172153373.3531.268.camel-NIQFrBLA1CpScpXdPBN83iCwEArCW2h5@public.gmane.org>
2007-02-22 14:20                                             ` Avi Kivity
     [not found]                                               ` <45DDA6CB.4020603-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-23 14:55                                                 ` Anthony Liguori
2007-02-22 13:31                                     ` [PATCH 10/13] KVM: Wire up hypercall handlers toa " Dor Laor
2007-02-23 21:14                               ` Booting from PV disk driver (Was: Re: [PATCH 10/13] KVM: Wire up hypercall handler ..) Anthony Liguori
     [not found]                                 ` <45DF5943.3090304-rdkfGonbjUSkNkDKm+mE6A@public.gmane.org>
2007-02-23 22:33                                   ` Arjan van de Ven
     [not found]                                     ` <1172269981.3241.57.camel-NIQFrBLA1CpScpXdPBN83iCwEArCW2h5@public.gmane.org>
2007-02-24 22:35                                       ` Booting from PV disk driver (Was: Re: [PATCH 10/13]KVM: " Dor Laor
     [not found]                                         ` <64F9B87B6B770947A9F8391472E032160A91C385-yEcIvxbTEBqsx+V+t5oei8rau4O3wl8o3fe8/T/H7NteoWH0uzbU5w@public.gmane.org>
2007-02-25  5:20                                           ` Avi Kivity
     [not found]                                             ` <45E11C8A.9070006-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-25  5:48                                               ` Avi Kivity
2007-02-25  5:17                                       ` Booting from PV disk driver (Was: Re: [PATCH 10/13] KVM: " Avi Kivity
     [not found]                                         ` <45E11C00.2000602-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-25  6:31                                           ` Arjan van de Ven
     [not found]                                             ` <1172385067.3265.1.camel-NIQFrBLA1CpScpXdPBN83iCwEArCW2h5@public.gmane.org>
2007-02-25  6:34                                               ` Avi Kivity [this message]
     [not found]                                                 ` <45E12E08.8040402-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-25  6:42                                                   ` Arjan van de Ven
     [not found]                                                     ` <1172385720.3265.12.camel-NIQFrBLA1CpScpXdPBN83iCwEArCW2h5@public.gmane.org>
2007-02-25  6:52                                                       ` Avi Kivity
     [not found]                                                         ` <45E13213.5070502-atKUWr5tajBWk0Htik3J/w@public.gmane.org>
2007-02-25  6:56                                                           ` Arjan van de Ven
2007-02-19 10:31   ` [PATCH 11/13] KVM: svm: init cr0 with the wp bit set Avi Kivity
2007-02-19 10:33   ` [PATCH 13/13] KVM: More 0 -> NULL conversions Avi Kivity
2007-02-19 10:32 ` [PATCH 12/13] KVM: SVM: intercept SMI to handle it at host level Avi Kivity

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=45E12E08.8040402@qumranet.com \
    --to=avi-atkuwr5tajbwk0htik3j/w@public.gmane.org \
    --cc=arjan-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \
    --cc=kvm-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.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