linux-kernel - Re: [PATCH v1 2/3] drm/ci: Add jobs to validate devicetrees

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: <20250401-devious-victorious-bullfinch-bab3a6@houat>
Date: Tue, 1 Apr 2025 17:14:55 +0200
From: Maxime Ripard <mripard@...nel.org>
To: Vignesh Raman <vignesh.raman@...labora.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>, 
	dri-devel@...ts.freedesktop.org, daniels@...labora.com, helen.fornazier@...il.com, 
	airlied@...il.com, simona.vetter@...ll.ch, robdclark@...il.com, 
	guilherme.gallo@...labora.com, sergi.blanch.torne@...labora.com, valentine.burley@...labora.com, 
	lumag@...nel.org, quic_abhinavk@...cinc.com, maarten.lankhorst@...ux.intel.com, 
	tzimmermann@...e.de, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 2/3] drm/ci: Add jobs to validate devicetrees

On Tue, Apr 01, 2025 at 07:26:44AM +0530, Vignesh Raman wrote:
> Hi Dmitry,
> 
> On 30/03/25 22:36, Dmitry Baryshkov wrote:
> > On Thu, Mar 27, 2025 at 09:31:11PM +0530, Vignesh Raman wrote:
> > > Add jobs to run dt_binding_check and dtbs_check. If warnings are seen,
> > > exit with a non-zero error code while configuring them as warning in
> > > the GitLab CI pipeline.
> > 
> > Can it really succeed or is it going to be an always-failing job? The
> > dt_binding_check generally succeed, dtbs_check generates tons of
> > warnings. We are trying to make progress there, but it's still very far
> > from being achevable.
> 
> Even though it fails, it will be shown as a warning in the pipeline.
> https://gitlab.freedesktop.org/vigneshraman/linux/-/pipelines/1390797

I think what Dmitry was pointing out is that a warning that is always
warning is going to be completely ignored, and thus doesn't add any
value.

Maxime

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)