On 2013年02月26日 14:04, Daniel Veillard wrote:
On Tue, Feb 26, 2013 at 01:03:17PM +0800, Li Zhang wrote:
> On 2013年02月26日 12:51, Daniel Veillard wrote:
>> On Tue, Feb 26, 2013 at 11:10:35AM +0800, Li Zhang wrote:
>>> On 2013年02月25日 19:52, Daniel Veillard wrote:
>>>> I have just tagged git and pushed the tarball. The rpms for F17
>>>> are following too at the usual place:
>>>>
ftp://libvirt.org/libvirt/
>>>>
>>>> I gave a try to the set of rpms and this seems to work fine for
>>>> relatively simple tests, but of course more testing and checking on
>>>> other architectures is needed !
>>>> If everything goes fine, I will make an rc2 in a couple of days
>>>> and then we can decide if a final release on Friday is fine or if
>>>> we need to postpone to next week,
>>> There are too few changes in this release for pp64.
>> By "too few" do you mean that we are missing anything important ?
> I have sent out 2 patches which are not accepted for this release.
>
> One patch is to fix one bug for ppc64 which is:
> [PATCHv2 1/1] Optimize machine option to set more options with it.
> Without this patch, USB controllers doesn't work well with VGA
> enabled for ppc64.
> And testing with VDSM, it will always report errors when using VGA.
Hum, i just read your patch but i don't have enough knowledge of
qemu parameters to know if there is large side effect to your patch.
Also you remove code which is about core and replace it with something
completely different.
Thanks so much for your reviewing.
In qemu, it provides some options for some kind of devices or hardware
by -machine option.
I think its purpose is to provide the configurations for machines to
compare with -device xxx.
There will be more and more options to be set by -machine according to
QEMU's development.
So, in my patch, -machine is used to be set more options not only
dump_core with newer QEMU versions
As is and with the amount of details you provided I can't make
any
jugement of how bad or good those changes are. One thing is sure,
bug fixes can go in, they are fine during freeze !
Got it, thanks. :)
I will wait for more comments from mailing list.
Daniel