[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/4] qdev realize, -object, and -object-late
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 0/4] qdev realize, -object, and -object-late |
Date: |
Tue, 26 Jun 2012 10:15:50 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) |
Anthony Liguori <address@hidden> writes:
> As part of implementing the virtio-rng backend support, I decided it was best
> to split off the generic part of the series to make review easier.
>
> This series does a few things. It adds a realized property to DeviceState.
> It does so in the least invasive way possible and I think this is the right
> way
> to break the log jam here.
>
> It also introduced two new command line options which are basically the same:
> -object and -object-late. These options are like -device but let you create
> arbitrary QOM objects. -device is now equivalent to -object with a little bit
> of additional magic to do bus resolution.
>
> Instead of:
>
> -device virtio-rng-pci,rng=rng0
>
> You can do:
>
> -late-object virtio-rng-pci,parent_bus=pci.0,rng=rng0,id=dev0
>
> The reason for having both -object and -late-object is that late-object is
> made
> after machine init whereas -object is made before machine init. As soon as
> machine init becomes a fully QOM mechanism, the distinction between these two
> commands will disappear.
Any particular reason for deleting the diffstat from your cover letter?
Also: please try harder to make commit message header lines shorter.
Wisdom straight from the source[*]:
For these reasons, the "summary" must be no more than 70-75
characters, and it must describe both what the patch changes, as well
as why the patch might be necessary. It is challenging to be both
succinct and descriptive, but that is what a well-written summary
should do.
[*]
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=blob;f=Documentation/SubmittingPatches;h=c379a2a6949f1c1cac04fb6f185c633512f37061;hb=HEAD#l505
- [Qemu-devel] [PATCH 1/4] object: add object_property_add_bool (v2), (continued)
- [Qemu-devel] [PATCH 1/4] object: add object_property_add_bool (v2), Anthony Liguori, 2012/06/25
- [Qemu-devel] [PATCH 2/4] qdev: add realized property and make adding child bus implied by realize, Anthony Liguori, 2012/06/25
- [Qemu-devel] [PATCH 4/4] vl: add -late-object to create QOM objects after machine init, Anthony Liguori, 2012/06/25
- [Qemu-devel] [PATCH 3/4] vl: add -object option to create QOM objects from the command line, Anthony Liguori, 2012/06/25
- Re: [Qemu-devel] [PATCH 3/4] vl: add -object option to create QOM objects from the command line, Blue Swirl, 2012/06/26
- Re: [Qemu-devel] [PATCH 0/4] qdev realize, -object, and -object-late,
Markus Armbruster <=