[Intel-gfx] [PATCH v3 2/2] drm/i915/guc : Decouple logs from submission

[Daniele Ceraolo Spurio]

Since we're not decoupling just logs, maybe the commit could be renamed 
to "Decouple ADS and logs from submission"

On 19/09/17 09:15, Sujaritha Sundaresan wrote:
> The Additional Data Struct (ADS) contains objects that are required by
> guc post FW load and are not necessarily submission-only (although that's
> our current only use-case). If in the future we load GuC with submission
> disabled to use some other GuC feature we might still end up requiring
> something inside the ADS, so it makes more sense for them to be always
> created if GuC is loaded.

To make a concrete example, the pages used by GuC to save state during 
suspend are allocated as part of the ADS.

> Similarly, we still want to access GuC logs even if GuC submission is
> disable to debug issues with GuC loading or with wathever we're using GuC for.
> 
> v2: Decoupling ads together with logs
> v3: Group initialization of GuC objects
> 
> Cc: Michal Wajdeczko <michal.wajdeczko at intel.com>
> Cc: Oscar Mateo <oscar.mateo at intel.com>
> Cc: Anusha Srivatsa <anusha.srivatsa at intel.com>
> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>
> Signed-off-by: Sujaritha Sundaresan <sujaritha.sundaresan at intel.com>

Acked-by: Daniele Ceraolo Spurio <daniele.ceraolospurio at intel.com>