From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6111 invoked by alias); 12 Sep 2014 08:01:17 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 6092 invoked by uid 89); 12 Sep 2014 08:01:13 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_HELO_PASS,SPF_PASS autolearn=no version=3.3.2 X-HELO: na01-bl2-obe.outbound.protection.outlook.com Received: from mail-bl2on0062.outbound.protection.outlook.com (HELO na01-bl2-obe.outbound.protection.outlook.com) (65.55.169.62) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Fri, 12 Sep 2014 08:01:11 +0000 Received: from BL2FFO11FD032.protection.gbl (10.173.160.33) by BL2FFO11HUB014.protection.gbl (10.173.160.106) with Microsoft SMTP Server (TLS) id 15.0.1019.14; Fri, 12 Sep 2014 08:01:08 +0000 Received: from xsj-pvapsmtpgw01 (149.199.60.83) by BL2FFO11FD032.mail.protection.outlook.com (10.173.160.73) with Microsoft SMTP Server (TLS) id 15.0.1019.14 via Frontend Transport; Fri, 12 Sep 2014 08:01:08 +0000 Received: from unknown-38-66.xilinx.com ([149.199.38.66] helo=xsj-smtp1) by xsj-pvapsmtpgw01 with esmtp (Exim 4.63) (envelope-from ) id 1XSLlj-00063w-El; Fri, 12 Sep 2014 00:59:47 -0700 From: Ajit Kumar Agarwal To: Joel Brobecker , Michael Eager CC: "gdb-patches@sourceware.org" , Vinod Kathail , Vidhumouli Hunsigida , "Nagaraju Mekala" Subject: RE: [Patch, microblaze]: Port of Linux gdbserver Date: Fri, 12 Sep 2014 08:01:00 -0000 References: <2570e3c7-f55b-45cd-aa6e-7f4fa145f32a@BN1BFFO11FD002.protection.gbl> <541052B5.5080503@eagercon.com> <20140910134606.GO28404@adacore.com> <050c6461-c35c-441d-9b63-7636d9164e2e@BL2FFO11FD048.protection.gbl> <20140910144313.GP28404@adacore.com> In-Reply-To: <20140910144313.GP28404@adacore.com> Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-RCIS-Action: ALLOW Message-ID: <273d7e70-1fda-4b70-a987-e6831ef4b12a@BL2FFO11FD032.protection.gbl> X-EOPAttributedMessage: 0 X-Forefront-Antispam-Report: CIP:149.199.60.83;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(438002)(13464003)(54534003)(51704005)(377454003)(199003)(189002)(377424004)(50466002)(85306004)(93886004)(74316001)(92566001)(31696002)(76482001)(87936001)(81542001)(4396001)(70736001)(46406003)(46102001)(77982001)(99396002)(83322001)(81342001)(97756001)(79102001)(64706001)(1496007)(50986999)(85852003)(54356999)(76176999)(44976005)(31966008)(19580395003)(74662001)(6806004)(80022001)(47776003)(20776003)(90102001)(74502001)(107046002)(104016003)(77096002)(106466001)(106116001)(2656002)(53416004)(92726001)(19580405001)(86362001)(83072002)(21056001)(23726002)(95666004)(107986001)(23106004);DIR:OUT;SFP:1101;SCL:1;SRVR:BL2FFO11HUB014;H:xsj-pvapsmtpgw01;FPR:;MLV:sfv;PTR:unknown-60-83.xilinx.com;A:1;MX:1;LANG:en; X-Microsoft-Antispam: BCL:0;PCL:0;RULEID:(12001);UriScan:; X-Forefront-PRVS: 0332AACBC3 Received-SPF: Pass (protection.outlook.com: domain of xilinx.com designates 149.199.60.83 as permitted sender) receiver=protection.outlook.com; client-ip=149.199.60.83; helo=xsj-pvapsmtpgw01; Authentication-Results: spf=pass (sender IP is 149.199.60.83) smtp.mailfrom=ajit.kumar.agarwal@xilinx.com; X-OriginatorOrg: xilinx.com X-SW-Source: 2014-09/txt/msg00382.txt.bz2 With feedback comments incorporated, Please find the updated patch. [Patch, microblaze]: Port of Linux gdbserver This patch is the port of Linux gdbserver. gdb/ChangeLog: 2014-10-12 Ajit Agarwal * configure.tgt (build_gdbserver): New Definition. * features/microblaze-linux-core.xml: New file. * features/microblaze-linux-stack-protect.xml: New file. * features/microblaze-linux-stack-protect.c: New file. gdb/gdbserver/ChangeLog: * gdbserver/Makefile.in (microblaze-linux.c): New target. * gdbserver/configure.srv (microblaze*-*-linux*): New target. * gdbserver/linux-microblaze-low.c: New file. Signed-off-by:Ajit Agarwal ajitkum@xilinx.com Thanks & Regards Ajit -----Original Message----- From: Joel Brobecker [mailto:brobecker@adacore.com]=20 Sent: Wednesday, September 10, 2014 8:13 PM To: Ajit Kumar Agarwal Cc: Michael Eager; gdb-patches@sourceware.org; Vinod Kathail; Vidhumouli Hu= nsigida; Nagaraju Mekala Subject: Re: [Patch, microblaze]: Port of Linux gdbserver [with my patch-champion hat] I can't review the gdbserver Changes, but I can look at the rest. > ChangeLog: > 2014-10-09 Ajit Agarwal >=20 > * configure.host (microblaze): New. > (microblaze*-*-linux*): New. > * configure.tgt (build_gdbserver): New Definition. > * gdbserver/Makefile.in (microblaze-linux.c): New target. > * gdbserver/configure.srv (microblaze*-*-linux*): New target. > * gdbserver/linux-microblaze-low.c: New file. Sorry Ajit. I should have seen this in the previous iteration, but there ar= e a couple more nits in the ChangeLog entry that you'll need to fix. First, gdb and gdb/gdbserver have distinct ChangeLog files, so you'll need = to have 2 ChangeLog entries if you touch both areas at the same time. This= means that the filenames in gdbserver should not be prefixed with "gdbserv= er/" in your ChangeLog entry. Also, change "ChangeLog:" to "gdb/ChangeLog:" and "gdbserver/ChangeLog:". Please exclude the configure.host change. This looks unrelated. > diff --git a/gdb/configure.tgt b/gdb/configure.tgt index=20 > 01311b2..e4894da 100644 > --- a/gdb/configure.tgt > +++ b/gdb/configure.tgt > @@ -343,6 +343,7 @@ microblaze*-linux-*|microblaze*-*-linux*) > gdb_target_obs=3D"microblaze-tdep.o microblaze-linux-tdep.o microblaze-= rom.o \ > monitor.o dsrec.o solib-svr4.o symfile-mem.o linux-tdep.o" > gdb_sim=3D../sim/microblaze/libsim.a > + build_gdbserver=3Dyes > ;; > microblaze*-*-*) > # Target: Xilinx MicroBlaze running standalone diff --git=20 > a/gdb/gdbserver/Makefile.in b/gdb/gdbserver/Makefile.in index=20 > 1447e61..b2a01f5 100644 > --- a/gdb/gdbserver/Makefile.in > +++ b/gdb/gdbserver/Makefile.in > @@ -153,6 +153,7 @@ SFILES=3D $(srcdir)/gdbreplay.c $(srcdir)/inferiors.c= $(srcdir)/dll.c \ > $(srcdir)/linux-ia64-low.c $(srcdir)/linux-low.c \ > $(srcdir)/linux-m32r-low.c \ > $(srcdir)/linux-m68k-low.c $(srcdir)/linux-mips-low.c \ > + $(srcdir)/linux-microblaze-low.c \ > $(srcdir)/linux-nios2-low.c \ > $(srcdir)/linux-ppc-low.c \ > $(srcdir)/linux-s390-low.c \ > @@ -364,6 +365,7 @@ clean: > rm -f amd64-mpx.c amd64-mpx-linux.c > rm -f amd64-avx512.c amd64-avx512-linux.c > rm -f i386-mmx.c i386-mmx-linux.c > + rm -f microblaze-linux.c > rm -f x32.c x32-linux.c > rm -f x32-avx.c x32-avx-linux.c > rm -f x32-avx512.c x32-avx512-linux.c There are indentation errors in the two hunks above. Use tabs. > @@ -634,6 +636,8 @@ mips64-linux.c : $(srcdir)/../regformats/mips64-linux= .dat $(regdat_sh) > $(SHELL) $(regdat_sh) $(srcdir)/../regformats/mips64-linux.dat=20 > mips64-linux.c mips64-dsp-linux.c : $(srcdir)/../regformats/mips64-dsp-l= inux.dat $(regdat_sh) > $(SHELL) $(regdat_sh) $(srcdir)/../regformats/mips64-dsp-linux.dat=20 > mips64-dsp-linux.c > +microblaze-linux.c : $(srcdir)/../regformats/microblaze-with-stack-prote= ct.dat $(regdat_sh) > + $(SHELL) $(regdat_sh)=20 > +$(srcdir)/../regformats/microblaze-with-stack-protect.dat=20=20 > +microblaze-linux.c > nios2-linux.c : $(srcdir)/../regformats/nios2-linux.dat $(regdat_sh) > $(SHELL) $(regdat_sh) $(srcdir)/../regformats/nios2-linux.dat=20 > nios2-linux.c powerpc-32.c :=20 > $(srcdir)/../regformats/rs6000/powerpc-32.dat $(regdat_sh) diff --git=20 > a/gdb/gdbserver/configure.srv b/gdb/gdbserver/configure.srv index=20 > 679fc9f..16e44ee 100644 > --- a/gdb/gdbserver/configure.srv > +++ b/gdb/gdbserver/configure.srv > @@ -194,6 +194,12 @@ case "${target}" in > srv_linux_usrregs=3Dyes > srv_linux_thread_db=3Dyes > ;; > + microblaze*-*-linux*) srv_regobj=3Dmicroblaze-linux.o > + srv_tgtobj=3D"$srv_linux_obj linux-microblaze-lo= w.o" > + srv_linux_usrregs=3Dyes > + srv_linux_regsets=3Dyes > + srv_linux_thread_db=3Dyes > + ;; > nios2*-*-linux*) srv_regobj=3D"nios2-linux.o" > srv_tgtobj=3D"$srv_linux_obj linux-nios2-low.o" > srv_xmlfiles=3D"nios2-linux.xml" > diff --git a/gdb/gdbserver/linux-microblaze-low.c=20 > b/gdb/gdbserver/linux-microblaze-low.c > new file mode 100644 > index 0000000..b86b4f5 > --- /dev/null > +++ b/gdb/gdbserver/linux-microblaze-low.c It looks to me like you have many many trailing spaces in the file you are = submitting. Please strip them. Some lines are too long, also. -- Joel