lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241122204157.GA125569@pevik>
Date: Fri, 22 Nov 2024 21:41:57 +0100
From: Petr Vorel <pvorel@...e.cz>
To: Masahiro Yamada <masahiroy@...nel.org>
Cc: Rob Clark <robdclark@...il.com>, linux-arm-msm@...r.kernel.org,
	Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	linux-kbuild@...r.kernel.org, dri-devel@...ts.freedesktop.org,
	freedreno@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 1/2] init/Kconfig: add python3 availability config

> On Thu, Nov 21, 2024 at 10:49 AM Rob Clark <robdclark@...il.com> wrote:

> > On Wed, Nov 20, 2024 at 5:17 PM Petr Vorel <pvorel@...e.cz> wrote:

> > > > On Thu, Nov 21, 2024 at 5:41 AM Petr Vorel <pvorel@...e.cz> wrote:

> > > > > It will be used in the next commit for DRM_MSM.

> > > > > Suggested-by: Rob Clark <robdclark@...il.com>
> > > > > Signed-off-by: Petr Vorel <pvorel@...e.cz>
> > > > > ---
> > > > > Changes v3->v4:
> > > > > * Move definition to the end of the file


> > > > I prefer to not check the tool.

> > > Ack.

> > > > Why don't you install python3?

> > > Everybody installs it when it's required, the question is how to inform about
> > > the dependency.

> > > There build environments are minimal environments:
> > > * chroot (e.g. cross compilation)
> > > * container

> > > These are used by both developers and distros.

> > I don't think py3 is an _onerous_ dependency, but it has come up as a
> > surprise in minimal distro build environments at least once.. so I'd
> > be a fan of surfacing this dependency in a predictable/understandable
> > way (ie. I'm in favor of this patchset)


> "once" is a keyword here.

> "/bin/sh: python3: not found" provides sufficient information
> about why the compilation failed, and you know what to do
> to fix the problem.
> This is good.

> If you hide CONFIG_DRM_MSM silently
> due to missing python3, you may scratch your head
> "why drm/msm was not compiled?".
It's not on the list, but still visible in help (via search).

> This is worse.

I'm ok with this being refused. Yes, it's a trivial thing to find that python3
is not installed. I wasn't sure myself if this is really better. Having
something like "requires $(PYTHON3)" would be best solution (e.g. not disable
the config, but exit before starting to build), but of course unless this
feature is needed for many modules it does not make sense to have it.
It's because kernel mostly contains everything (unless languages like python
or any other dependency starts to be added). For this reason I like that
mconf-cfg.sh warns when missing ncurses devel files (even suggesting package
names).

Just to explain what was my motivation. CONFIG_DRM_MSM in in arm64 defconfig,
thus it will affect anybody who uses the defconfig (any distro will need to add
it).

It's needed only for Qualcomm arm64 devices only. But only for these devices
which are mainlined enough to really use CONFIG_DRM_MSM (many of them aren't in
that state).

postmarketOS is the distribution which supports Qualcomm. It stores kernel
config for each device and devices often have individual maintainer. E.g. 175x
"once" :).

Kind regards,
Petr

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ