The arm and arm64 with gcc-8 builds failed on Linux next-20240507 tag
with gcc-8 due to following warnings / errors.
arm64 and arm:
defconfig - gcc-8 - failed
Reported-by: Linux Kernel Functional Testing <[email protected]>
Build log:
---------
arch/arm64/boot/dts/rockchip/rk3568-wolfvision-pf5-display.dtsi:113.6-121.3:
Warning (graph_port): /fragment@4/__overlay__: graph port node name
should be 'port'
Traceback (most recent call last):
File "drivers/gpu/drm/msm/registers/gen_header.py", line 970, in <module>
main()
File "drivers/gpu/drm/msm/registers/gen_header.py", line 951, in main
parser.add_argument('--validate', action=argparse.BooleanOptionalAction)
AttributeError: module 'argparse' has no attribute 'BooleanOptionalAction'
metadata:
-----
git_describe: next-20240507
git_repo: https://gitlab.com/Linaro/lkft/mirrors/next/linux-next
git_sha: 93a39e4766083050ca0ecd6a3548093a3b9eb60c
git_short_log: 93a39e476608 ("Add linux-next specific files for 20240507")
Links:
---
- https://qa-reports.linaro.org/lkft/linux-next-master/build/next-20240507/testrun/23817782/suite/build/tests/
- https://qa-reports.linaro.org/lkft/linux-next-master/build/next-20240507/testrun/23818260/suite/build/test/gcc-8-defconfig/details/
--
Linaro LKFT
https://lkft.linaro.org
On Tue, 7 May 2024 at 17:13, Naresh Kamboju <[email protected]> wrote:
>
> The arm and arm64 with gcc-8 builds failed on Linux next-20240507 tag
> with gcc-8 due to following warnings / errors.
>
> arm64 and arm:
> defconfig - gcc-8 - failed
Anders bisected this build problem and found the first bad commit:
07a2f8716c41 drm/msm/gen_header: allow skipping the validation
Steps to reproduce:
-----
# tuxmake --runtime podman --target-arch arm64 --toolchain gcc-8
--kconfig defconfig
--
Linaro LKFT
https://lkft.linaro.org
On Tue, 7 May 2024 at 16:13, Naresh Kamboju <[email protected]> wrote:
>
> On Tue, 7 May 2024 at 17:13, Naresh Kamboju <[email protected]> wrote:
> >
> > The arm and arm64 with gcc-8 builds failed on Linux next-20240507 tag
> > with gcc-8 due to following warnings / errors.
> >
> > arm64 and arm:
> > defconfig - gcc-8 - failed
>
> Anders bisected this build problem and found the first bad commit:
>
> 07a2f8716c41 drm/msm/gen_header: allow skipping the validation
>
> Steps to reproduce:
> -----
> # tuxmake --runtime podman --target-arch arm64 --toolchain gcc-8
> --kconfig defconfig
What is the python version on that platform / system?
--
With best wishes
Dmitry
On Tue, 7 May 2024 at 20:58, Dmitry Baryshkov
<[email protected]> wrote:
>
> On Tue, 7 May 2024 at 16:13, Naresh Kamboju <[email protected]> wrote:
> >
> > On Tue, 7 May 2024 at 17:13, Naresh Kamboju <[email protected]> wrote:
> > >
> > > The arm and arm64 with gcc-8 builds failed on Linux next-20240507 tag
> > > with gcc-8 due to following warnings / errors.
> > >
> > > arm64 and arm:
> > > defconfig - gcc-8 - failed
> >
> > Anders bisected this build problem and found the first bad commit:
> >
> > 07a2f8716c41 drm/msm/gen_header: allow skipping the validation
> >
> > Steps to reproduce:
> > -----
> > # tuxmake --runtime podman --target-arch arm64 --toolchain gcc-8
> > --kconfig defconfig
>
> What is the python version on that platform / system?
# python --version
Python 3.7.3
> --
> With best wishes
> Dmitry
- Naresh
On Tue, 7 May 2024 at 21:06, Naresh Kamboju <[email protected]> wrote:
>
> On Tue, 7 May 2024 at 20:58, Dmitry Baryshkov
> <[email protected]> wrote:
> >
> > On Tue, 7 May 2024 at 16:13, Naresh Kamboju <[email protected]> wrote:
> > >
> > > On Tue, 7 May 2024 at 17:13, Naresh Kamboju <[email protected]> wrote:
> > > >
> > > > The arm and arm64 with gcc-8 builds failed on Linux next-20240507 tag
> > > > with gcc-8 due to following warnings / errors.
> > > >
> > > > arm64 and arm:
> > > > defconfig - gcc-8 - failed
> > >
> > > Anders bisected this build problem and found the first bad commit:
> > >
> > > 07a2f8716c41 drm/msm/gen_header: allow skipping the validation
> > >
> > > Steps to reproduce:
> > > -----
> > > # tuxmake --runtime podman --target-arch arm64 --toolchain gcc-8
> > > --kconfig defconfig
> >
> > What is the python version on that platform / system?
>
> # python --version
> Python 3.7.3
EOL.
I'll check what we can do, but please consider updating the platform.
--
With best wishes
Dmitry
On Tue, May 7, 2024, at 20:19, Dmitry Baryshkov wrote:
> On Tue, 7 May 2024 at 21:06, Naresh Kamboju <[email protected]> wrote:
>> On Tue, 7 May 2024 at 20:58, Dmitry Baryshkov
>> <[email protected]> wrote:
>> >
>> > On Tue, 7 May 2024 at 16:13, Naresh Kamboju <[email protected]> wrote:
>> > >
>> > > On Tue, 7 May 2024 at 17:13, Naresh Kamboju <[email protected]> wrote:
>> > > >
>> > > > The arm and arm64 with gcc-8 builds failed on Linux next-20240507 tag
>> > > > with gcc-8 due to following warnings / errors.
>> > > >
>> > > > arm64 and arm:
>> > > > defconfig - gcc-8 - failed
>> > >
>> > > Anders bisected this build problem and found the first bad commit:
>> > >
>> > > 07a2f8716c41 drm/msm/gen_header: allow skipping the validation
>> > >
>> > > Steps to reproduce:
>> > > -----
>> > > # tuxmake --runtime podman --target-arch arm64 --toolchain gcc-8
>> > > --kconfig defconfig
>> >
>> > What is the python version on that platform / system?
>>
>> # python --version
>> Python 3.7.3
>
> EOL.
> I'll check what we can do, but please consider updating the platform.
I don't think we can rely on everyone to have 3.9, 3.8
is not EOL yet, and we do want to still test in distros with
older compilers that may also come with old python.
It should just work with a regular optional argument instead
of the BooleanOptionalAction (without the --no-validate flag):
https://docs.python.org/3/howto/argparse.html#introducing-optional-arguments
> @@ -538,6 +538,9 @@ class Parser(object):
> self.variants.add(reg.domain)
>
> def do_validate(self, schemafile):
> + if self.validate == False:
> + return
> +
> try:
> from lxml import etree
This bit may have to become 'if not self.validate',
since the flag in this case is None rather than equal
to False. This also improves readability.
Arnd
On Tue, 7 May 2024 at 22:18, Arnd Bergmann <[email protected]> wrote:
>
> On Tue, May 7, 2024, at 20:19, Dmitry Baryshkov wrote:
> > On Tue, 7 May 2024 at 21:06, Naresh Kamboju <[email protected]> wrote:
> >> On Tue, 7 May 2024 at 20:58, Dmitry Baryshkov
> >> <[email protected]> wrote:
> >> >
> >> > On Tue, 7 May 2024 at 16:13, Naresh Kamboju <[email protected]> wrote:
> >> > >
> >> > > On Tue, 7 May 2024 at 17:13, Naresh Kamboju <[email protected]> wrote:
> >> > > >
> >> > > > The arm and arm64 with gcc-8 builds failed on Linux next-20240507 tag
> >> > > > with gcc-8 due to following warnings / errors.
> >> > > >
> >> > > > arm64 and arm:
> >> > > > defconfig - gcc-8 - failed
> >> > >
> >> > > Anders bisected this build problem and found the first bad commit:
> >> > >
> >> > > 07a2f8716c41 drm/msm/gen_header: allow skipping the validation
> >> > >
> >> > > Steps to reproduce:
> >> > > -----
> >> > > # tuxmake --runtime podman --target-arch arm64 --toolchain gcc-8
> >> > > --kconfig defconfig
> >> >
> >> > What is the python version on that platform / system?
> >>
> >> # python --version
> >> Python 3.7.3
> >
> > EOL.
> > I'll check what we can do, but please consider updating the platform.
>
> I don't think we can rely on everyone to have 3.9, 3.8
> is not EOL yet, and we do want to still test in distros with
> older compilers that may also come with old python.
>
> It should just work with a regular optional argument instead
> of the BooleanOptionalAction (without the --no-validate flag):
>
> https://docs.python.org/3/howto/argparse.html#introducing-optional-arguments
>
> > @@ -538,6 +538,9 @@ class Parser(object):
> > self.variants.add(reg.domain)
> >
> > def do_validate(self, schemafile):
> > + if self.validate == False:
> > + return
> > +
> > try:
> > from lxml import etree
>
> This bit may have to become 'if not self.validate',
> since the flag in this case is None rather than equal
> to False. This also improves readability.
Could you please send a patch?
--
With best wishes
Dmitry