From: Daniel Vetter <daniel@ffwll.ch> To: Masahiro Yamada <masahiroy@kernel.org> Cc: Stephen Rothwell <sfr@canb.auug.org.au>, intel-gfx <intel-gfx@lists.freedesktop.org>, Joonas Lahtinen <joonas.lahtinen@linux.intel.com>, "Nikula, Jani" <jani.nikula@linux.intel.com>, Rodrigo Vivi <rodrigo.vivi@intel.com>, Dave Airlie <airlied@linux.ie>, DRI <dri-devel@lists.freedesktop.org>, John Harrison <John.C.Harrison@intel.com>, Matthew Brost <matthew.brost@intel.com>, Alexey Dobriyan <adobriyan@gmail.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Linux Next Mailing List <linux-next@vger.kernel.org> Subject: Re: linux-next: build failure after merge of the drm tree Date: Wed, 8 Sep 2021 11:07:03 +0200 [thread overview] Message-ID: <CAKMK7uEjGA01bhPYVJt5aHfvh1i6roV0w-P5Bq9-tohS65esdA@mail.gmail.com> (raw) In-Reply-To: <CAK7LNAQdgr7pn0j9mdAGfB_0fGOVMn+uq=Kv7buRaCOcoF+p7A@mail.gmail.com> On Wed, Sep 8, 2021 at 5:14 AM Masahiro Yamada <masahiroy@kernel.org> wrote: > > On Mon, Sep 6, 2021 at 4:34 PM Daniel Vetter <daniel@ffwll.ch> wrote: > > > > On Mon, Sep 6, 2021 at 12:49 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > Hi all, > > > > > > On Thu, 2 Sep 2021 07:50:38 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > > > > > On Fri, 20 Aug 2021 15:23:34 +0900 Masahiro Yamada <masahiroy@kernel.org> wrote: > > > > > > > > > > On Fri, Aug 20, 2021 at 11:33 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > > > > > > > > After merging the drm tree, today's linux-next build (x86_64 allmodconfig) > > > > > > failed like this: > > > > > > > > > > > > In file included from drivers/gpu/drm/i915/i915_debugfs.c:39: > > > > > > drivers/gpu/drm/i915/gt/intel_gt_requests.h:9:10: fatal error: stddef.h: No such file or directory > > > > > > 9 | #include <stddef.h> > > > > > > | ^~~~~~~~~~ > > > > > > > > > > > > Caused by commit > > > > > > > > > > > > 564f963eabd1 ("isystem: delete global -isystem compile option") > > > > > > > > > > > > from the kbuild tree interacting with commit > > > > > > > > > > > > b97060a99b01 ("drm/i915/guc: Update intel_gt_wait_for_idle to work with GuC") > > > > > > > > > > > > I have applied the following patch for today. > > > > > > > > > > > > > > > Thanks. > > > > > > > > > > This fix-up does not depend on my kbuild tree in any way. > > > > > > > > > > So, the drm maintainer can apply it to his tree. > > > > > > > > > > Perhaps with > > > > > > > > > > Fixes: b97060a99b01 ("drm/i915/guc: Update intel_gt_wait_for_idle to > > > > > work with GuC") > > > > > > > > OK, so that didn't happen so I will now apply the merge fix up to the > > > > merge of the kbuild tree. > > > > > > > > > > From: Stephen Rothwell <sfr@canb.auug.org.au> > > > > > > Date: Fri, 20 Aug 2021 12:24:19 +1000 > > > > > > Subject: [PATCH] drm/i915: use linux/stddef.h due to "isystem: trim/fixup stdarg.h and other headers" > > > > > > > > > > > > Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> > > > > > > --- > > > > > > drivers/gpu/drm/i915/gt/intel_gt_requests.h | 2 +- > > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > > > diff --git a/drivers/gpu/drm/i915/gt/intel_gt_requests.h b/drivers/gpu/drm/i915/gt/intel_gt_requests.h > > > > > > index 51dbe0e3294e..d2969f68dd64 100644 > > > > > > --- a/drivers/gpu/drm/i915/gt/intel_gt_requests.h > > > > > > +++ b/drivers/gpu/drm/i915/gt/intel_gt_requests.h > > > > > > @@ -6,7 +6,7 @@ > > > > > > #ifndef INTEL_GT_REQUESTS_H > > > > > > #define INTEL_GT_REQUESTS_H > > > > > > > > > > > > -#include <stddef.h> > > > > > > +#include <linux/stddef.h> > > > > > > > > > > > > struct intel_engine_cs; > > > > > > struct intel_gt; > > > > > > -- > > > > > > 2.32.0 > > > > > > Ping? I am still applying this ... > > > > Apologies, this fell through a lot of cracks. I applied this to drm-next now. > > > > Rather, I was planning to apply this fix to my kbuild tree. > > Since you guys did not fix the issue in time, > I ended up with dropping [1] from my pull request. > > I want to get [1] merged in this MW. > > If I postponed it, somebody would add new > <stddef.h> or <stdint.h> inclusion in the next development > cycle, I will never make it in the mainline. > > [1] https://lore.kernel.org/linux-kernel/YQhY40teUJcTc5H4@localhost.localdomain/ Yeah no problem if you apply it too. For that: Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> I just figured I make sure this is at least not lost. -Daniel > > > > > > > Matt/John, as author/committer it's your job to make sure issues and > > fixes for the stuff you're pushing don't get lost. I'd have expected > > John to apply this to at least drm-intel-gt-next (it's not even > > there). > > > > Joonas, I think this is the 2nd or 3rd or so issue this release cycle > > where some compile fix got stuck a bit because drm-intel-gt-next isn't > > in linux-next. Can we please fix that? It probably needs some changes > > to the dim script. > > > > Cheers, Daniel > > -- > > Daniel Vetter > > Software Engineer, Intel Corporation > > http://blog.ffwll.ch > > > > -- > Best Regards > Masahiro Yamada -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Vetter <daniel@ffwll.ch> To: Masahiro Yamada <masahiroy@kernel.org> Cc: Stephen Rothwell <sfr@canb.auug.org.au>, intel-gfx <intel-gfx@lists.freedesktop.org>, Joonas Lahtinen <joonas.lahtinen@linux.intel.com>, "Nikula, Jani" <jani.nikula@linux.intel.com>, Rodrigo Vivi <rodrigo.vivi@intel.com>, Dave Airlie <airlied@linux.ie>, DRI <dri-devel@lists.freedesktop.org>, John Harrison <John.C.Harrison@intel.com>, Matthew Brost <matthew.brost@intel.com>, Alexey Dobriyan <adobriyan@gmail.com>, Linux Kernel Mailing List <linux-kernel@vger.kernel.org>, Linux Next Mailing List <linux-next@vger.kernel.org> Subject: Re: [Intel-gfx] linux-next: build failure after merge of the drm tree Date: Wed, 8 Sep 2021 11:07:03 +0200 [thread overview] Message-ID: <CAKMK7uEjGA01bhPYVJt5aHfvh1i6roV0w-P5Bq9-tohS65esdA@mail.gmail.com> (raw) In-Reply-To: <CAK7LNAQdgr7pn0j9mdAGfB_0fGOVMn+uq=Kv7buRaCOcoF+p7A@mail.gmail.com> On Wed, Sep 8, 2021 at 5:14 AM Masahiro Yamada <masahiroy@kernel.org> wrote: > > On Mon, Sep 6, 2021 at 4:34 PM Daniel Vetter <daniel@ffwll.ch> wrote: > > > > On Mon, Sep 6, 2021 at 12:49 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > Hi all, > > > > > > On Thu, 2 Sep 2021 07:50:38 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > > > > > On Fri, 20 Aug 2021 15:23:34 +0900 Masahiro Yamada <masahiroy@kernel.org> wrote: > > > > > > > > > > On Fri, Aug 20, 2021 at 11:33 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote: > > > > > > > > > > > After merging the drm tree, today's linux-next build (x86_64 allmodconfig) > > > > > > failed like this: > > > > > > > > > > > > In file included from drivers/gpu/drm/i915/i915_debugfs.c:39: > > > > > > drivers/gpu/drm/i915/gt/intel_gt_requests.h:9:10: fatal error: stddef.h: No such file or directory > > > > > > 9 | #include <stddef.h> > > > > > > | ^~~~~~~~~~ > > > > > > > > > > > > Caused by commit > > > > > > > > > > > > 564f963eabd1 ("isystem: delete global -isystem compile option") > > > > > > > > > > > > from the kbuild tree interacting with commit > > > > > > > > > > > > b97060a99b01 ("drm/i915/guc: Update intel_gt_wait_for_idle to work with GuC") > > > > > > > > > > > > I have applied the following patch for today. > > > > > > > > > > > > > > > Thanks. > > > > > > > > > > This fix-up does not depend on my kbuild tree in any way. > > > > > > > > > > So, the drm maintainer can apply it to his tree. > > > > > > > > > > Perhaps with > > > > > > > > > > Fixes: b97060a99b01 ("drm/i915/guc: Update intel_gt_wait_for_idle to > > > > > work with GuC") > > > > > > > > OK, so that didn't happen so I will now apply the merge fix up to the > > > > merge of the kbuild tree. > > > > > > > > > > From: Stephen Rothwell <sfr@canb.auug.org.au> > > > > > > Date: Fri, 20 Aug 2021 12:24:19 +1000 > > > > > > Subject: [PATCH] drm/i915: use linux/stddef.h due to "isystem: trim/fixup stdarg.h and other headers" > > > > > > > > > > > > Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au> > > > > > > --- > > > > > > drivers/gpu/drm/i915/gt/intel_gt_requests.h | 2 +- > > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > > > diff --git a/drivers/gpu/drm/i915/gt/intel_gt_requests.h b/drivers/gpu/drm/i915/gt/intel_gt_requests.h > > > > > > index 51dbe0e3294e..d2969f68dd64 100644 > > > > > > --- a/drivers/gpu/drm/i915/gt/intel_gt_requests.h > > > > > > +++ b/drivers/gpu/drm/i915/gt/intel_gt_requests.h > > > > > > @@ -6,7 +6,7 @@ > > > > > > #ifndef INTEL_GT_REQUESTS_H > > > > > > #define INTEL_GT_REQUESTS_H > > > > > > > > > > > > -#include <stddef.h> > > > > > > +#include <linux/stddef.h> > > > > > > > > > > > > struct intel_engine_cs; > > > > > > struct intel_gt; > > > > > > -- > > > > > > 2.32.0 > > > > > > Ping? I am still applying this ... > > > > Apologies, this fell through a lot of cracks. I applied this to drm-next now. > > > > Rather, I was planning to apply this fix to my kbuild tree. > > Since you guys did not fix the issue in time, > I ended up with dropping [1] from my pull request. > > I want to get [1] merged in this MW. > > If I postponed it, somebody would add new > <stddef.h> or <stdint.h> inclusion in the next development > cycle, I will never make it in the mainline. > > [1] https://lore.kernel.org/linux-kernel/YQhY40teUJcTc5H4@localhost.localdomain/ Yeah no problem if you apply it too. For that: Acked-by: Daniel Vetter <daniel.vetter@ffwll.ch> I just figured I make sure this is at least not lost. -Daniel > > > > > > > Matt/John, as author/committer it's your job to make sure issues and > > fixes for the stuff you're pushing don't get lost. I'd have expected > > John to apply this to at least drm-intel-gt-next (it's not even > > there). > > > > Joonas, I think this is the 2nd or 3rd or so issue this release cycle > > where some compile fix got stuck a bit because drm-intel-gt-next isn't > > in linux-next. Can we please fix that? It probably needs some changes > > to the dim script. > > > > Cheers, Daniel > > -- > > Daniel Vetter > > Software Engineer, Intel Corporation > > http://blog.ffwll.ch > > > > -- > Best Regards > Masahiro Yamada -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch
next prev parent reply other threads:[~2021-09-08 9:07 UTC|newest] Thread overview: 192+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-08-20 2:33 linux-next: build failure after merge of the drm tree Stephen Rothwell 2021-08-20 6:23 ` Masahiro Yamada 2021-09-01 21:50 ` Stephen Rothwell 2021-09-05 22:49 ` Stephen Rothwell 2021-09-06 7:34 ` Daniel Vetter 2021-09-06 7:34 ` [Intel-gfx] " Daniel Vetter 2021-09-06 7:34 ` Daniel Vetter 2021-09-08 3:13 ` Masahiro Yamada 2021-09-08 3:13 ` [Intel-gfx] " Masahiro Yamada 2021-09-08 9:07 ` Daniel Vetter [this message] 2021-09-08 9:07 ` Daniel Vetter 2021-09-08 9:07 ` Daniel Vetter -- strict thread matches above, loose matches on Subject: below -- 2023-04-11 6:47 Stephen Rothwell 2023-04-11 6:47 ` Stephen Rothwell 2023-01-19 2:12 Stephen Rothwell 2023-01-19 2:12 ` Stephen Rothwell 2022-09-30 10:54 broonie 2022-10-04 2:20 ` Stephen Rothwell 2022-10-04 2:20 ` Stephen Rothwell 2022-10-04 2:24 ` David Airlie 2022-10-04 2:24 ` David Airlie 2022-10-04 3:05 ` Stephen Rothwell 2022-10-04 3:05 ` Stephen Rothwell 2022-10-04 11:39 ` Mark Brown 2022-10-04 11:39 ` Mark Brown 2022-10-05 15:30 ` Alex Deucher 2022-10-05 15:30 ` Alex Deucher 2022-10-05 16:45 ` Hamza Mahfooz 2022-10-05 16:45 ` Hamza Mahfooz 2022-10-05 22:28 ` Stephen Rothwell 2022-10-05 22:28 ` Stephen Rothwell 2022-10-06 8:12 ` Stephen Rothwell 2022-10-06 8:12 ` Stephen Rothwell 2022-10-06 13:56 ` Alex Deucher 2022-10-06 13:56 ` Alex Deucher 2022-10-06 16:52 ` Stephen Rothwell 2022-10-06 16:52 ` Stephen Rothwell 2022-10-06 16:25 ` Hamza Mahfooz 2022-10-06 16:25 ` Hamza Mahfooz 2022-02-25 16:42 broonie 2022-02-25 16:42 ` broonie 2022-02-28 3:46 ` Stephen Rothwell 2022-02-28 3:46 ` Stephen Rothwell 2022-02-28 6:44 ` Hsin-Yi Wang 2022-02-28 6:44 ` Hsin-Yi Wang 2022-02-28 6:48 ` Dave Airlie 2022-02-28 6:48 ` Dave Airlie 2022-02-28 15:33 ` Robert Foss 2022-02-28 15:33 ` Robert Foss 2022-03-01 6:58 ` Dave Airlie 2021-04-09 3:36 Stephen Rothwell 2021-04-09 3:36 ` Stephen Rothwell 2021-01-08 0:55 Stephen Rothwell 2021-01-08 0:55 ` Stephen Rothwell 2021-01-08 1:25 ` Stephen Rothwell 2021-01-08 1:25 ` Stephen Rothwell 2021-01-10 23:56 ` Stephen Rothwell 2021-01-10 23:56 ` Stephen Rothwell 2021-01-18 0:59 ` Stephen Rothwell 2021-01-18 0:59 ` Stephen Rothwell 2021-01-18 1:06 ` Dave Airlie 2021-01-18 1:06 ` Dave Airlie 2021-01-20 12:12 ` Daniel Vetter 2021-01-20 12:12 ` Daniel Vetter 2021-01-20 20:44 ` Stephen Rothwell 2021-01-20 20:44 ` Stephen Rothwell 2020-12-09 1:53 Stephen Rothwell 2020-12-09 1:53 ` Stephen Rothwell 2020-09-28 3:54 Stephen Rothwell 2020-09-28 3:54 ` Stephen Rothwell 2020-09-28 6:04 ` Christoph Hellwig 2020-09-28 6:08 ` Dave Airlie 2020-09-28 6:08 ` Dave Airlie 2020-09-28 6:14 ` Christoph Hellwig 2020-09-28 10:15 ` Paul Cercueil 2020-09-28 10:15 ` Paul Cercueil 2020-09-28 11:34 ` Christoph Hellwig 2020-09-28 11:46 ` Paul Cercueil 2020-09-28 11:46 ` Paul Cercueil 2020-09-28 12:10 ` Christoph Hellwig 2020-09-28 13:31 ` Paul Cercueil 2020-09-28 13:31 ` Paul Cercueil 2020-09-30 9:02 ` Christoph Hellwig 2020-09-30 13:33 ` Paul Cercueil 2020-09-30 13:33 ` Paul Cercueil 2020-09-30 16:11 ` Christoph Hellwig 2020-09-30 16:39 ` Paul Cercueil 2020-09-30 16:39 ` Paul Cercueil 2020-09-30 16:40 ` Christoph Hellwig 2020-09-30 16:45 ` Paul Cercueil 2020-09-30 16:45 ` Paul Cercueil 2020-09-30 16:52 ` Christoph Hellwig 2020-05-08 4:29 Stephen Rothwell 2020-05-08 4:29 ` Stephen Rothwell 2019-11-05 2:54 Stephen Rothwell 2019-11-05 2:54 ` Stephen Rothwell 2019-10-10 0:22 Stephen Rothwell 2019-10-10 0:22 ` Stephen Rothwell 2019-07-08 4:48 Stephen Rothwell 2019-07-08 4:48 ` Stephen Rothwell 2019-04-26 2:17 Stephen Rothwell 2019-04-26 2:17 ` Stephen Rothwell 2018-05-16 6:53 Stephen Rothwell 2018-05-16 6:53 ` Stephen Rothwell 2018-05-16 8:09 ` Oded Gabbay 2018-05-16 8:09 ` Oded Gabbay 2018-05-17 5:00 ` Dave Airlie 2018-05-17 5:00 ` Dave Airlie 2018-03-13 5:46 Stephen Rothwell 2018-01-02 1:48 Stephen Rothwell 2017-09-28 4:57 Stephen Rothwell 2017-03-20 0:51 Stephen Rothwell 2017-03-20 0:51 ` Stephen Rothwell 2017-03-20 8:03 ` Daniel Vetter 2017-03-20 8:03 ` Daniel Vetter 2017-03-20 8:05 ` Daniel Vetter 2017-03-20 8:05 ` Daniel Vetter 2017-02-23 23:25 Stephen Rothwell 2017-01-24 1:25 Stephen Rothwell 2017-01-25 10:52 ` Vincent ABRIOU 2016-07-15 3:40 Stephen Rothwell 2016-07-15 3:40 ` Stephen Rothwell 2016-07-15 4:32 ` Sedat Dilek 2016-04-28 1:57 Stephen Rothwell 2016-04-28 1:57 ` Stephen Rothwell 2016-04-28 8:11 ` Jani Nikula 2016-04-28 8:11 ` Jani Nikula 2016-03-17 4:41 Stephen Rothwell 2016-03-17 8:41 ` Christian König 2016-03-17 8:41 ` Christian König 2016-02-22 0:45 Stephen Rothwell 2015-12-31 10:31 Stephen Rothwell 2016-01-07 2:44 ` Stephen Rothwell 2016-01-11 22:54 ` Stephen Rothwell 2016-01-12 0:31 ` Dave Airlie 2016-01-12 4:35 ` Stephen Rothwell 2016-01-12 7:48 ` Zhu, Rex 2015-12-31 2:40 Stephen Rothwell 2015-12-31 5:14 ` Mark yao 2015-11-04 9:22 Stephen Rothwell 2015-11-09 16:45 ` Guenter Roeck 2015-11-10 4:55 ` Alexandre Courbot 2015-10-06 1:09 Stephen Rothwell 2015-06-09 13:42 Stephen Rothwell 2015-06-09 14:02 ` Deucher, Alexander 2015-06-10 0:28 ` Stephen Rothwell 2015-05-20 3:08 Stephen Rothwell 2015-05-20 4:41 ` Dave Airlie 2015-05-20 5:25 ` Stephen Rothwell 2015-05-20 5:31 ` Stephen Rothwell 2015-05-20 6:01 ` Oded Gabbay 2015-05-20 6:08 ` Dave Airlie 2015-01-29 2:38 Stephen Rothwell 2015-01-29 8:21 ` Oded Gabbay 2015-01-29 8:21 ` Oded Gabbay 2015-01-29 9:35 ` Stephen Rothwell 2015-01-29 9:35 ` Stephen Rothwell 2014-09-08 4:26 Stephen Rothwell 2014-06-05 4:12 Stephen Rothwell 2014-06-05 4:28 ` Stephen Rothwell 2012-09-24 3:18 Stephen Rothwell 2012-09-24 11:31 ` Daniel Vetter 2012-09-24 11:58 ` Stephen Rothwell 2012-08-24 3:49 Stephen Rothwell 2012-05-17 3:47 Stephen Rothwell 2012-04-30 3:22 Stephen Rothwell 2012-04-23 3:42 Stephen Rothwell 2012-03-16 2:53 Stephen Rothwell 2012-03-19 2:45 ` Stephen Rothwell 2011-11-28 2:49 Stephen Rothwell 2011-11-16 0:30 Stephen Rothwell 2011-11-16 10:16 ` Alan Cox 2011-11-16 16:42 ` Jesse Barnes 2011-11-25 1:49 ` Stephen Rothwell 2011-11-25 10:30 ` Alan Cox 2011-11-25 10:34 ` Dave Airlie 2011-11-28 2:53 ` Stephen Rothwell 2011-11-28 11:30 ` Alan Cox 2011-11-28 13:13 ` Stephen Rothwell 2011-11-28 13:18 ` Alan Cox 2011-11-28 13:53 ` Stephen Rothwell 2011-11-28 14:05 ` Dave Airlie 2011-11-28 14:08 ` Alan Cox 2011-11-28 2:52 ` Stephen Rothwell 2011-11-28 5:07 ` Greg KH 2011-11-28 11:33 ` Alan Cox 2011-03-15 3:49 Stephen Rothwell 2010-04-08 3:12 Stephen Rothwell 2010-04-08 3:52 ` Dave Airlie 2010-04-08 4:14 ` Stephen Rothwell 2010-04-08 16:19 ` Tony Luck 2010-04-08 20:50 ` Pauli Nieminen
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=CAKMK7uEjGA01bhPYVJt5aHfvh1i6roV0w-P5Bq9-tohS65esdA@mail.gmail.com \ --to=daniel@ffwll.ch \ --cc=John.C.Harrison@intel.com \ --cc=adobriyan@gmail.com \ --cc=airlied@linux.ie \ --cc=dri-devel@lists.freedesktop.org \ --cc=intel-gfx@lists.freedesktop.org \ --cc=jani.nikula@linux.intel.com \ --cc=joonas.lahtinen@linux.intel.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-next@vger.kernel.org \ --cc=masahiroy@kernel.org \ --cc=matthew.brost@intel.com \ --cc=rodrigo.vivi@intel.com \ --cc=sfr@canb.auug.org.au \ /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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.