From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 31438 invoked by alias); 23 May 2014 06:54:05 -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 31422 invoked by uid 89); 23 May 2014 06:54:03 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.7 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 X-HELO: na01-bn1-obe.outbound.protection.outlook.com Received: from mail-bn1blp0189.outbound.protection.outlook.com (HELO na01-bn1-obe.outbound.protection.outlook.com) (207.46.163.189) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-SHA encrypted) ESMTPS; Fri, 23 May 2014 06:54:01 +0000 Received: from BY2FFO11FD021.protection.gbl (10.1.14.32) by BY2FFO11HUB032.protection.gbl (10.1.14.177) with Microsoft SMTP Server (TLS) id 15.0.949.9; Fri, 23 May 2014 06:53:57 +0000 Received: from xsj-gw1 (149.199.60.83) by BY2FFO11FD021.mail.protection.outlook.com (10.1.15.210) with Microsoft SMTP Server id 15.0.949.9 via Frontend Transport; Fri, 23 May 2014 06:53:57 +0000 Received: from unknown-38-66.xilinx.com ([149.199.38.66] helo=xsj-smtp1) by xsj-gw1 with esmtp (Exim 4.63) (envelope-from ) id 1WnjMb-0006m3-86; Thu, 22 May 2014 23:53:57 -0700 From: Ajit Kumar Agarwal To: "gdb-patches@sourceware.org" CC: Joel Brobecker , Michael Eager , Vinod Kathail , Vidhumouli Hunsigida , Nagaraju Mekala Subject: [Patch, microblaze]: Add slr and shr regs Date: Fri, 23 May 2014 06:54:00 -0000 Content-Type: multipart/mixed; boundary="_002_37378DC5BCD0EE48BA4B082E0B55DFAA25E354CAXAPPVEXMBX01xln_" MIME-Version: 1.0 X-RCIS-Action: ALLOW Message-ID: X-EOPAttributedMessage: 0 X-Forefront-Antispam-Report: CIP:149.199.60.83;CTRY:US;IPV:NLI;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(6009001)(438001)(377424004)(189002)(199002)(83322001)(70736001)(19580395003)(19580405001)(53416003)(83072002)(31696002)(44976005)(81342001)(81542001)(80022001)(46102001)(4396001)(71186001)(92726001)(86362001)(33646001)(87936001)(92566001)(85852003)(74316001)(54356999)(64706001)(50986999)(99936001)(77982001)(79102001)(76482001)(512954002)(20776003)(568964001)(2656002)(74662001)(84326002)(21056001)(31966008)(74502001)(23106003);DIR:OUT;SFP:;SCL:1;SRVR:BY2FFO11HUB032;H:xsj-gw1;FPR:;MLV:sfv;PTR:unknown-60-83.xilinx.com;MX:1;A:1;LANG:en; X-OriginatorOrg: xilinx.onmicrosoft.com X-Forefront-PRVS: 0220D4B98D Received-SPF: Pass (: domain of xilinx.com designates 149.199.60.83 as permitted sender) receiver=; client-ip=149.199.60.83; helo=xsj-gw1; Authentication-Results: spf=pass (sender IP is 149.199.60.83) smtp.mailfrom=ajit.kumar.agarwal@xilinx.com; X-SW-Source: 2014-05/txt/msg00578.txt.bz2 --_002_37378DC5BCD0EE48BA4B082E0B55DFAA25E354CAXAPPVEXMBX01xln_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Content-length: 2719 Hello Michael: Based on the feedback, resubmitting the patch once again with all your re= plies and suggestions. [Patch, microblaze]: Add slr and shr regs =20=20=20=20 This patch add the support of slr and shr regs and also solves the prob= lem related to process_g_packet where the buf_len > 2 * rsa->sizeof_g_packet and throwing the Error that 'g' packet message reply is too long. This = is because the buf_len calculated in the init_remote_state function for microblaze target is based On the sizeof_g_packet and remote_packet_size and the memory_packet_config->size. The sizeof_g_packet is 236 because = the number of reg num is 59 and 2* sizeof_g_packet comes to 472 .With shr a= nd shl entry and the buf_len is 472. This does not match the greater than conditional statement and works fine. Without shr and shl entry,the sizeof_g_packets comes to 57*4 *2 =3D 456. This doesn't match the crit= eria in the process_g_packet function leading to throwing of error message = as " 'g' packet message reply is too long". =20=20=20=20 ChangeLog: 2014-05-20 Ajit Agarwal =20=20=20=20 * gdb/gdbserver/Makefile.in (microblaze-linux.c): New rule. =20=20=20=20 * gdb/microblaze-tdep.c (microblaze_register_names): Added the rshr and rslr register names. =20=20=20=20 * gdb/microblaze-tdep.h (microblaze_reg_num): Addition of field MICROBLAZE_SLR_REGNUM and MICROBLAZE_SHR_REGNUM. (microblaze_frame_cache): Change in the index of register_offsets. =20=20=20=20 * gdb/regformats/reg-microblaze.dat: New Register data file. =20=20=20=20 Signed-off-by:Ajit Agarwal ajitkum@xilinx.com I am confused with your comments whereas it seems I have answered all queri= es(including yours) and incorporated your review comments.=20 Just to make sure I repeat, see below answers to your queries. >>Make sure you address my comments and incorporate my suggestions as well. I made sure that the description is not too long at the same time it gives = the complete picture. Hope this addresses your comments. >>>I asked what is running on the target which is returning a different siz= ed G packet. It is the gdbserver which is checking for the buf_len and 2* size_of_g_pac= ket in process_g_packet function which is not matching=20 and returning with error mentioning that 'g' packet is too long. The G Pack= et initialization is done in init_remote_state function which=20 is all happening when tar remote machine:1234 command is used. Hope this clarifies as this error is nothing to do with what is running on = target. On target the XMD is running. Thanks & Regards Ajit --_002_37378DC5BCD0EE48BA4B082E0B55DFAA25E354CAXAPPVEXMBX01xln_ Content-Type: application/octet-stream; name="0001-Patch-microblaze-Add-slr-and-shr-regs.patch" Content-Description: 0001-Patch-microblaze-Add-slr-and-shr-regs.patch Content-Disposition: attachment; filename="0001-Patch-microblaze-Add-slr-and-shr-regs.patch"; size=5022; creation-date="Wed, 21 May 2014 13:37:15 GMT"; modification-date="Thu, 22 May 2014 17:51:27 GMT" Content-Transfer-Encoding: base64 Content-length: 6808 RnJvbSBhOTM5ZTMyZTAwMTJmYjYxMWZjYzc4MzE3NTBkZGQ5MjVlOTcxOGIx IE1vbiBTZXAgMTcgMDA6MDA6MDAgMjAwMQpGcm9tOiBBaml0IEt1bWFyIEFn YXJ3YWwgPGFqaXRrdW1AeGhkc3BkZ251Lihub25lKT4KRGF0ZTogV2VkLCAy MSBNYXkgMjAxNCAxODo1NToyMyArMDUzMApTdWJqZWN0OiBbUEFUQ0hdIFtQ YXRjaCwgbWljcm9ibGF6ZV06IEFkZCBzbHIgYW5kIHNociByZWdzCgpUaGlz IHBhdGNoIGFkZCB0aGUgc3VwcG9ydCBvZiBzbHIgYW5kIHNociByZWdzIGFu ZCBhbHNvIHNvbHZlcyB0aGUgcHJvYmxlbQpyZWxhdGVkIHRvIHByb2Nlc3Nf Z19wYWNrZXQgd2hlcmUgdGhlIGJ1Zl9sZW4gPiAyICogcnNhLT5zaXplb2Zf Z19wYWNrZXQKYW5kIHRocm93aW5nIHRoZSBFcnJvciB0aGF0ICdnJyBwYWNr ZXQgbWVzc2FnZSByZXBseSBpcyB0b28gbG9uZy4gVGhpcyBpcwpiZWNhdXNl IHRoZSBidWZfbGVuIGNhbGN1bGF0ZWQgaW4gdGhlIGluaXRfcmVtb3RlX3N0 YXRlIGZ1bmN0aW9uIGZvcgptaWNyb2JsYXplIHRhcmdldCBpcyBiYXNlZCBP biB0aGUgc2l6ZW9mX2dfcGFja2V0IGFuZCByZW1vdGVfcGFja2V0X3NpemUK YW5kIHRoZSBtZW1vcnlfcGFja2V0X2NvbmZpZy0+c2l6ZS4gVGhlIHNpemVv Zl9nX3BhY2tldCBpcyAyMzYgYmVjYXVzZSB0aGUKbnVtYmVyIG9mIHJlZyBu dW0gaXMgNTkgYW5kIDIqIHNpemVvZl9nX3BhY2tldCBjb21lcyB0byA0NzIg LldpdGggc2hyIGFuZApzaGwgZW50cnkgYW5kIHRoZSBidWZfbGVuIGlzIDQ3 Mi4gVGhpcyBkb2VzIG5vdCBtYXRjaCB0aGUgZ3JlYXRlciB0aGFuCmNvbmRp dGlvbmFsIHN0YXRlbWVudCAgYW5kIHdvcmtzIGZpbmUuIFdpdGhvdXQgc2hy IGFuZCBzaGwgZW50cnksdGhlCnNpemVvZl9nX3BhY2tldHMgY29tZXMgdG8g NTcqNCAqMiA9IDQ1Ni4gIFRoaXMgZG9lc24ndCBtYXRjaCB0aGUgY3JpdGVy aWEKaW4gdGhlIHByb2Nlc3NfZ19wYWNrZXQgZnVuY3Rpb24gIGxlYWRpbmcg dG8gdGhyb3dpbmcgb2YgZXJyb3IgbWVzc2FnZSBhcwoiICdnJyBwYWNrZXQg bWVzc2FnZSByZXBseSBpcyB0b28gbG9uZyIuCgpDaGFuZ2VMb2c6CjIwMTQt MDUtMjAgQWppdCBBZ2Fyd2FsIDxhaml0a3VtQHhpbGlueC5jb20+CgoJKiBn ZGIvZ2Ric2VydmVyL01ha2VmaWxlLmluIChtaWNyb2JsYXplLWxpbnV4LmMp OiBOZXcgcnVsZS4KCgkqIGdkYi9taWNyb2JsYXplLXRkZXAuYyAobWljcm9i bGF6ZV9yZWdpc3Rlcl9uYW1lcyk6IEFkZGVkCgl0aGUgcnNociBhbmQgcnNs ciByZWdpc3RlciBuYW1lcy4KCgkqIGdkYi9taWNyb2JsYXplLXRkZXAuaCAo bWljcm9ibGF6ZV9yZWdfbnVtKTogQWRkaXRpb24gb2YKCWZpZWxkIE1JQ1JP QkxBWkVfU0xSX1JFR05VTSBhbmQgTUlDUk9CTEFaRV9TSFJfUkVHTlVNLgoJ KG1pY3JvYmxhemVfZnJhbWVfY2FjaGUpOiBDaGFuZ2UgaW4gdGhlIGluZGV4 IG9mCglyZWdpc3Rlcl9vZmZzZXRzLgoKCSogZ2RiL3JlZ2Zvcm1hdHMvcmVn LW1pY3JvYmxhemUuZGF0OiBOZXcgUmVnaXN0ZXIgZGF0YSBmaWxlLgoKU2ln bmVkLW9mZi1ieTpBaml0IEFnYXJ3YWwgYWppdGt1bUB4aWxpbnguY29tCi0t LQogZ2RiL2dkYnNlcnZlci9NYWtlZmlsZS5pbiAgICAgICAgIHwgICAgNSAr KystCiBnZGIvbWljcm9ibGF6ZS10ZGVwLmMgICAgICAgICAgICAgfCAgICAz ICstCiBnZGIvbWljcm9ibGF6ZS10ZGVwLmggICAgICAgICAgICAgfCAgICA2 ICsrKy0KIGdkYi9yZWdmb3JtYXRzL3JlZy1taWNyb2JsYXplLmRhdCB8ICAg NDEgKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKysrKwogNCBm aWxlcyBjaGFuZ2VkLCA1MSBpbnNlcnRpb25zKCspLCA0IGRlbGV0aW9ucygt KQogY3JlYXRlIG1vZGUgMTAwNjQ0IGdkYi9yZWdmb3JtYXRzL3JlZy1taWNy b2JsYXplLmRhdAoKZGlmZiAtLWdpdCBhL2dkYi9nZGJzZXJ2ZXIvTWFrZWZp bGUuaW4gYi9nZGIvZ2Ric2VydmVyL01ha2VmaWxlLmluCmluZGV4IGY3NzNm YTIuLjI0NTQwMDMgMTAwNjQ0Ci0tLSBhL2dkYi9nZGJzZXJ2ZXIvTWFrZWZp bGUuaW4KKysrIGIvZ2RiL2dkYnNlcnZlci9NYWtlZmlsZS5pbgpAQCAtMzQ4 LDcgKzM0OCw4IEBAIGNsZWFuOgogCXJtIC1mIGFtZDY0LWF2eC5jIGFtZDY0 LWF2eC1saW51eC5jCiAJcm0gLWYgYW1kNjQtbXB4LmMgYW1kNjQtbXB4LWxp bnV4LmMKIAlybSAtZiBhbWQ2NC1hdng1MTIuYyBhbWQ2NC1hdng1MTItbGlu dXguYwotCXJtIC1mIGkzODYtbW14LmMgaTM4Ni1tbXgtbGludXguYworCXJt IC1mIGkzODYtbW14LmMgaTM4Ni1tbXgtbGludXguYyAKKwlybSAtZiBtaWNy b2JsYXplLWxpbnV4LmMKIAlybSAtZiB4MzIuYyB4MzItbGludXguYwogCXJt IC1mIHgzMi1hdnguYyB4MzItYXZ4LWxpbnV4LmMKIAlybSAtZiB4MzItYXZ4 NTEyLmMgeDMyLWF2eDUxMi1saW51eC5jCkBAIC02MTQsNiArNjE1LDggQEAg cmVnLWNmLmMgOiAkKHNyY2RpcikvLi4vcmVnZm9ybWF0cy9yZWctY2YuZGF0 ICQocmVnZGF0X3NoKQogCSQoU0hFTEwpICQocmVnZGF0X3NoKSAkKHNyY2Rp cikvLi4vcmVnZm9ybWF0cy9yZWctY2YuZGF0IHJlZy1jZi5jCiBtaXBzLWxp bnV4LmMgOiAkKHNyY2RpcikvLi4vcmVnZm9ybWF0cy9taXBzLWxpbnV4LmRh dCAkKHJlZ2RhdF9zaCkKIAkkKFNIRUxMKSAkKHJlZ2RhdF9zaCkgJChzcmNk aXIpLy4uL3JlZ2Zvcm1hdHMvbWlwcy1saW51eC5kYXQgbWlwcy1saW51eC5j CittaWNyb2JsYXplLWxpbnV4LmMgOiAkKHNyY2RpcikvLi4vcmVnZm9ybWF0 cy9yZWctbWljcm9ibGF6ZS5kYXQgJChyZWdkYXRfc2gpCisJJChTSEVMTCkg JChyZWdkYXRfc2gpICQoc3JjZGlyKS8uLi9yZWdmb3JtYXRzL3JlZy1taWNy b2JsYXplLmRhdCBtaXBzLWxpbnV4LmMKIG1pcHMtZHNwLWxpbnV4LmMgOiAk KHNyY2RpcikvLi4vcmVnZm9ybWF0cy9taXBzLWRzcC1saW51eC5kYXQgJChy ZWdkYXRfc2gpCiAJJChTSEVMTCkgJChyZWdkYXRfc2gpICQoc3JjZGlyKS8u Li9yZWdmb3JtYXRzL21pcHMtZHNwLWxpbnV4LmRhdCBtaXBzLWRzcC1saW51 eC5jCiBtaXBzNjQtbGludXguYyA6ICQoc3JjZGlyKS8uLi9yZWdmb3JtYXRz L21pcHM2NC1saW51eC5kYXQgJChyZWdkYXRfc2gpCmRpZmYgLS1naXQgYS9n ZGIvbWljcm9ibGF6ZS10ZGVwLmMgYi9nZGIvbWljcm9ibGF6ZS10ZGVwLmMK aW5kZXggMTRjMWI1Mi4uNGQ2MzkwOSAxMDA2NDQKLS0tIGEvZ2RiL21pY3Jv YmxhemUtdGRlcC5jCisrKyBiL2dkYi9taWNyb2JsYXplLXRkZXAuYwpAQCAt NzMsNyArNzMsOCBAQCBzdGF0aWMgY29uc3QgY2hhciAqbWljcm9ibGF6ZV9y ZWdpc3Rlcl9uYW1lc1tdID0KICAgInJwYyIsICAicm1zciIsICJyZWFyIiwg InJlc3IiLCAicmZzciIsICJyYnRyIiwKICAgInJwdnIwIiwgInJwdnIxIiwg InJwdnIyIiwgInJwdnIzIiwgInJwdnI0IiwgInJwdnI1IiwgInJwdnI2IiwK ICAgInJwdnI3IiwgInJwdnI4IiwgInJwdnI5IiwgInJwdnIxMCIsICJycHZy MTEiLAotICAicmVkciIsICJycGlkIiwgInJ6cHIiLCAicnRsYngiLCAicnRs YnN4IiwgInJ0bGJsbyIsICJydGxiaGkiCisgICJyZWRyIiwgInJwaWQiLCAi cnpwciIsICJydGxieCIsICJydGxic3giLCAicnRsYmxvIiwgInJ0bGJoaSIs CisgICJyc2xyIiwgInJzaHIiCiB9OwogCiAjZGVmaW5lIE1JQ1JPQkxBWkVf TlVNX1JFR1MgQVJSQVlfU0laRSAobWljcm9ibGF6ZV9yZWdpc3Rlcl9uYW1l cykKZGlmZiAtLWdpdCBhL2dkYi9taWNyb2JsYXplLXRkZXAuaCBiL2dkYi9t aWNyb2JsYXplLXRkZXAuaAppbmRleCBhNTMyMDkyLi5mN2FjZmQ2IDEwMDY0 NAotLS0gYS9nZGIvbWljcm9ibGF6ZS10ZGVwLmgKKysrIGIvZ2RiL21pY3Jv YmxhemUtdGRlcC5oCkBAIC00Miw3ICs0Miw3IEBAIHN0cnVjdCBtaWNyb2Js YXplX2ZyYW1lX2NhY2hlCiAgIGludCBmcF9yZWdudW07CiAKICAgLyogT2Zm c2V0cyB0byBzYXZlZCByZWdpc3RlcnMuICAqLwotICBpbnQgcmVnaXN0ZXJf b2Zmc2V0c1s1N107CS8qIE11c3QgbWF0Y2ggTUlDUk9CTEFaRV9OVU1fUkVH Uy4gICovCisgIGludCByZWdpc3Rlcl9vZmZzZXRzWzU5XTsJLyogTXVzdCBt YXRjaCBNSUNST0JMQVpFX05VTV9SRUdTLiAgKi8KIAogICAvKiBUYWJsZSBv ZiBzYXZlZCByZWdpc3RlcnMuICAqLwogICBzdHJ1Y3QgdHJhZF9mcmFtZV9z YXZlZF9yZWcgKnNhdmVkX3JlZ3M7CkBAIC0xMDcsNyArMTA3LDkgQEAgZW51 bSBtaWNyb2JsYXplX3JlZ251bQogICBNSUNST0JMQVpFX1JUTEJYX1JFR05V TSwKICAgTUlDUk9CTEFaRV9SVExCU1hfUkVHTlVNLAogICBNSUNST0JMQVpF X1JUTEJMT19SRUdOVU0sCi0gIE1JQ1JPQkxBWkVfUlRMQkhJX1JFR05VTQor ICBNSUNST0JMQVpFX1JUTEJISV9SRUdOVU0sCisgIE1JQ1JPQkxBWkVfU0xS X1JFR05VTSwKKyAgTUlDUk9CTEFaRV9TSFJfUkVHTlVNCiB9OwogCiAvKiBB bGwgcmVnaXN0ZXJzIGFyZSAzMiBiaXRzLiAgKi8KZGlmZiAtLWdpdCBhL2dk Yi9yZWdmb3JtYXRzL3JlZy1taWNyb2JsYXplLmRhdCBiL2dkYi9yZWdmb3Jt YXRzL3JlZy1taWNyb2JsYXplLmRhdApuZXcgZmlsZSBtb2RlIDEwMDY0NApp bmRleCAwMDAwMDAwLi45MzZiYzQ0Ci0tLSAvZGV2L251bGwKKysrIGIvZ2Ri L3JlZ2Zvcm1hdHMvcmVnLW1pY3JvYmxhemUuZGF0CkBAIC0wLDAgKzEsNDEg QEAKK25hbWU6bWljcm9ibGF6ZSAKK2V4cGVkaXRlOnIxLHBjIAorMzI6cjAg CiszMjpyMSAKKzMyOnIyIAorMzI6cjMgCiszMjpyNCAKKzMyOnI1IAorMzI6 cjYgCiszMjpyNyAKKzMyOnI4IAorMzI6cjkgCiszMjpyMTAgCiszMjpyMTEg CiszMjpyMTIgCiszMjpyMTMgCiszMjpyMTQgCiszMjpyMTUgCiszMjpyMTYg CiszMjpyMTcgCiszMjpyMTggCiszMjpyMTkgCiszMjpyMjAgCiszMjpyMjEg CiszMjpyMjIgCiszMjpyMjMgCiszMjpyMjQgCiszMjpyMjUgCiszMjpyMjYg CiszMjpyMjcgCiszMjpyMjggCiszMjpyMjkgCiszMjpyMzAgCiszMjpyMzEg CiszMjpwYyAKKzMyOm1zciAKKzMyOmVhciAKKzMyOmVzciAKKzMyOmZzciAK KzMyOnNscgorMzI6c2hyCi0tIAoxLjcuMQoK --_002_37378DC5BCD0EE48BA4B082E0B55DFAA25E354CAXAPPVEXMBX01xln_--