Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <201912060818.dgGGxpRK%lkp@intel.com> <CAK8P3a23_rWdmjyZNezd9k=-KL4VZyV+DCLvh-UgCQUQTsysyw@mail.gmail.com>
 <CAEbi=3fN83s1sp6Yt2B6d5M-uJ+TGz_a7-mWTt2LAfrX8B3JmA@mail.gmail.com>
 <CAEbi=3cojt1shc84zH3tGtxK98rQ=PbGhkirWCe7C+JSj5jpFw@mail.gmail.com> <CAEbi=3fJM9B_UnkuVOKW2oD8AcdvOYh1ozheGi9pAGvm7MjtFw@mail.gmail.com>
In-Reply-To: <CAEbi=3fJM9B_UnkuVOKW2oD8AcdvOYh1ozheGi9pAGvm7MjtFw@mail.gmail.com>
From:   Greentime Hu <green.hu@gmail.com>
Date:   Mon, 9 Dec 2019 15:08:24 +0800
Message-ID: <CAEbi=3cX1g6cJhXPRp6bP2fT4Q=LJ38YuCBCrunU6sR-5mo+pQ@mail.gmail.com>
Subject: Re: drivers/scsi/.tmp_mc_st.s:3: Error: invalid operands (*UND* and
 *UND* sections) for `^'
To:     Arnd Bergmann <arnd@arndb.de>, Nickhu <nickhu@andestech.com>,
        nelson.chu1990@gmail.com
Cc:     kbuild test robot <lkp@intel.com>, kbuild-all@lists.01.org,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Vincent Chen <deanbo422@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Greentime Hu <green.hu@gmail.com> =E6=96=BC 2019=E5=B9=B412=E6=9C=886=E6=97=
=A5 =E9=80=B1=E4=BA=94 =E4=B8=8B=E5=8D=886:15=E5=AF=AB=E9=81=93=EF=BC=9A
>
> Greentime Hu <green.hu@gmail.com> =E6=96=BC 2019=E5=B9=B412=E6=9C=886=E6=
=97=A5 =E9=80=B1=E4=BA=94 =E4=B8=8B=E5=8D=884:38=E5=AF=AB=E9=81=93=EF=BC=9A
> >
> > Greentime Hu <green.hu@gmail.com> =E6=96=BC 2019=E5=B9=B412=E6=9C=886=
=E6=97=A5 =E9=80=B1=E4=BA=94 =E4=B8=8B=E5=8D=884:19=E5=AF=AB=E9=81=93=EF=BC=
=9A
> > >
> > > Arnd Bergmann <arnd@arndb.de> =E6=96=BC 2019=E5=B9=B412=E6=9C=886=E6=
=97=A5 =E9=80=B1=E4=BA=94 =E4=B8=8B=E5=8D=884:00=E5=AF=AB=E9=81=93=EF=BC=9A
> > > >
> > > > On Fri, Dec 6, 2019 at 2:05 AM kbuild test robot <lkp@intel.com> wr=
ote:
> > > > >
> > > > > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/=
linux.git master
> > > > > head:   2f13437b8917627119d163d62f73e7a78a92303a
> > > > > commit: 1207045da5a7c94344e0ea9a9e7495985eef499a compat_ioctl: mo=
ve tape handling into drivers
> > > > > date:   6 weeks ago
> > > > > config: nds32-allyesconfig (attached as .config)
> > > > > compiler: nds32le-linux-gcc (GCC) 9.2.0
> > > > > reproduce:
> > > > >         wget https://raw.githubusercontent.com/intel/lkp-tests/ma=
ster/sbin/make.cross -O ~/bin/make.cross
> > > > >         chmod +x ~/bin/make.cross
> > > > >         git checkout 1207045da5a7c94344e0ea9a9e7495985eef499a
> > > > >         # save the attached .config to linux build tree
> > > > >         GCC_VERSION=3D9.2.0 make.cross ARCH=3Dnds32
> > > > >
> > > > > If you fix the issue, kindly add following tag
> > > > > Reported-by: kbuild test robot <lkp@intel.com>
> > > > >
> > > > > All errors (new ones prefixed by >>):
> > > > >
> > > > >    drivers/scsi/.tmp_mc_st.s: Assembler messages:
> > > > > >> drivers/scsi/.tmp_mc_st.s:3: Error: invalid operands (*UND* an=
d *UND* sections) for `^'
> > > > >    drivers/scsi/.tmp_mc_st.s:4: Error: invalid operands (*UND* an=
d *UND* sections) for `^'
> > > > >    drivers/scsi/.tmp_mc_st.s:5: Error: invalid operands (*UND* an=
d *UND* sections) for `^'
> > > > >    drivers/scsi/.tmp_mc_st.s:6: Error: invalid operands (*UND* an=
d *UND* sections) for `^'
> > > > >    drivers/scsi/.tmp_mc_st.s:7: Error: invalid operands (*UND* an=
d *UND* sections) for `^'
> > > > >    drivers/scsi/.tmp_mc_st.s:8: Error: invalid operands (*UND* an=
d *UND* sections) for `^'
> > > > >    drivers/scsi/.tmp_mc_st.s:9: Error: invalid operands (*UND* an=
d *UND* sections) for `^'
> > > >
> > > > Adding nds32 maintainers to Cc:
> > > >
> > > > It looks like a regression caused by my patch, but I don't think it=
's something
> > > > I did wrong, but rather a toolchain bug being uncovered by the modi=
fied sources.
> > > >
> > > > Are you able to reproduce this?
> > >
> > > Hi Arnd,
> > >
> > > I am trying to reproduce this problem, but it happened to me.
> > >
> > > greentimeh@gamma07:/scratch/greentimeh/nds32/linux <(1207045da5a7...)=
>
> > > $ GCC_VERSION=3D9.2.0 make.cross ARCH=3Dnds32
> > > cd: received redirection to `https://download.01.org/0day-ci/cross-pa=
ckage/'
> > > Cannot find nds32-linux under
> > > https://download.01.org/0day-ci/cross-package check
> > > /tmp/crosstool-files
> > >
> > > Can you reproduce it?
> >
> > I can reproduce it now by adding these 2 lines to make.cross.
> > 177                 nds32)
> > 178                         gcc_arch=3Dnds32le-linux
> >
> > It will be built failed with gcc-9.2.0-nolibc toolchain, but it can be
> > built pass with gcc-8.1.0-nolibc toolchain.
>
> It seems nds32le-linux-objdump -hdr xxx.o will generate different strings=
.
> For example:
> 00000050 <.L2^B1>:
> This string is different when generated by 9.2 toolchain and 8.1 toolchai=
n.
> "^B" is treated as 2 character in 9.2 toolchain
> "^B" is treated as 1 character in 8.1 toolchain
>
> This causes ./scripts/recordmcount.pl to call `$cc -o $mcount_o -c
> $mcount_s`; failed to cause this issue.
> I have a quick patch to fix this build error, but I am not sure if it
> is a good solution.
> Maybe Nick could have a look at it?
>
> diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl
> index 3f77a5d695c1..807b61fc5f5d 100755
> --- a/scripts/recordmcount.pl
> +++ b/scripts/recordmcount.pl
> @@ -551,7 +551,8 @@ while (<IN>) {
>         # if this is either a local function or a weak function
>         # keep looking for functions that are global that
>         # we can use safely.
> -       if (!defined($locals{$text}) && !defined($weak{$text})) {
> +       if (!defined($locals{$text}) && !defined($weak{$text}) &&
> +           $text !~ /^\.L/) {
>             $ref_func =3D $text;
>             $read_function =3D 0;
>             $offset =3D hex $1;

Add Nelson Chu to Cc.
He found this commit in binutil to cause the different "^B".
91d6fa6a (Nick Clifton 2009-12-11 13:42:17 +0000 1273)
(*inf->fprintf_func) (inf->stream, " <%s",
fd361982 (Alan Modra 2019-09-16 20:25:17 +0930 1274) sanitize_string
(bfd_section_name (sec)));