From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:34394) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WH9Kn-0004sF-55 for qemu-devel@nongnu.org; Sat, 22 Feb 2014 04:57:32 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WH9Kf-0006yd-Q6 for qemu-devel@nongnu.org; Sat, 22 Feb 2014 04:57:25 -0500 Received: from mail-pa0-f47.google.com ([209.85.220.47]:33443) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WH9Kf-0006yX-KC for qemu-devel@nongnu.org; Sat, 22 Feb 2014 04:57:17 -0500 Received: by mail-pa0-f47.google.com with SMTP id kp14so4499819pab.6 for ; Sat, 22 Feb 2014 01:57:16 -0800 (PST) Message-ID: <53087475.2080601@ozlabs.ru> Date: Sat, 22 Feb 2014 20:57:09 +1100 From: Alexey Kardashevskiy MIME-Version: 1.0 References: <1391265952-31788-1-git-send-email-aik@ozlabs.ru> <52F9CF22.7070609@ozlabs.ru> <5307552F.5000704@ozlabs.ru> <5308096B.5000406@ozlabs.ru> <1393037513.6771.124.camel@pasglop> In-Reply-To: <1393037513.6771.124.camel@pasglop> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH v7 0/2] target-ppc: CPU device tree id List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Benjamin Herrenschmidt Cc: Michael Roth , qemu-devel@nongnu.org, Alexander Graf , Mike Day , qemu-ppc@nongnu.org, Anthony Liguori , Scott Wood , Paolo Bonzini , =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= On 02/22/2014 01:51 PM, Benjamin Herrenschmidt wrote: > On Sat, 2014-02-22 at 13:20 +1100, Alexey Kardashevskiy wrote: > >> Sorry for bothering again. This is quite simple patchset, it does not touch >> a lot and just a small rework. This is v7, v1 was 31/10/2013 (~4 months), I >> sent "ping" to v5 5 times (took 2 months to get any attention), there was >> no comment that it breaks anything, few people commented that the patchset > > "few people", you should name them. And they should ack it if they are > ok with it. Nobody specifically did "acked-by". Mike Day "would ack them if it would help", Andreas Färber has "No objection from my side, but I'm not really familiar with the topic", others commented some older versions and did not comment newer fixed versions. This is my confusion. Should I personally ask everyone who commented to re-review new version again? > >> is ok, we use it internally for quite a while but it is still not in any >> tree but mine. I have other patchsets like that and I am not posting new >> ones because I do not really see the point until I figure out what is wrong >> with my old patches or with the way I posting them and trying to get attention. >> >> If anyone feels ok to "ack" these patches, please do so. Thanks and sorry >> again. >> >> >>>> >>>>> >>>>> Changes: >>>>> v7: >>>>> * cleaned up a bit of QOM use >>>>> >>>>> v6: >>>>> * removed kvmppc_fixup_cpu() >>>>> >>>>> v5: >>>>> * cleanup >>>>> * removed cpustate::kvm_cpu_id >>>>> * split into 2 patches - new PPC API and the usage of the API >>>>> >>>>> >>>>> Alexey Kardashevskiy (2): >>>>> target-ppc: add PowerPCCPU::cpu_dt_id >>>>> target-ppc: spapr: e500: fix to use cpu_dt_id >>>>> >>>>> hw/intc/openpic_kvm.c | 2 +- >>>>> hw/intc/xics.c | 15 +++++++++++++-- >>>>> hw/intc/xics_kvm.c | 10 +++++----- >>>>> hw/ppc/e500.c | 7 +++++-- >>>>> hw/ppc/ppc.c | 22 ++++++++++++++++++++++ >>>>> hw/ppc/spapr.c | 9 +++++---- >>>>> hw/ppc/spapr_hcall.c | 6 +++--- >>>>> hw/ppc/spapr_rtas.c | 14 +++++++------- >>>>> target-ppc/cpu-qom.h | 2 ++ >>>>> target-ppc/cpu.h | 18 ++++++++++++++++++ >>>>> target-ppc/kvm.c | 15 +-------------- >>>>> target-ppc/kvm_ppc.h | 6 ------ >>>>> target-ppc/translate_init.c | 11 +++++------ >>>>> 13 files changed, 87 insertions(+), 50 deletions(-) >>>>> >>>> >>>> >>> >>> >> >> > > -- Alexey