Re: [Intel-gfx] [i-g-t] dumb_buffer@clear_create triggers OOM since 0b0eaa353

2020-04-25 Thread Li Zhijian
On 4/23/20 5:38 PM, Li Zhijian wrote: Hi guys 0Day noticed that dumb_buffer@clear_create triggers OOM  since commit: 0b0eaa353 ("tests/dumb_buffer: Try to compute the largest possible dumb buffer") our platform is - CPU: Intel(R) Xeon(R) CPU D-1541 @ 2.10GHz - memory: 48G - kernel: v5.5, v

Re: [Intel-gfx] [PULL] gvt-next

2020-04-25 Thread Zhenyu Wang
On 2020.04.22 13:12:30 +0800, Zhenyu Wang wrote: > > Hi, > > Here's current gvt-next. This removes left non-upstream xen support bits > which will be kept out of tree instead. And several guest context shadow > optimizations from Yan. > > Thanks > -- Ping for merge.. > > The following changes

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists (rev2)

2020-04-25 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists (rev2) URL : https://patchwork.freedesktop.org/series/76486/ State : failure == Summary == CI Bug Log - changes from CI_DRM_8368_full -> Patchwork_17464_full ===

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists (rev2)

2020-04-25 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists (rev2) URL : https://patchwork.freedesktop.org/series/76486/ State : success == Summary == CI Bug Log - changes from CI_DRM_8368 -> Patchwork_17464 =

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists (rev2)

2020-04-25 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists (rev2) URL : https://patchwork.freedesktop.org/series/76486/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.6.0 Commit: drm/i915/gt: Always enable busy-sta

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists (rev2)

2020-04-25 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists (rev2) URL : https://patchwork.freedesktop.org/series/76486/ State : warning == Summary == $ dim checkpatch origin/drm-tip bd5217caba42 drm/i915/gt: Always enable busy-stats for execlis

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists

2020-04-25 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists URL : https://patchwork.freedesktop.org/series/76486/ State : success == Summary == CI Bug Log - changes from CI_DRM_8368 -> Patchwork_17463

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists

2020-04-25 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists URL : https://patchwork.freedesktop.org/series/76486/ State : warning == Summary == $ dim sparse origin/drm-tip Sparse version: v0.6.0 Commit: drm/i915/gt: Always enable busy-stats for

[Intel-gfx] [PATCH] drm/i915/gt: Switch to manual evaluation of RPS

2020-04-25 Thread Chris Wilson
As with the realisation for soft-rc6, we respond to idling the engines within microseconds, far faster than the response times for HW RC6 and RPS. Furthermore, our fast parking upon idle, prevents HW RPS from running for many desktop workloads, as the RPS evaluation intervals are on the order of te

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists

2020-04-25 Thread Patchwork
== Series Details == Series: series starting with [1/6] drm/i915/gt: Always enable busy-stats for execlists URL : https://patchwork.freedesktop.org/series/76486/ State : warning == Summary == $ dim checkpatch origin/drm-tip 310c2635e565 drm/i915/gt: Always enable busy-stats for execlists 130a

Re: [Intel-gfx] [PATCH 4/6] drm/i915/gt: Switch to manual evaluation of RPS

2020-04-25 Thread Chris Wilson
Quoting Chris Wilson (2020-04-25 18:57:49) > +static bool has_busy_stats(struct intel_rps *rps) > +{ > + struct intel_engine_cs *engine; > + enum intel_engine_id id; > + > + for_each_engine(engine, rps_to_gt(rps), id) { > + if (!intel_engine_supports_stats(engine))

[Intel-gfx] [PATCH 4/6] drm/i915/gt: Switch to manual evaluation of RPS

2020-04-25 Thread Chris Wilson
As with the realisation for soft-rc6, we respond to idling the engines within microseconds, far faster than the response times for HW RC6 and RPS. Furthermore, our fast parking upon idle, prevents HW RPS from running for many desktop workloads, as the RPS evaluation intervals are on the order of te

[Intel-gfx] [PATCH 2/6] drm/i915/gt: Move rps.enabled/active to flags

2020-04-25 Thread Chris Wilson
Pull the boolean intel_rps.enabled and intel_rps.active into a single flags field, in preparation for more. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gt/debugfs_gt_pm.c | 5 +-- drivers/gpu/drm/i915/gt/intel_rps.c | 37 +-- drivers/gpu/drm/i915/gt/intel_rp

[Intel-gfx] [PATCH 3/6] drm/i915/gt: Track use of RPS interrupts in flags

2020-04-25 Thread Chris Wilson
Use the new intel_rps.flags field to store whether or not interrupts are being used with RPS. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gt/intel_rps.c | 13 - drivers/gpu/drm/i915/gt/intel_rps.h | 10 ++ drivers/gpu/drm/i915/gt/intel_rps_types.h | 1 +

[Intel-gfx] [PATCH 5/6] drm/i915/gt: Apply the aggressive downclocking to parking

2020-04-25 Thread Chris Wilson
We treat parking as a manual RPS timeout event, and downclock the GPU for the next unpark and batch execution. However, having restored the aggressive downclocking and observed that we have very light workloads whose only interaction is through the manual parking events, carry over the aggressive d

[Intel-gfx] [PATCH 6/6] drm/i915/gt: Restore aggressive post-boost downclocking

2020-04-25 Thread Chris Wilson
We reduced the clocks slowly after a boost event based on the observation that the smoothness of animations suffered. However, since reducing the evalution intervals, we should be able to respond to the rapidly fluctuating workload of a simple desktop animation and so restore the more aggressive do

[Intel-gfx] [PATCH 1/6] drm/i915/gt: Always enable busy-stats for execlists

2020-04-25 Thread Chris Wilson
In the near future, we will utilize the busy-stats on each engine to approximate the C0 cycles of each, and use that as an input to a manual RPS mechanism. That entails having busy-stats always enabled and so we can remove the enable/disable routines and simplify the pmu setup. As a consequence of

Re: [Intel-gfx] [PATCH 46/59] drm/arc: Align with simple pipe helpers

2020-04-25 Thread Sam Ravnborg
Hi Daniel. On Wed, Apr 15, 2020 at 09:40:21AM +0200, Daniel Vetter wrote: > Simple pipe helpers only have an enable and disable hook, no more > mode_set_nofb. Call it from our enable hook to align with that > conversions. > > Atomic helpers always call mode_set_nofb and enable together, so > ther

Re: [Intel-gfx] [CI 1/2] drm/i915/selftests: Verify context isolation

2020-04-25 Thread Abodunrin, Akeem G
> -Original Message- > From: Intel-gfx On Behalf Of Chris > Wilson > Sent: Friday, April 24, 2020 1:33 AM > To: intel-gfx@lists.freedesktop.org > Subject: [Intel-gfx] [CI 1/2] drm/i915/selftests: Verify context isolation > > No unprivileged context should ever be allowed to modify logi