Received: by 2002:a25:23cc:0:0:0:0:0 with SMTP id j195csp616696ybj; Thu, 7 May 2020 03:42:36 -0700 (PDT) X-Google-Smtp-Source: APiQypIiKJVbczQ6MoRgoBfb73fIE3lnUcBR/mSI+NDxooOnvbDlTsizo6Wyupgl96mglLoS+lA/ X-Received: by 2002:a17:906:2548:: with SMTP id j8mr11267081ejb.378.1588848156610; Thu, 07 May 2020 03:42:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1588848156; cv=none; d=google.com; s=arc-20160816; b=NRtNJgEE0tPsNKnAwSw1wdUc1OSYeFVss4125wg2nitiv/WlBmrO7ueKbgNYxFNy2C O6I5EkZDs4GypEwgSSWrj6Zgrmfn41ij4kpvtL+5y2dw1SWOIpt/fDxX7BQqXE0Mbu3Y k8HznBK8EmGwRAn4zpMuZXPF20yo6elgB/uLS0MrCXPN1ovLHYb1lnVIIpL22gWy+TBH EDCdaVv1mLVf8+hXH/kyywF01P9pV8oO5rKmFwoKzlRuS/1zSXNqgmdjHdXrNq6iznpR D0P9tD1DVzEFN84XcEmAsJVdwoSpE+1aLbmIb5BClgR/HQic7Rt9LSgqWlY62sip/k9s jD9g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature:dkim-filter; bh=lOxb8f86Kz4Qx/da1Nx9cKkfwC10sWdTZOHVkCSBvsY=; b=Te9tRjXiT3sWWIsLcmTf4dIoKPATs6LATMTh38qQc5kFo3oz+gx4NZd/d9gmjeZrQF iPQUJJrwntykPoRXL+/ZX3btv/7/it09DRjO4ab6lcvXNjf6Kr5x6/nytbb19ktYZzEM 7fC0+78hbAo0b6BD9Tkl8ICY6JQ4OksR3oBKfEX76VjyeGtlDPuPBbiCHP//poymMzXI yFvGdEP2eYiivMUbvvmroWrlSi/lhVB8wBaqCqAC6KDX4Ie40apilowEqt6tT7keBBVq DgGalihVF2qMZ0WYTHq2Fds7rPv2LokUzzUtYQyJGgSz2yRkTSNRR4HujfUUcdZJq+Zd +g+Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@nifty.com header.s=dec2015msa header.b=FxIPosAP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id by4si1048372edb.231.2020.05.07.03.42.12; Thu, 07 May 2020 03:42:36 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@nifty.com header.s=dec2015msa header.b=FxIPosAP; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726218AbgEGKj7 (ORCPT + 99 others); Thu, 7 May 2020 06:39:59 -0400 Received: from conssluserg-03.nifty.com ([210.131.2.82]:19592 "EHLO conssluserg-03.nifty.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725809AbgEGKj7 (ORCPT ); Thu, 7 May 2020 06:39:59 -0400 Received: from mail-vs1-f49.google.com (mail-vs1-f49.google.com [209.85.217.49]) (authenticated) by conssluserg-03.nifty.com with ESMTP id 047AdRIF003000 for ; Thu, 7 May 2020 19:39:28 +0900 DKIM-Filter: OpenDKIM Filter v2.10.3 conssluserg-03.nifty.com 047AdRIF003000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nifty.com; s=dec2015msa; t=1588847968; bh=lOxb8f86Kz4Qx/da1Nx9cKkfwC10sWdTZOHVkCSBvsY=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=FxIPosAPHVFKoVTylyUva1ucqgjg4LPf4iojY6P3QW531JJzC7dH6l109yc916HAL ismKMTRrXALTf3m49Ie4wf1R4KUujOUqss+X3wXc7Vz6YrdkvEHegt2+RXoligaGjk ru+I0JZGfuBbCRHj3azFfYrzs1PhEpkyTf+hn2kjcu6PeM/MOSflM6gCyjP4ZvtLiN Sx1UI67sZsIwLrqQ9Y6eVR7zwHO0lH4AVtCQNUta5fA76WNX/aI9MCm2YP9jIfwYef TjdfQ7K/NOaSszySwxhHsSfHxeg6xXhAVZPQ983C50UHh1nWAxsd79SxNrjwG1OdRS AM68MnHgyjzBA== X-Nifty-SrcIP: [209.85.217.49] Received: by mail-vs1-f49.google.com with SMTP id z1so3038276vsn.11 for ; Thu, 07 May 2020 03:39:28 -0700 (PDT) X-Gm-Message-State: AGi0PuYG03zqCyH5t+y+RE3ubb0YEJ/B1PL5NxEJI10SvB9pW6xQH6X5 wwToEBupgKJcV4y0OojFcm2DzzeUt0sCsUXUc3g= X-Received: by 2002:a67:e94d:: with SMTP id p13mr11206427vso.215.1588847967057; Thu, 07 May 2020 03:39:27 -0700 (PDT) MIME-Version: 1.0 References: <1588250972-8507-1-git-send-email-maninder1.s@samsung.com> <1588250972-8507-4-git-send-email-maninder1.s@samsung.com> In-Reply-To: <1588250972-8507-4-git-send-email-maninder1.s@samsung.com> From: Masahiro Yamada Date: Thu, 7 May 2020 19:38:51 +0900 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 4/4] scripts/checkstack.pl: fix arm sp regex To: Maninder Singh Cc: "George G. Davis" , Linux Kernel Mailing List , a.sahrawat@samsung.com, Vaneet Narang Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Apr 30, 2020 at 9:50 PM Maninder Singh wrote: > > if objdump has below entries; > c01ed608 : > c01ed614: e24ddff7 sub sp, sp, #120 ; 0x78 > > c01f0d50 : > c01f0d50: e24dd094 sub sp, sp, #140 ; 0x8c > > scripts fails to read stack usage. > so making regex $re for ARM similar to aarch64 > > Signed-off-by: Vaneet Narang > Signed-off-by: Maninder Singh This looks good to me, and it is a bug fix. Just a question about the SOB. Maninder Singh is the author and also the submitter, right? What does "Signed-off-by: Vaneet Narang " mean? Co-developed-by or something else? > --- > scripts/checkstack.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/checkstack.pl b/scripts/checkstack.pl > index b292ef4..e80de70 100755 > --- a/scripts/checkstack.pl > +++ b/scripts/checkstack.pl > @@ -60,7 +60,7 @@ my (@stack, $re, $dre, $sub, $x, $xs, $funcre, $min_stack); > $dre = qr/^.*sub.*sp, sp, #(0x$x{1,8})/o; > } elsif ($arch eq 'arm') { > #c0008ffc: e24dd064 sub sp, sp, #100 ; 0x64 > - $re = qr/.*sub.*sp, sp, #(([0-9]{2}|[3-9])[0-9]{2})/o; > + $re = qr/.*sub.*sp, sp, #([0-9]{1,4})/o; > $sub = \&arm_push_handling; > } elsif ($arch =~ /^x86(_64)?$/ || $arch =~ /^i[3456]86$/) { > #c0105234: 81 ec ac 05 00 00 sub $0x5ac,%esp > -- > 1.9.1 > -- Best Regards Masahiro Yamada