From: Jason Gunthorpe <jgg@nvidia.com>
To: Jani Nikula <jani.nikula@intel.com>
Cc: Masahiro Yamada <masahiroy@kernel.org>,
Linus Torvalds <torvalds@linux-foundation.org>,
linux-kernel@vger.kernel.org, David Airlie <airlied@gmail.com>,
Simona Vetter <simona.vetter@ffwll.ch>,
linux-kbuild@vger.kernel.org, dri-devel@lists.freedesktop.org,
intel-xe@lists.freedesktop.org, intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH v2 0/4] kbuild: resurrect generic header check facility
Date: Mon, 7 Apr 2025 14:12:09 -0300 [thread overview]
Message-ID: <20250407171209.GJ1557073@nvidia.com> (raw)
In-Reply-To: <878qoczbhn.fsf@intel.com>
On Mon, Apr 07, 2025 at 10:17:40AM +0300, Jani Nikula wrote:
> Even with Jason's idea [1], you *still* have to start small and opt-in
> (i.e. the patch series at hand). You can't just start off by testing
> every header in one go, because it's a flag day switch.
You'd add something like 'make header_check' that does not run
automatically. Making it run automatically after everything is fixed
to keep it fixed would be the flag day change. It is how we have
managed to introduce other warning levels in the past.
If you added the infrastructure there is a whole list of people on
kernel-janitors that would probably help with the trivial cleanups to
make it run clean.
> With this type of antagonistic rather than encouraging attitude towards
> contributions, there's just no way I can justify to myself (or my
> employer) spending more time on what looks like a wild goose chase. I
> have zero confidence that no matter what I do I'd get approval from you.
I think you've been given a clear direction on what would be accepted
and have the option to persue it. Claiming that is "antagonistic"
seems unnecessary.
> And this is the primary reason subsystems and drivers hack up stuff in
> their little corners of the kernel instead of sticking their necks out
> and trying to generalize anything.
Seems to me like this is the usual case of generalizing being actually
hard, you almost always have to actually do more work to succeed.
Jason
next prev parent reply other threads:[~2025-04-07 17:12 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-02 12:46 [PATCH v2 0/4] kbuild: resurrect generic header check facility Jani Nikula
2025-04-02 12:46 ` [PATCH v2 1/4] kbuild: add " Jani Nikula
2025-04-02 12:46 ` [PATCH v2 2/4] drm: switch to " Jani Nikula
2025-04-02 12:46 ` [PATCH v2 3/4] drm/i915: " Jani Nikula
2025-04-02 12:46 ` [PATCH v2 4/4] drm/xe: " Jani Nikula
2025-04-02 16:06 ` [PATCH v2 0/4] kbuild: resurrect " Linus Torvalds
2025-04-04 6:17 ` Masahiro Yamada
2025-04-07 7:17 ` Jani Nikula
2025-04-07 17:12 ` Jason Gunthorpe [this message]
2025-04-08 8:27 ` Jani Nikula
2025-04-08 16:01 ` Jason Gunthorpe
2025-04-08 18:42 ` Jani Nikula
2025-04-08 20:15 ` Jason Gunthorpe
2025-04-08 19:48 ` Linus Torvalds
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250407171209.GJ1557073@nvidia.com \
--to=jgg@nvidia.com \
--cc=airlied@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
--cc=intel-xe@lists.freedesktop.org \
--cc=jani.nikula@intel.com \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=masahiroy@kernel.org \
--cc=simona.vetter@ffwll.ch \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox