From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:33165) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UV0aM-0007Ig-CQ for qemu-devel@nongnu.org; Wed, 24 Apr 2013 10:22:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UV0aJ-00019R-TG for qemu-devel@nongnu.org; Wed, 24 Apr 2013 10:22:14 -0400 Message-ID: <5177EA89.1080503@suse.de> Date: Wed, 24 Apr 2013 16:22:01 +0200 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1366768096-2846-1-git-send-email-lig.fnst@cn.fujitsu.com> <1366768096-2846-2-git-send-email-lig.fnst@cn.fujitsu.com> <1366787706.20507.10.camel@liguang.fnst.cn.fujitsu.com> <1366788738.20507.18.camel@liguang.fnst.cn.fujitsu.com> <1366789222.20507.20.camel@liguang.fnst.cn.fujitsu.com> In-Reply-To: <1366789222.20507.20.camel@liguang.fnst.cn.fujitsu.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 2/2] target-*/cpu.h: remove cs_base for other targets List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: li guang Cc: Peter Maydell , jan.kiszka@siemens.com, green@moxielogic.com, qemu-devel@nongnu.org, blauwirbel@gmail.com, jcmvbkbc@gmail.com, edgar.iglesias@gmail.com, gxt@mprc.pku.edu.cn, proljc@gmail.com, agraf@suse.de, evgenyvoevodin@gmail.com, ehabkost@redhat.com, sw@weilnetz.de, paul@codesourcery.com, stefanha@redhat.com, imammedo@redhat.com, rth@twiddle.net, aliguori@us.ibm.com, Laurent@vivier.eu, michael@walle.cc, qemu-ppc@nongnu.org, pbonzini@redhat.com, aurelien@aurel32.net Am 24.04.2013 09:40, schrieb li guang: > =E5=9C=A8 2013-04-24=E4=B8=89=E7=9A=84 08:36 +0100=EF=BC=8CPeter Maydel= l=E5=86=99=E9=81=93=EF=BC=9A >> On 24 April 2013 08:32, li guang wrote: >>> I think even others want to use something like you said, >>> it should not 'cs_base', or, it's a bad name. >> >> Yes, this is why I said "has a less than helpful name". >> >>>> >>>>>>> --- a/target-sparc/cpu.h >>>>>>> +++ b/target-sparc/cpu.h >>>>>>> @@ -715,7 +715,7 @@ trap_state* cpu_tsptr(CPUSPARCState* env); >>>>>>> #define TB_FLAG_AM_ENABLED (1 << 5) >>>>>>> >>>>>>> static inline void cpu_get_tb_cpu_state(CPUSPARCState *env, targ= et_ulong *pc, >>>>>>> - target_ulong *cs_base, i= nt *flags) >>>>>>> + int *flags) >>>>>>> { >>>>>>> *pc =3D env->pc; >>>>>>> *cs_base =3D env->npc; >> >>>> You clearly have a problem with your compile and test >>>> process then, because it is clear from the patch that >>>> you've removed the cs_base argument from this function >>>> but the function still has a use of 'cs_base' in it. >>> >>> ???, sorry, where do I miss 'cs_base' removing? >> >> Last quoted line of source: "*cs_base =3D env->npc". >=20 > OK, thanks! > that remove by overshoot script! Some general reminders: We're in Soft Freeze, so in general no new big patch series will go into 1.5 unless there's a maintainer willing to take care of it - for i386 there is none, and random code cleanups do not look like something we must absolutely have in the release last minute. At least no one brought up on yesterday's call that this is a must-have, so maybe after the release would be a better time to let people review this? Whenever you send more than one patch, please include a cover letter. When you resend a series modified, please include a version such as v2, v3, etc. and a change log in the cover letter rather than resending with [updated] or in a way that can't be distinguished at all. You're expected to assure that your patches compile and don't break `make check` at least. Repeatedly sending patches that cannot possibly build okay means you're either overlooking error output from your build or maybe building the wrong source directory? Generally, touching the innards of TCG requires a good code understanding and testing of multiple targets; personally I try to avoid unnecessary changes for fear of breaking rare corner cases. ;) Regards, Andreas --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC= rnberg