* [PATCH] drm/i915/guc: Drop comment on fwif autogeneration
@ 2016-12-05 16:04 Arkadiusz Hiler
2016-12-05 18:59 ` ✗ Fi.CI.BAT: warning for " Patchwork
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Arkadiusz Hiler @ 2016-12-05 16:04 UTC (permalink / raw)
To: intel-gfx
The firmware interface file was initially partially autogenerated, but
this is no longer the case.
It was never updated automatically, and a lot manual changes were
introduced since.
From now on any changes to the firmware interface will be managed by
hand, which gives us flexibility when it comes to structure reuse
(HuC/GuC) and naming conventions.
Cc: Anusha Srivatsa <anusha.srivatsa@intel.com>
Cc: Jeff Mcgee <jeff.mcgee@intel.com>
Cc: Sagar A. Kamble <sagar.a.kamble@intel.com>
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
---
drivers/gpu/drm/i915/intel_guc_fwif.h | 9 ---------
1 file changed, 9 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_guc_fwif.h b/drivers/gpu/drm/i915/intel_guc_fwif.h
index 00ca0df..3202b32 100644
--- a/drivers/gpu/drm/i915/intel_guc_fwif.h
+++ b/drivers/gpu/drm/i915/intel_guc_fwif.h
@@ -23,15 +23,6 @@
#ifndef _INTEL_GUC_FWIF_H
#define _INTEL_GUC_FWIF_H
-/*
- * This file is partially autogenerated, although currently with some manual
- * fixups afterwards. In future, it should be entirely autogenerated, in order
- * to ensure that the definitions herein remain in sync with those used by the
- * GuC's own firmware.
- *
- * EDITING THIS FILE IS THEREFORE NOT RECOMMENDED - YOUR CHANGES MAY BE LOST.
- */
-
#define GFXCORE_FAMILY_GEN9 12
#define GFXCORE_FAMILY_UNKNOWN 0x7fffffff
--
2.7.4
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 8+ messages in thread* ✗ Fi.CI.BAT: warning for drm/i915/guc: Drop comment on fwif autogeneration 2016-12-05 16:04 [PATCH] drm/i915/guc: Drop comment on fwif autogeneration Arkadiusz Hiler @ 2016-12-05 18:59 ` Patchwork 2016-12-06 5:41 ` Arkadiusz Hiler 2016-12-05 21:37 ` [PATCH] " Jeff McGee 2016-12-05 23:59 ` Srivatsa, Anusha 2 siblings, 1 reply; 8+ messages in thread From: Patchwork @ 2016-12-05 18:59 UTC (permalink / raw) To: Arkadiusz Hiler; +Cc: intel-gfx == Series Details == Series: drm/i915/guc: Drop comment on fwif autogeneration URL : https://patchwork.freedesktop.org/series/16373/ State : warning == Summary == Series 16373v1 drm/i915/guc: Drop comment on fwif autogeneration https://patchwork.freedesktop.org/api/1.0/series/16373/revisions/1/mbox/ Test gem_busy: Subgroup basic-hang-default: fail -> PASS (fi-hsw-4770r) Test kms_pipe_crc_basic: Subgroup suspend-read-crc-pipe-a: pass -> DMESG-WARN (fi-skl-6770hq) fi-bdw-5557u total:247 pass:232 dwarn:0 dfail:0 fail:0 skip:15 fi-bsw-n3050 total:247 pass:207 dwarn:0 dfail:0 fail:0 skip:40 fi-byt-j1900 total:247 pass:219 dwarn:0 dfail:0 fail:0 skip:28 fi-byt-n2820 total:247 pass:215 dwarn:0 dfail:0 fail:0 skip:32 fi-hsw-4770 total:247 pass:227 dwarn:0 dfail:0 fail:0 skip:20 fi-hsw-4770r total:247 pass:227 dwarn:0 dfail:0 fail:0 skip:20 fi-ilk-650 total:247 pass:194 dwarn:0 dfail:0 fail:0 skip:53 fi-ivb-3520m total:247 pass:225 dwarn:0 dfail:0 fail:0 skip:22 fi-ivb-3770 total:247 pass:225 dwarn:0 dfail:0 fail:0 skip:22 fi-kbl-7500u total:247 pass:225 dwarn:0 dfail:0 fail:0 skip:22 fi-skl-6260u total:247 pass:233 dwarn:0 dfail:0 fail:0 skip:14 fi-skl-6700hq total:247 pass:226 dwarn:0 dfail:0 fail:0 skip:21 fi-skl-6770hq total:247 pass:232 dwarn:1 dfail:0 fail:0 skip:14 fi-snb-2520m total:247 pass:215 dwarn:0 dfail:0 fail:0 skip:32 fi-snb-2600 total:247 pass:214 dwarn:0 dfail:0 fail:0 skip:33 a947529bb652fdcf50e8c0e8ee5102b737bae23e drm-tip: 2016y-12m-05d-14h-27m-06s UTC integration manifest abb5dbd drm/i915/guc: Drop comment on fwif autogeneration == Logs == For more details see: https://intel-gfx-ci.01.org/CI/Patchwork_3196/ _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: ✗ Fi.CI.BAT: warning for drm/i915/guc: Drop comment on fwif autogeneration 2016-12-05 18:59 ` ✗ Fi.CI.BAT: warning for " Patchwork @ 2016-12-06 5:41 ` Arkadiusz Hiler 2016-12-07 7:14 ` Tvrtko Ursulin 0 siblings, 1 reply; 8+ messages in thread From: Arkadiusz Hiler @ 2016-12-06 5:41 UTC (permalink / raw) To: intel-gfx On Mon, Dec 05, 2016 at 06:59:01PM +0000, Patchwork wrote: > == Series Details == > > Series: drm/i915/guc: Drop comment on fwif autogeneration > URL : https://patchwork.freedesktop.org/series/16373/ > State : warning > > == Summary == > > Series 16373v1 drm/i915/guc: Drop comment on fwif autogeneration > https://patchwork.freedesktop.org/api/1.0/series/16373/revisions/1/mbox/ > > Test gem_busy: > Subgroup basic-hang-default: > fail -> PASS (fi-hsw-4770r) The cange is only a comment removal. Could not affect the pass. > Test kms_pipe_crc_basic: > Subgroup suspend-read-crc-pipe-a: > pass -> DMESG-WARN (fi-skl-6770hq) > [ 408.780897] [drm:skl_set_cdclk [i915]] *ERROR* failed to inform PCU about cdclk change https://bugs.freedesktop.org/show_bug.cgi?id=97929 > > a947529bb652fdcf50e8c0e8ee5102b737bae23e drm-tip: 2016y-12m-05d-14h-27m-06s UTC integration manifest > abb5dbd drm/i915/guc: Drop comment on fwif autogeneration > > == Logs == > > For more details see: https://intel-gfx-ci.01.org/CI/Patchwork_3196/ -- Cheers, Arek _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: ✗ Fi.CI.BAT: warning for drm/i915/guc: Drop comment on fwif autogeneration 2016-12-06 5:41 ` Arkadiusz Hiler @ 2016-12-07 7:14 ` Tvrtko Ursulin 0 siblings, 0 replies; 8+ messages in thread From: Tvrtko Ursulin @ 2016-12-07 7:14 UTC (permalink / raw) To: Arkadiusz Hiler, intel-gfx On 06/12/2016 05:41, Arkadiusz Hiler wrote: > On Mon, Dec 05, 2016 at 06:59:01PM +0000, Patchwork wrote: >> == Series Details == >> >> Series: drm/i915/guc: Drop comment on fwif autogeneration >> URL : https://patchwork.freedesktop.org/series/16373/ >> State : warning >> >> == Summary == >> >> Series 16373v1 drm/i915/guc: Drop comment on fwif autogeneration >> https://patchwork.freedesktop.org/api/1.0/series/16373/revisions/1/mbox/ >> >> Test gem_busy: >> Subgroup basic-hang-default: >> fail -> PASS (fi-hsw-4770r) > > The cange is only a comment removal. Could not affect the pass. > >> Test kms_pipe_crc_basic: >> Subgroup suspend-read-crc-pipe-a: >> pass -> DMESG-WARN (fi-skl-6770hq) >> > > [ 408.780897] [drm:skl_set_cdclk [i915]] *ERROR* failed to inform PCU about cdclk change > > https://bugs.freedesktop.org/show_bug.cgi?id=97929 > >> >> a947529bb652fdcf50e8c0e8ee5102b737bae23e drm-tip: 2016y-12m-05d-14h-27m-06s UTC integration manifest >> abb5dbd drm/i915/guc: Drop comment on fwif autogeneration Pushed to dinq, thanks for the patch and review! Regards, Tvrtko _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915/guc: Drop comment on fwif autogeneration 2016-12-05 16:04 [PATCH] drm/i915/guc: Drop comment on fwif autogeneration Arkadiusz Hiler 2016-12-05 18:59 ` ✗ Fi.CI.BAT: warning for " Patchwork @ 2016-12-05 21:37 ` Jeff McGee 2016-12-05 23:59 ` Srivatsa, Anusha 2 siblings, 0 replies; 8+ messages in thread From: Jeff McGee @ 2016-12-05 21:37 UTC (permalink / raw) To: Arkadiusz Hiler; +Cc: intel-gfx Reviewed-by: Jeff McGee <jeff.mcgee@intel.com> On Mon, Dec 05, 2016 at 05:04:29PM +0100, Arkadiusz Hiler wrote: > The firmware interface file was initially partially autogenerated, but > this is no longer the case. > > It was never updated automatically, and a lot manual changes were > introduced since. > > From now on any changes to the firmware interface will be managed by > hand, which gives us flexibility when it comes to structure reuse > (HuC/GuC) and naming conventions. > > Cc: Anusha Srivatsa <anusha.srivatsa@intel.com> > Cc: Jeff Mcgee <jeff.mcgee@intel.com> > Cc: Sagar A. Kamble <sagar.a.kamble@intel.com> > Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> > --- > drivers/gpu/drm/i915/intel_guc_fwif.h | 9 --------- > 1 file changed, 9 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_guc_fwif.h b/drivers/gpu/drm/i915/intel_guc_fwif.h > index 00ca0df..3202b32 100644 > --- a/drivers/gpu/drm/i915/intel_guc_fwif.h > +++ b/drivers/gpu/drm/i915/intel_guc_fwif.h > @@ -23,15 +23,6 @@ > #ifndef _INTEL_GUC_FWIF_H > #define _INTEL_GUC_FWIF_H > > -/* > - * This file is partially autogenerated, although currently with some manual > - * fixups afterwards. In future, it should be entirely autogenerated, in order > - * to ensure that the definitions herein remain in sync with those used by the > - * GuC's own firmware. > - * > - * EDITING THIS FILE IS THEREFORE NOT RECOMMENDED - YOUR CHANGES MAY BE LOST. > - */ > - > #define GFXCORE_FAMILY_GEN9 12 > #define GFXCORE_FAMILY_UNKNOWN 0x7fffffff > > -- > 2.7.4 > _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915/guc: Drop comment on fwif autogeneration 2016-12-05 16:04 [PATCH] drm/i915/guc: Drop comment on fwif autogeneration Arkadiusz Hiler 2016-12-05 18:59 ` ✗ Fi.CI.BAT: warning for " Patchwork 2016-12-05 21:37 ` [PATCH] " Jeff McGee @ 2016-12-05 23:59 ` Srivatsa, Anusha 2016-12-06 5:37 ` Arkadiusz Hiler 2 siblings, 1 reply; 8+ messages in thread From: Srivatsa, Anusha @ 2016-12-05 23:59 UTC (permalink / raw) To: Hiler, Arkadiusz, intel-gfx@lists.freedesktop.org >-----Original Message----- >From: Hiler, Arkadiusz >Sent: Monday, December 5, 2016 8:04 AM >To: intel-gfx@lists.freedesktop.org >Cc: Srivatsa, Anusha <anusha.srivatsa@intel.com>; Mcgee, Jeff ><jeff.mcgee@intel.com>; Kamble, Sagar A <sagar.a.kamble@intel.com> >Subject: [PATCH] drm/i915/guc: Drop comment on fwif autogeneration > >The firmware interface file was initially partially autogenerated, but this is no >longer the case. > >It was never updated automatically, and a lot manual changes were introduced >since. > >From now on any changes to the firmware interface will be managed by hand, >which gives us flexibility when it comes to structure reuse >(HuC/GuC) and naming conventions. > >Cc: Anusha Srivatsa <anusha.srivatsa@intel.com> >Cc: Jeff Mcgee <jeff.mcgee@intel.com> >Cc: Sagar A. Kamble <sagar.a.kamble@intel.com> >Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> >--- > drivers/gpu/drm/i915/intel_guc_fwif.h | 9 --------- > 1 file changed, 9 deletions(-) > >diff --git a/drivers/gpu/drm/i915/intel_guc_fwif.h >b/drivers/gpu/drm/i915/intel_guc_fwif.h >index 00ca0df..3202b32 100644 >--- a/drivers/gpu/drm/i915/intel_guc_fwif.h >+++ b/drivers/gpu/drm/i915/intel_guc_fwif.h >@@ -23,15 +23,6 @@ > #ifndef _INTEL_GUC_FWIF_H > #define _INTEL_GUC_FWIF_H > >-/* >- * This file is partially autogenerated, although currently with some manual >- * fixups afterwards. In future, it should be entirely autogenerated, in order >- * to ensure that the definitions herein remain in sync with those used by the >- * GuC's own firmware. >- * >- * EDITING THIS FILE IS THEREFORE NOT RECOMMENDED - YOUR CHANGES >MAY BE LOST. >- */ With this removal of comment, do you feel moving the contents of intel_guc_fwif.h to intel_uc.c or renaming the file to intel_uc_fwif.h makes a lot of difference? Cheers, Anusha > #define GFXCORE_FAMILY_GEN9 12 > #define GFXCORE_FAMILY_UNKNOWN 0x7fffffff > >-- >2.7.4 _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915/guc: Drop comment on fwif autogeneration 2016-12-05 23:59 ` Srivatsa, Anusha @ 2016-12-06 5:37 ` Arkadiusz Hiler 2016-12-06 17:31 ` Srivatsa, Anusha 0 siblings, 1 reply; 8+ messages in thread From: Arkadiusz Hiler @ 2016-12-06 5:37 UTC (permalink / raw) To: Srivatsa, Anusha; +Cc: intel-gfx@lists.freedesktop.org On Tue, Dec 06, 2016 at 12:59:03AM +0100, Srivatsa, Anusha wrote: > >-----Original Message----- > >From: Hiler, Arkadiusz > >Sent: Monday, December 5, 2016 8:04 AM > >To: intel-gfx@lists.freedesktop.org > >Cc: Srivatsa, Anusha <anusha.srivatsa@intel.com>; Mcgee, Jeff > ><jeff.mcgee@intel.com>; Kamble, Sagar A <sagar.a.kamble@intel.com> > >Subject: [PATCH] drm/i915/guc: Drop comment on fwif autogeneration > > > >The firmware interface file was initially partially autogenerated, but this is no > >longer the case. > > > >It was never updated automatically, and a lot manual changes were introduced > >since. > > > >From now on any changes to the firmware interface will be managed by hand, > >which gives us flexibility when it comes to structure reuse > >(HuC/GuC) and naming conventions. > > > >Cc: Anusha Srivatsa <anusha.srivatsa@intel.com> > >Cc: Jeff Mcgee <jeff.mcgee@intel.com> > >Cc: Sagar A. Kamble <sagar.a.kamble@intel.com> > >Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> > >--- > > drivers/gpu/drm/i915/intel_guc_fwif.h | 9 --------- > > 1 file changed, 9 deletions(-) > > > >diff --git a/drivers/gpu/drm/i915/intel_guc_fwif.h > >b/drivers/gpu/drm/i915/intel_guc_fwif.h > >index 00ca0df..3202b32 100644 > >--- a/drivers/gpu/drm/i915/intel_guc_fwif.h > >+++ b/drivers/gpu/drm/i915/intel_guc_fwif.h > >@@ -23,15 +23,6 @@ > > #ifndef _INTEL_GUC_FWIF_H > > #define _INTEL_GUC_FWIF_H > > > >-/* > >- * This file is partially autogenerated, although currently with some manual > >- * fixups afterwards. In future, it should be entirely autogenerated, in order > >- * to ensure that the definitions herein remain in sync with those used by the > >- * GuC's own firmware. > >- * > >- * EDITING THIS FILE IS THEREFORE NOT RECOMMENDED - YOUR CHANGES > >MAY BE LOST. > >- */ > > With this removal of comment, do you feel moving the contents of intel_guc_fwif.h to intel_uc.c or renaming the file to intel_uc_fwif.h makes a lot of difference? I think this area could use some reorganization, but we should give it more thought before we do anything. Keeping all the GuC-related defines/structs that are used to communicate with FW and are prone to change in a separate file seems like a good idea. This would make them easier to manage in cases of any turbulence. I agree that `intel_uc_fwif.h` would be a good name, as your HuC patchset is reusing some of the intrfaces (e.g. css headers) for HuC. The rename can be done as part of the HuC effort. I'll give it some more thought today and send you a RFC version of the reorg to include in your patchset. -- Cheers, Arek _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] drm/i915/guc: Drop comment on fwif autogeneration 2016-12-06 5:37 ` Arkadiusz Hiler @ 2016-12-06 17:31 ` Srivatsa, Anusha 0 siblings, 0 replies; 8+ messages in thread From: Srivatsa, Anusha @ 2016-12-06 17:31 UTC (permalink / raw) To: Hiler, Arkadiusz; +Cc: intel-gfx@lists.freedesktop.org >-----Original Message----- >From: Hiler, Arkadiusz >Sent: Monday, December 5, 2016 9:37 PM >To: Srivatsa, Anusha <anusha.srivatsa@intel.com> >Cc: intel-gfx@lists.freedesktop.org; Mcgee, Jeff <jeff.mcgee@intel.com>; >Kamble, Sagar A <sagar.a.kamble@intel.com> >Subject: Re: [PATCH] drm/i915/guc: Drop comment on fwif autogeneration > >On Tue, Dec 06, 2016 at 12:59:03AM +0100, Srivatsa, Anusha wrote: >> >-----Original Message----- >> >From: Hiler, Arkadiusz >> >Sent: Monday, December 5, 2016 8:04 AM >> >To: intel-gfx@lists.freedesktop.org >> >Cc: Srivatsa, Anusha <anusha.srivatsa@intel.com>; Mcgee, Jeff >> ><jeff.mcgee@intel.com>; Kamble, Sagar A <sagar.a.kamble@intel.com> >> >Subject: [PATCH] drm/i915/guc: Drop comment on fwif autogeneration >> > >> >The firmware interface file was initially partially autogenerated, >> >but this is no longer the case. >> > >> >It was never updated automatically, and a lot manual changes were >> >introduced since. >> > >> >From now on any changes to the firmware interface will be managed by >> >hand, which gives us flexibility when it comes to structure reuse >> >(HuC/GuC) and naming conventions. >> > >> >Cc: Anusha Srivatsa <anusha.srivatsa@intel.com> >> >Cc: Jeff Mcgee <jeff.mcgee@intel.com> >> >Cc: Sagar A. Kamble <sagar.a.kamble@intel.com> >> >Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> >> >--- >> > drivers/gpu/drm/i915/intel_guc_fwif.h | 9 --------- >> > 1 file changed, 9 deletions(-) >> > >> >diff --git a/drivers/gpu/drm/i915/intel_guc_fwif.h >> >b/drivers/gpu/drm/i915/intel_guc_fwif.h >> >index 00ca0df..3202b32 100644 >> >--- a/drivers/gpu/drm/i915/intel_guc_fwif.h >> >+++ b/drivers/gpu/drm/i915/intel_guc_fwif.h >> >@@ -23,15 +23,6 @@ >> > #ifndef _INTEL_GUC_FWIF_H >> > #define _INTEL_GUC_FWIF_H >> > >> >-/* >> >- * This file is partially autogenerated, although currently with >> >some manual >> >- * fixups afterwards. In future, it should be entirely >> >autogenerated, in order >> >- * to ensure that the definitions herein remain in sync with those >> >used by the >> >- * GuC's own firmware. >> >- * >> >- * EDITING THIS FILE IS THEREFORE NOT RECOMMENDED - YOUR CHANGES >MAY >> >BE LOST. >> >- */ >> >> With this removal of comment, do you feel moving the contents of >intel_guc_fwif.h to intel_uc.c or renaming the file to intel_uc_fwif.h makes a lot >of difference? > >I think this area could use some reorganization, but we should give it more >thought before we do anything. > >Keeping all the GuC-related defines/structs that are used to communicate with >FW and are prone to change in a separate file seems like a good idea. > >This would make them easier to manage in cases of any turbulence. > >I agree that `intel_uc_fwif.h` would be a good name, as your HuC patchset is >reusing some of the intrfaces (e.g. css headers) for HuC. > >The rename can be done as part of the HuC effort. I'll give it some more thought >today and send you a RFC version of the reorg to include in your patchset. > Sure. Thanks! Anusha >Cheers, >Arek _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-12-07 7:14 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-12-05 16:04 [PATCH] drm/i915/guc: Drop comment on fwif autogeneration Arkadiusz Hiler 2016-12-05 18:59 ` ✗ Fi.CI.BAT: warning for " Patchwork 2016-12-06 5:41 ` Arkadiusz Hiler 2016-12-07 7:14 ` Tvrtko Ursulin 2016-12-05 21:37 ` [PATCH] " Jeff McGee 2016-12-05 23:59 ` Srivatsa, Anusha 2016-12-06 5:37 ` Arkadiusz Hiler 2016-12-06 17:31 ` Srivatsa, Anusha
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox