Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20220713194112.15557-1-wafgo01@gmail.com> <fa77d640-a18d-ca77-ba57-6735094a224f@microchip.com>
In-Reply-To: <fa77d640-a18d-ca77-ba57-6735094a224f@microchip.com>
From:   Masahiro Yamada <masahiroy@kernel.org>
Date:   Tue, 26 Jul 2022 13:47:47 +0900
Message-ID: <CAK7LNARu4OU_hGQHu8B7F=5jLzU1jp7L=RHVcyo3eLkEyWNcQw@mail.gmail.com>
Subject: Re: [PATCH] checkstack: add riscv support for scripts/checkstack.pl
To:     Conor.Dooley@microchip.com
Cc:     wafgo01@gmail.com, Paul Walmsley <paul.walmsley@sifive.com>,
        Palmer Dabbelt <palmer@dabbelt.com>,
        Albert Ou <aou@eecs.berkeley.edu>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        "open list:SIFIVE DRIVERS" <linux-riscv@lists.infradead.org>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Tue, Jul 26, 2022 at 2:40 AM <Conor.Dooley@microchip.com> wrote:
>
> On 13/07/2022 20:41, Wadim Mueller wrote:
> > [You don't often get email from wafgo01@gmail.com. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
> >
>
> I feel like you might've been mislead by get_maintainer for this
> patch. It looks like the maintainer covering this is Masahiro
> Yamada <masahiroy@kernel.org> (at least he's taken the last few
> patches for it).


I can pick this up.

Applied to linux-kbuild. Thanks.


>
> I'll be honest, I don't know how to verify that the output is
> correct - but I did run it & it does output something that looks
> sane /shrug
>
> Thanks,
> Conor.
>
> > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> >
> > scripts/checkstack.pl lacks support for the riscv architecture. Add
> > support to detect "addi sp,sp,-FRAME_SIZE" stack frame generation instruction
> >
> > Signed-off-by: Wadim Mueller <wafgo01@gmail.com>
> > ---
> >  scripts/checkstack.pl | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/scripts/checkstack.pl b/scripts/checkstack.pl
> > index d2c38584ece6..d48dfed6d3db 100755
> > --- a/scripts/checkstack.pl
> > +++ b/scripts/checkstack.pl
> > @@ -16,6 +16,7 @@
> >  #      AArch64, PARISC ports by Kyle McMartin
> >  #      sparc port by Martin Habets <errandir_news@mph.eclipse.co.uk>
> >  #      ppc64le port by Breno Leitao <leitao@debian.org>
> > +#      riscv port by Wadim Mueller <wafgo01@gmail.com>
> >  #
> >  #      Usage:
> >  #      objdump -d vmlinux | scripts/checkstack.pl [arch]
> > @@ -108,6 +109,9 @@ my (@stack, $re, $dre, $sub, $x, $xs, $funcre, $min_stack);
> >         } elsif ($arch eq 'sparc' || $arch eq 'sparc64') {
> >                 # f0019d10:       9d e3 bf 90     save  %sp, -112, %sp
> >                 $re = qr/.*save.*%sp, -(([0-9]{2}|[3-9])[0-9]{2}), %sp/o;
> > +       } elsif ($arch =~ /^riscv(64)?$/) {
> > +               #ffffffff8036e868:      c2010113                addi    sp,sp,-992
> > +               $re = qr/.*addi.*sp,sp,-(([0-9]{2}|[3-9])[0-9]{2})/o;
> >         } else {
> >                 print("wrong or unknown architecture \"$arch\"\n");
> >                 exit
> > --
> > 2.25.1
> >
> >
> > _______________________________________________
> > linux-riscv mailing list
> > linux-riscv@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/linux-riscv
>


-- 
Best Regards
Masahiro Yamada