[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC PATCH v2 2/3] virtio-pci : add a virtio-bus interf
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [RFC PATCH v2 2/3] virtio-pci : add a virtio-bus interface |
Date: |
Fri, 23 Nov 2012 14:26:36 +0000 |
On 23 November 2012 14:23, Konrad Frederic <address@hidden> wrote:
> On 23/11/2012 13:34, Peter Maydell wrote:
>> On 23 November 2012 12:29, Stefan Hajnoczi<address@hidden> wrote:
>>> Eventually VirtIOBindings can probably be inlined into VirtioBusInfo. I
>>> don't see a need for separate structs.
>>
>> I agree. It might (or might not) be convenient to retain it
>> temporarily while converting all the transports, but
>> VirtIOBindings is part of the old code which we're trying
>> to refactor here, and I'd expect it to go away when we're done.
> Yes, for the moment, I didn't refactor this VirtIOBindings, so it
> is better to separate struct to keep the virtiodevice binding function.
Where you're deliberately not changing something as a temporary
step you need to comment it to make that clear. Otherwise
people trying to review the code won't be able to tell...
-- PMM
Re: [Qemu-devel] [RFC PATCH v2 2/3] virtio-pci : add a virtio-bus interface, Anthony Liguori, 2012/11/26
[Qemu-devel] [RFC PATCH v2 1/3] virtio-bus : Introduce VirtioBus., fred . konrad, 2012/11/22
Re: [Qemu-devel] [RFC PATCH v2 1/3] virtio-bus : Introduce VirtioBus., Stefan Hajnoczi, 2012/11/23