Re: [PATCH v2 2/2] drm/i915/tgl: Add perf support on TGL

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Quoting Umesh Nerlige Ramappa (2019-10-21 23:52:49)
> From: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>
> 
> The design of the OA unit has been split into several units. We now
> have a global unit (OAG) and a render specific unit (OAR). This leads
> to some changes on how we program things. Some details :
> 
> OAR:
>   - has its own set of counter registers, they are per-context
>     saved/restored
>   - counters are not written to the circular OA buffer
>   - a snapshot of the counters can be acquired with
>     MI_RECORD_PERF_COUNT, or a single counter can be read with
>     MI_STORE_REGISTER_MEM.
> 
> OAG:
>   - has global counters that increment across context switches
>   - counters are written into the circular OA buffer (if requested)
> 
> v2: Fix checkpatch warnings on code style (Lucas)
> v3: (Umesh)
>   - Update register from which tail, status and head are read
>   - Update logic to sample context reports
>   - Update whitelist mux and b counter regs
> v4: Fix a bug when updating context image for new contexts (Umesh)
> v5: Squash patch enabling save/restore of counters into context image
> 
>     We want this so we can preempt performance queries and keep the
>     system responsive even when long running queries are ongoing. We
>     avoid doing it for all contexts.
> 
>     - use LRI to modify context control (Chris)
>     - use MASKED_FIELD to program just the masked bits (Chris)
>     - disable save/restore of counters on cleanup (Chris)
> 
> BSpec: 28727, 30021
> 
> Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@xxxxxxxxx>
> Signed-off-by: Umesh Nerlige Ramappa <umesh.nerlige.ramappa@xxxxxxxxx>
> Signed-off-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx>
> ---
> @@ -292,7 +293,8 @@ static u32 i915_perf_stream_paranoid = true;
>  #define INVALID_CTX_ID 0xffffffff
>  
>  /* On Gen8+ automatically triggered OA reports include a 'reason' field... */
> -#define OAREPORT_REASON_MASK           0x3f
> +#define OAREPORT_REASON_MASK           (IS_GEN(stream->perf->i915, 12) ? \
> +                                       0x7f : 0x3f)

Resist temptations to do implicit parameters.

I can't guarantee that all the registers are correct, but the request
emission etc for config setup seems fine.

Acked-by: Chris Wilson <chris.p.wilson@xxxxxxxxx>
-Chris
---------------------------------------------------------------------
Intel Corporation (UK) Limited
Registered No. 1134945 (England)
Registered Office: Pipers Way, Swindon SN3 1RJ
VAT No: 860 2173 47

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux