From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 71661 invoked by alias); 1 Sep 2016 16:44:38 -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 70754 invoked by uid 89); 1 Sep 2016 16:44:38 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.7 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,RCVD_IN_SORBS_SPAM,SPF_PASS autolearn=no version=3.3.2 spammy=beside, momentarily, proceeding X-HELO: mail-oi0-f53.google.com Received: from mail-oi0-f53.google.com (HELO mail-oi0-f53.google.com) (209.85.218.53) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 01 Sep 2016 16:44:28 +0000 Received: by mail-oi0-f53.google.com with SMTP id r205so45107979oih.3 for ; Thu, 01 Sep 2016 09:44:27 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=RzAYtQX5W7mWB4+XGXL4NcW6jz6GLfnSCznC2wGcCvc=; b=Luj46axSmubUhs50Y6eH3nIl82yC5Occm2hJsGH83IdxUz5hzS5Ds1YdRqGO6IBecF 1R1I294IrOaakZ3OlBGRLVwX/T/O98MesaanPQKOKaqTHhNa2iX0Lc/1DitvGXlQ/tyr iQlhwB+FzPPZ88xQTnLiPWmCaPB6ENtDHVTXZshS8NStnlhU/+vWgNUQXgO8froyCSEW ZzUxf8Xvdo77sFRndr4kxj/pf7/YxS60dfSS8iJ0opwzob4JDyhQ+/T1vs5jrdZobgYY woDhUeCFPS3qH94NgCucwLaMFwsk2m8wjDe6+d9cm14ByNiYJaOCfBeJcFNWEU35ZpfG uCiA== X-Gm-Message-State: AE9vXwNNdlaZq5HctzeotWe52MgchYt61sqXsvlpyRHJ7hhJ1+6TGazxA7eXuRBnOewGr86Mz7ZT5cIB1UrjFw== X-Received: by 10.157.35.4 with SMTP id j4mr17221796otb.38.1472748266480; Thu, 01 Sep 2016 09:44:26 -0700 (PDT) MIME-Version: 1.0 Received: by 10.202.79.201 with HTTP; Thu, 1 Sep 2016 09:44:25 -0700 (PDT) In-Reply-To: References: <20160831171406.24057-1-antoine.tremblay@ericsson.com> <20160831171406.24057-2-antoine.tremblay@ericsson.com> <3fdb7193-60c7-49c9-ccf5-bc040aa157ea@redhat.com> From: Yao Qi Date: Thu, 01 Sep 2016 16:44:00 -0000 Message-ID: Subject: Re: [PATCH 2/2] Enable range stepping for ARM on GDBServer To: Pedro Alves Cc: Antoine Tremblay , "gdb-patches@sourceware.org" Content-Type: multipart/mixed; boundary=001a1141c4b2448864053b74ed70 X-IsSubscribed: yes X-SW-Source: 2016-09/txt/msg00022.txt.bz2 --001a1141c4b2448864053b74ed70 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Content-length: 1597 [sigh, I am testing my arm range stepping patches today...] On Thu, Sep 1, 2016 at 4:59 PM, Pedro Alves wrote: >>> >>>> I'm sorry I can't be more helpful at the moment but I wanted to post >>>> this issue before I have to leave for a while. >>> >>> Understood. Does enabling range stepping unblock something else? >> >> It would unblock ARM tracepoints, as per Yao's requirements... > > Tracepoints make gdbserver single-step and then not report the event > to gdb, so I do see the parallel with range-stepping. Throwing > while-stepping into the equation would make it even more clear. > Range-stepping makes gdbserver single-step and then not report the event to gdb if thread pc is within the range. It is similar to tracepoint, but = much simpler. Both range-stepping and tracepoing needs to remove reinsert_breakpoint when gdbserver gets an event but doesn't report it back to gdb. However, gdbserver doesn't do so now. That is the reason I believe we need to support range-stepping first, and I am working on this (but interrupted by 7.12 release). The draft patch attached removes reinsert_breakpoint when gdbserver gets an event but not to report it back to gdb. Beside "removing reinsert_breakpoint on gdbserver internal event", we'd better to think that "each backend unwinders don't have to worry about unavailable data". I posted a draft here https://sourceware.org/ml/gdb-patches/2016-05/msg00060.html, I need some review comments. Pedro, can you take a look? This is not a hard requirement for ARM tracepoint support. --=20 Yao (=E9=BD=90=E5=B0=A7) --001a1141c4b2448864053b74ed70 Content-Type: text/plain; charset=US-ASCII; name="1.diff" Content-Disposition: attachment; filename="1.diff" Content-Transfer-Encoding: base64 X-Attachment-Id: f_iskk95tk0 Content-length: 2184 ZGlmZiAtLWdpdCBhL2dkYi9nZGJzZXJ2ZXIvbGludXgtbG93LmMgYi9nZGIv Z2Ric2VydmVyL2xpbnV4LWxvdy5jCmluZGV4IDQ1MDYxYWMuLjJmMzBiYzEg MTAwNjQ0Ci0tLSBhL2dkYi9nZGJzZXJ2ZXIvbGludXgtbG93LmMKKysrIGIv Z2RiL2dkYnNlcnZlci9saW51eC1sb3cuYwpAQCAtMzY2MiwxNyArMzY2Miwz MSBAQCBsaW51eF93YWl0XzEgKHB0aWRfdCBwdGlkLAogCSAgKCp0aGVfbG93 X3RhcmdldC5zZXRfcGMpIChyZWdjYWNoZSwgZXZlbnRfY2hpbGQtPnN0b3Bf cGMpOwogCX0KIAotICAgICAgLyogV2UgbWF5IGhhdmUgZmluaXNoZWQgc3Rl cHBpbmcgb3ZlciBhIGJyZWFrcG9pbnQuICBJZiBzbywKLQkgd2UndmUgc3Rv cHBlZCBhbmQgc3VzcGVuZGVkIGFsbCBMV1BzIG1vbWVudGFyaWx5IGV4Y2Vw dCB0aGUKLQkgc3RlcHBpbmcgb25lLiAgVGhpcyBpcyB3aGVyZSB3ZSByZXN1 bWUgdGhlbSBhbGwgYWdhaW4uICBXZSdyZQotCSBnb2luZyB0byBrZWVwIHdh aXRpbmcsIHNvIHVzZSBwcm9jZWVkLCB3aGljaCBoYW5kbGVzIHN0ZXBwaW5n Ci0JIG92ZXIgdGhlIG5leHQgYnJlYWtwb2ludC4gICovCi0gICAgICBpZiAo ZGVidWdfdGhyZWFkcykKLQlkZWJ1Z19wcmludGYgKCJwcm9jZWVkaW5nIGFs bCB0aHJlYWRzLlxuIik7Ci0KICAgICAgIGlmIChzdGVwX292ZXJfZmluaXNo ZWQpCi0JdW5zdXNwZW5kX2FsbF9sd3BzIChldmVudF9jaGlsZCk7CisJewor CSAgLyogSWYgd2UgaGF2ZSBmaW5pc2hlZCBzdGVwcGluZyBvdmVyIGEgYnJl YWtwb2ludCwgd2UndmUKKwkgICAgIHN0b3BwZWQgYW5kIHN1c3BlbmRlZCBh bGwgTFdQcyBtb21lbnRhcmlseSBleGNlcHQgdGhlCisJICAgICBzdGVwcGlu ZyBvbmUuICBUaGlzIGlzIHdoZXJlIHdlIHJlc3VtZSB0aGVtIGFsbCBhZ2Fp bi4KKwkgICAgIFdlJ3JlIGdvaW5nIHRvIGtlZXAgd2FpdGluZywgc28gdXNl IHByb2NlZWQsIHdoaWNoCisJICAgICBoYW5kbGVzIHN0ZXBwaW5nIG92ZXIg dGhlIG5leHQgYnJlYWtwb2ludC4gICovCisJICB1bnN1c3BlbmRfYWxsX2x3 cHMgKGV2ZW50X2NoaWxkKTsKKwl9CisgICAgICBlbHNlCisJeworCSAgLyog UmVtb3ZlIHRoZSBzaW5nbGVzdGVwIGJyZWFrcG9pbnRzIGlmIGFueS4gIE5v dGUgdGhhdAorCSAgICAgdGhlcmUgaXNuJ3Qgc2luZ2xlc3RlcCBicmVha3Bv aW50IGlmIHdlIGZpbmlzaGVkIHN0ZXBwaW5nCisJICAgICBvdmVyLiAgKi8K KwkgIGlmIChjYW5fc29mdHdhcmVfc2luZ2xlX3N0ZXAgKCkKKwkgICAgICAm JiBoYXNfcmVpbnNlcnRfYnJlYWtwb2ludHMgKGN1cnJlbnRfdGhyZWFkKSkK KwkgICAgeworCSAgICAgIHN0b3BfYWxsX2x3cHMgKDAsIGV2ZW50X2NoaWxk KTsKKwkgICAgICBkZWxldGVfcmVpbnNlcnRfYnJlYWtwb2ludHMgKGN1cnJl bnRfdGhyZWFkKTsKKwkgICAgICB1bnN0b3BfYWxsX2x3cHMgKDAsIGV2ZW50 X2NoaWxkKTsKKwkgICAgfQorCX0KIAorICAgICAgaWYgKGRlYnVnX3RocmVh ZHMpCisJZGVidWdfcHJpbnRmICgicHJvY2VlZGluZyBhbGwgdGhyZWFkcy5c biIpOwogICAgICAgcHJvY2VlZF9hbGxfbHdwcyAoKTsKICAgICAgIHJldHVy biBpZ25vcmVfZXZlbnQgKG91cnN0YXR1cyk7CiAgICAgfQo= --001a1141c4b2448864053b74ed70--