[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] How many msi-x vectors should be allocated for the virt
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device? |
Date: |
Wed, 27 Feb 2013 14:47:11 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
Il 27/02/2013 14:46, Gal Hammer ha scritto:
> On 27/02/2013 15:28, Paolo Bonzini wrote:
>> Il 27/02/2013 14:24, Gal Hammer ha scritto:
>>> On 27/02/2013 14:35, Amit Shah wrote:
>>>> On (Mon) 04 Feb 2013 [11:59:02], Michael S. Tsirkin wrote:
>>>>> On Mon, Feb 04, 2013 at 10:42:32AM +0100, Paolo Bonzini wrote:
>>>>>> Il 03/02/2013 13:11, Michael S. Tsirkin ha scritto:
>>>>>>>>> static Property virtio_serial_properties[] = {
>>>>>>>>> DEFINE_PROP_BIT("ioeventfd", VirtIOPCIProxy, flags,
>>>>>>>>> VIRTIO_PCI_FLAG_USE_IOEVENTFD_BIT, true),
>>>>>>>>> - DEFINE_PROP_UINT32("vectors", VirtIOPCIProxy, nvectors,
>>>>>>>>> DEV_NVECTORS_UNSPECIFIED),
>>>>>>>>> + DEFINE_PROP_UINT32("vectors", VirtIOPCIProxy, nvectors, 2),
>>>>>>>>> DEFINE_PROP_HEX32("class", VirtIOPCIProxy, class_code, 0),
>>>>>>>>> DEFINE_VIRTIO_COMMON_FEATURES(VirtIOPCIProxy,
>>>>>>>>> host_features),
>>>>>>>>> DEFINE_PROP_UINT32("max_ports", VirtIOPCIProxy,
>>>>>>>>> serial.max_virtserial_ports, 31),
>>>>>>>>>
>>>>>>>>> plus the backwards-compatibility stuff.
>>>>>>>>>
>>>>>>>>> Paolo
>>>>>>> Makes sense, but the logic in virtio_serial_init_pci is
>>>>>>> then dead code and should go away.
>>>>>>>
>>>>>>
>>>>>> It won't be dead code for the backwards-compatible machine types
>>>>>> (that
>>>>>> use DEV_NVECTORS_UNSPECIFIED).
>>>>>>
>>>>>> Paolo
>>>>>
>>>>> Good point. Ack. Want to post this with proper signature etc?
>>>>
>>>> Gal, do you want to submit a patch for this?
>>>>
>>>> Amit
>>>>
>>>
>>> Attached.
>>
>> You need to add backwards-compatibility properties in hw/pc.h.
>
> I've added the BC property. I hope it is in the right format.
Yes, it's okay though usually we send patches with git-send-email. It's
up to mst whether to take the attached patch or not. :)
Anyhow,
Reviewed-by: Paolo Bonzini <address@hidden>
Thanks!
Paolo
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Paolo Bonzini, 2013/02/01
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Michael S. Tsirkin, 2013/02/03
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Paolo Bonzini, 2013/02/04
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Michael S. Tsirkin, 2013/02/04
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Amit Shah, 2013/02/27
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Gal Hammer, 2013/02/27
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Paolo Bonzini, 2013/02/27
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Gal Hammer, 2013/02/27
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?,
Paolo Bonzini <=
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Michael S. Tsirkin, 2013/02/27
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Gal Hammer, 2013/02/27
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Michael S. Tsirkin, 2013/02/27
- Re: [Qemu-devel] How many msi-x vectors should be allocated for the virtio-serial device?, Michael S. Tsirkin, 2013/02/27