[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] qdev merge plans?
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] qdev merge plans? |
Date: |
Wed, 15 Jul 2009 08:21:34 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1b3pre) Gecko/20090513 Fedora/3.0-2.3.beta2.fc11 Lightning/1.0pre Thunderbird/3.0b2 |
On 07/15/09 00:25, Anthony Liguori wrote:
Gerd Hoffmann wrote:
Hi Anthony,
Your queue continues bitrotting. BlueSwirl did more qdev conversions
with the result that more fixups are needed to make the tree build
again once your queue is pushed ...
For now I did incremental fixups and didn't change the patches you
have queued up already. At least for the build failure this isn't
ideal though as it leaves a few revisions which don't build which is
bad for bisecting.
So what is your plan to handle the mess^H^H^H^Hmerge?
Early in your series, you introduce -device. But -device syntax doesn't
seem to be resolved yet.
The patch sits in your queue nevertheless, so I'm not sure what your
plan is ...
I don't think it's appropriate to pull in -device this close to the
feature freeze when the command line syntax is still being debated. You
only countered Paul's suggestions this morning so I don't think there's
any indication that this discussion has converged yet.
Ok, so it sounds like you'll plan to drop the patch(es) from the queue.
Ok. I'll go post fresh patch series based on upstream/master then, so
you can replace the qdev bits in your queue.
Quite a lot of your queue consists of things you've only posted as
Proof-of-concept series (like qdev/isa).
Yes, as mentioned there are quite a few work-in-progress patches.
So let me ask you, what is the set of patches that you think is
non-contended, posted as a non-RFC, but not yet merged?
I'll sort and repost the patches today. Rough plan is:
(1) Split the patch series in your queue into two:
- One with the properties bits (should be ready to merge).
- One with the -device stuff (for further discussion
and maybe merge).
(2) Repost qdev/compat bits, slightly updated according to
review comments (should be ready to merge too).
cheers,
Gerd