From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:57658) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Scy53-0001fI-Rt for qemu-devel@nongnu.org; Fri, 08 Jun 2012 08:14:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Scy4y-0000s8-VL for qemu-devel@nongnu.org; Fri, 08 Jun 2012 08:14:17 -0400 Received: from thoth.sbs.de ([192.35.17.2]:21836) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Scy4y-0000rs-Lb for qemu-devel@nongnu.org; Fri, 08 Jun 2012 08:14:12 -0400 Message-ID: <4FD1EC90.3070401@siemens.com> Date: Fri, 08 Jun 2012 14:14:08 +0200 From: Jan Kiszka MIME-Version: 1.0 References: <8efa27ccfaf23cc561d2583aa34ca834fd60c2f3.1338799936.git.jan.kiszka@siemens.com> <4FD0A548.3030005@suse.de> <4FD0C4A5.7030600@web.de> <4FD0CF2B.5050501@suse.de> <4FD1DA82.8060305@siemens.com> <4FD1EA2F.80908@suse.de> In-Reply-To: <4FD1EA2F.80908@suse.de> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 11/13] Move pci_parse_devaddr to qdev-properties List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?ISO-8859-15?Q?Andreas_F=E4rber?= Cc: Paolo Bonzini , Alex Williamson , qemu-devel , Anthony Liguori , "Michael S. Tsirkin" On 2012-06-08 14:03, Andreas F=E4rber wrote: > Am 08.06.2012 12:57, schrieb Jan Kiszka: >> On 2012-06-07 17:56, Andreas F=E4rber wrote: >>> Am 07.06.2012 17:11, schrieb Jan Kiszka: >>>> On 2012-06-07 14:57, Andreas F=E4rber wrote: >>>>> These last three patches collide with Paolo's QOM properties=20 >>>>> refactoring: qdev properties are being generalized to Object and >>>>> on my GitHub "realize" branch are being moved to >>>>> qom/object-properties.c (object.c in the original series). Please >>>>> defer this change. >>> >>>> Depends on how long merging of those branches shall take. This is >>>> some important piece for preparing device assignment for upstream, >>>> thus finally closing the qemu-kvm fork. I need all this back-merged >>>> in qemu-kvm soon to proceed. >>> >>>> Can you (both) comment on the merge schedule for your patches? Are >>>> we talking about a week or so? >>> >>> I'm working towards sending the updated patches from realize branch >>> today and the PULL by tomorrow. When it gets merged I cannot predict. >> >> To my understanding, patch 2 of your series faces some discussions, an= d >> the rest will require refactoring once that has settled. So I guess it= 's >> better now to proceed with my patches and rebase your changes on top o= f >> them, right? >=20 > No, there's a patch queue of 28 before the patches under discussion, > some of which touch qdev-properties.c as well. Those under discussion > are not on qom-next yet. > What's open is that the Makefile series has not been pulled yet and > pretty much everyone will need to rebase on that if it gets applied. > Hoping for a statement from Anthony there. As I was on CC in that second series, right after you told me you would CC me to allow tracking progress. So I was assuming it was related to this topic. >=20 > Can't understand that you're trying to push your v1 so hard now when ou= r > series have been on the list for much longer. ;) I explained the reason, and I think my patches (minus the dropped one) are ready based on the feedback. I'm fine if important central refactorings close master for a short period at the beginning of a merge window and require rebasing of other patches afterward. But that should really be *short*. Jan --=20 Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux