Re: [Intel-gfx] [PATCH v1 3/4] drm/i915:bxt: Enable Pooled EU support
On Fri, Jul 17, 2015 at 05:54:20PM +0100, Siluvery, Arun wrote: > On 17/07/2015 17:27, Chris Wilson wrote: > >On Fri, Jul 17, 2015 at 05:08:53PM +0100, Arun Siluvery wrote: > >>This mode allows to assign EUs to pools. > >>The command to enable this mode is sent in auxiliary golden context batch > >>as this is only issued once with each context initialization. Thanks to > >>Mika for the preliminary review. > > > >A quick explanation for why this has to be in the kernel would be nice. > >Privileged instruction? > > This purpose of auxiliary batch is explained in patch2, but I can > add some explanation about this one also. Here, I am looking for an explanation of why these commands in particular are desired. Mika's short explanation that must be the same for all contexts on the system is sufficient. -Chris -- Chris Wilson, Intel Open Source Technology Centre ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH v1 3/4] drm/i915:bxt: Enable Pooled EU support
On 17/07/2015 17:27, Chris Wilson wrote: On Fri, Jul 17, 2015 at 05:08:53PM +0100, Arun Siluvery wrote: This mode allows to assign EUs to pools. The command to enable this mode is sent in auxiliary golden context batch as this is only issued once with each context initialization. Thanks to Mika for the preliminary review. A quick explanation for why this has to be in the kernel would be nice. Privileged instruction? This purpose of auxiliary batch is explained in patch2, but I can add some explanation about this one also. Not fond of the split between this and patch 4. Patch 4 intoduces one feature flag that looks different to the one we use here to enable support. I will patch4 as separate as it deals with libdrm changes but use the feature flag in this one. regards Arun -Chris ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH v1 3/4] drm/i915:bxt: Enable Pooled EU support
Chris Wilson writes: > On Fri, Jul 17, 2015 at 05:08:53PM +0100, Arun Siluvery wrote: >> This mode allows to assign EUs to pools. >> The command to enable this mode is sent in auxiliary golden context batch >> as this is only issued once with each context initialization. Thanks to >> Mika for the preliminary review. > > A quick explanation for why this has to be in the kernel would be nice. > Privileged instruction? > The pooled mode is global. Once set, it has to stay same across all contexts until subsequent fw reset. -Mika > Not fond of the split between this and patch 4. Patch 4 intoduces one > feature flag that looks different to the one we use here to enable > support. > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre > ___ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH v1 3/4] drm/i915:bxt: Enable Pooled EU support
On Fri, Jul 17, 2015 at 05:08:53PM +0100, Arun Siluvery wrote: > This mode allows to assign EUs to pools. > The command to enable this mode is sent in auxiliary golden context batch > as this is only issued once with each context initialization. Thanks to > Mika for the preliminary review. A quick explanation for why this has to be in the kernel would be nice. Privileged instruction? Not fond of the split between this and patch 4. Patch 4 intoduces one feature flag that looks different to the one we use here to enable support. -Chris -- Chris Wilson, Intel Open Source Technology Centre ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx