From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:39913) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UXBiA-0003Z5-EG for qemu-devel@nongnu.org; Tue, 30 Apr 2013 10:39:22 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UXBi5-0006wy-Pp for qemu-devel@nongnu.org; Tue, 30 Apr 2013 10:39:18 -0400 Received: from mail-gg0-x234.google.com ([2607:f8b0:4002:c02::234]:45292) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UXBi5-0006vf-GR for qemu-devel@nongnu.org; Tue, 30 Apr 2013 10:39:13 -0400 Received: by mail-gg0-f180.google.com with SMTP id i25so72836ggk.25 for ; Tue, 30 Apr 2013 07:39:13 -0700 (PDT) From: Anthony Liguori In-Reply-To: <517FCE64.3080403@redhat.com> References: <1366303444-24620-1-git-send-email-pbonzini@redhat.com> <1366303444-24620-8-git-send-email-pbonzini@redhat.com> <517FB63A.6070303@suse.de> <517FCE64.3080403@redhat.com> Date: Tue, 30 Apr 2013 09:39:09 -0500 Message-ID: <87y5c0nkvm.fsf@codemonkey.ws> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 7/8] audio: look for the ISA and PCI buses List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini , Andreas =?utf-8?Q?F=C3=A4rber?= Cc: Igor Mammedov , av1474@comtv.ru, qemu-devel@nongnu.org Paolo Bonzini writes: > Il 30/04/2013 14:16, Andreas F=C3=A4rber ha scritto: >>=20 >> The patch version committed has an additional change in this file: >>=20 >> @@ -88,7 +88,6 @@ static void pc_init1(MemoryRegion *system_memory, >> void *fw_cfg =3D NULL; >>=20 >> pc_cpus_init(cpu_model); >> - pc_acpi_init("acpi-dsdt.aml"); >>=20 >> if (kvmclock_enabled) { >> kvmclock_create(); >>=20 >> Was that accidental? Noticed because of a conflict with Igor's patches. > > Yes, it must have been introduced when applying, because this wasn't a > pull request. I'll send a patch to revert. I had a dirty tree which normally doesn't create problems. However, in this case Paolo's patches no longer applied cleanly. I have a script that will apply each patch allowing fuzz, and then analyses the patch to do git add for any file touched by the patch. This restriction normally prevents this type of problem from happening. However, in this case, this patch touched the file containing the dirty bits so it got picked up by accident. I'll modify patches apply to fail if the tree is dirty which will prevent this in the future. > >> Further, my Reviewed-by is missing on patch 1/8, so was maybe the >> previous RFC version committed accidentally instead of this one? Your Reviewed-by is missing because I applied the series before you sent the Reviewed-by. Regards, Anthony Liguori > > No, the RFC only had 6 patches and patches 6-7-8 were completely > different (and broken). > > Paolo