Re: [PATCH] checkstack: add riscv support for scripts/checkstack.pl

[Wadim Mueller]

Hi Conor, Hi Masahiro,

thank you for your support. Your help is much appreciated.

Best Regards
Wadim

On Tue, Jul 26, 2022 at 01:47:47PM +0900, Masahiro Yamada wrote:
> On Tue, Jul 26, 2022 at 2:40 AM <Conor.Dooley@xxxxxxxxxxxxx> wrote:
> >
> > On 13/07/2022 20:41, Wadim Mueller wrote:
> > > [You don't often get email from wafgo01@xxxxxxxxx. 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@xxxxxxxxxx> (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@xxxxxxxxx>
> > > ---
> > >  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@xxxxxxxxxxxxxxxxx>
> > >  #      ppc64le port by Breno Leitao <leitao@xxxxxxxxxx>
> > > +#      riscv port by Wadim Mueller <wafgo01@xxxxxxxxx>
> > >  #
> > >  #      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@xxxxxxxxxxxxxxxxxxx
> > > http://lists.infradead.org/mailman/listinfo/linux-riscv
> >
> 
> 
> -- 
> Best Regards
> Masahiro Yamada