Re: [Intel-gfx] [PATCH i-g-t 1/2] tests/kms_setmode: Request the intiial vbl count with RELATIVE instead of ABSOLUTE

On Tue, Oct 10, 2017 at 11:16:23AM +0200, Daniel Vetter wrote: > On Mon, Oct 09, 2017 at 07:33:49PM +0300, Ville Syrjala wrote: > > From: Ville Syrjälä > > > > Asking for the initial vblank count by specifying and absolute vblank count > > of 0 > > doesn't make

[Intel-gfx] ✗ Fi.CI.BAT: failure for IGT PMU support (rev7)

== Series Details == Series: IGT PMU support (rev7) URL : https://patchwork.freedesktop.org/series/28253/ State : failure == Summary == IGT patchset build failed on latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2952901 syncobj: Add a test for SYNCOBJ_CREATE_SIGNALED make

Re: [Intel-gfx] [RFC PATCH 01/11] drm/i915: No need for RING_MAX_NONPRIV_SLOTS space

Oscar Mateo writes: > Now that we write RING_FORCE_TO_NONPRIV registers directly to hardware, > there is no need to save space for them in the list of context workarounds. > > Signed-off-by: Oscar Mateo > Cc: Chris Wilson

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES (rev2)

== Series Details == Series: drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES (rev2) URL : https://patchwork.freedesktop.org/series/30336/ State : success == Summary == Series 30336v2 drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Use rcu instead of stop_machine in set_wedged

Quoting Daniel Vetter (2017-10-09 17:44:01) > stop_machine is not really a locking primitive we should use, except > when the hw folks tell us the hw is broken and that's the only way to > work around it. > > This patch tries to address the locking abuse of stop_machine() from > > commit

[Intel-gfx] [PATCH v3 2/3] drm/i915: Don't give up waiting on INVALID_MODE

Our current logic to read LSPCON's current mode, stops retries and breaks wait-loop, if it gets LSPCON_MODE_INVALID as return from the core function. This doesn't allow us to try reading the mode again. This patch removes this condition and allows retries reading the currnt mode until timeout.

[Intel-gfx] [PATCH v3 0/3] Various retries for LSPCON

This patch series adds various retries and delays in DRM and I915 layer, to handle some of the read failures while dealing with LSPCON devices. LSPCON devices are sometimes slow to respond and the vendores expect us to retry while probing the device. The typical scenarios are getting fixed /

[Intel-gfx] [PATCH v3 3/3] drm/i915: Add retries for LSPCON detection

We read the dp dual mode Adapter identifier to detect the LSPCON device. It's been observed from the CI testing that in few cases, this read can get delayed or fail. For such scenarios, LSPCON vendors suggest to retry the read operation. This patch adds retry in the probe function, while reading

[Intel-gfx] [PATCH v3 1/3] drm: Add retries for lspcon mode detection

From the CI builds, its been observed that during a driver reload/insert, dp dual mode read function sometimes fails to read from LSPCON device over i2c-over-aux channel. This patch: - adds some delay and few retries, allowing a scope for these devices to settle down and respond. - changes one

[Intel-gfx] [PATCH] drm/i915: Silently fallback to 4k scratch

If we fail to allocate a 64k hugepage for scratch, we try again with a normal 4k page (with some loss of efficiency at runtime). As we handle this gracefully, we do not need a noisy allocation failure warning. Signed-off-by: Chris Wilson Cc: Matthew Auld

Re: [Intel-gfx] [PATCH i-g-t] tests/kms_sequence: Add tests for lease ioctls

On 10 October 2017 at 16:57, Dave Airlie wrote: > On 1 October 2017 at 13:52, wrote: >> From: Keith Packard >> >> Validate that the leasing API creates leases that allow access to a >> subset of the available resources and that lease

[Intel-gfx] [PATCH] drm/i915: Increase atomic update vblank evasion time with lockdep

All our mmio writes take forever with lockdep due to the constant lock acquire we do. Ville has some patches to only acquire the mmio spinlocks once instead for every single mmio, but those aren't ready yet. As an interim solution just extend our budget slightly when lockdep is enabled, to avoid

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Increase atomic update vblank evasion time with lockdep

== Series Details == Series: drm/i915: Increase atomic update vblank evasion time with lockdep URL : https://patchwork.freedesktop.org/series/31630/ State : success == Summary == Series 31630v1 drm/i915: Increase atomic update vblank evasion time with lockdep

Re: [Intel-gfx] [PATCH 0/5] drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES

On Thu, Sep 14, 2017 at 10:53:01AM +0300, Mika Kahola wrote: > This patch series introduces fixes to reduce dependency for > I915_MAX_PIPES and minor optimizations to reduce hardcoding. > > Kahola, Mika (1): > drm/i915: Remove I915_MAX_PIPES dependency for DDB allocation > > Mika Kahola (4): >

[Intel-gfx] [PATCH] drm/i915: Increase atomic update vblank evasion time with lockdep

All our mmio writes take forever with lockdep due to the constant lock acquire we do. Ville has some patches to only acquire the mmio spinlocks once instead for every single mmio, but those aren't ready yet. As an interim solution just extend our budget slightly when lockdep is enabled, to avoid

[Intel-gfx] [PATCH i-g-t 7/9] tests/perf_pmu: Tests for i915 PMU API

From: Tvrtko Ursulin A bunch of tests for the new i915 PMU feature. Parts of the code were initialy sketched by Dmitry Rogozhkin. v2: (Most suggestions by Chris Wilson) * Add new class/instance based engine list. * Add gem_has_engine/gem_require_engine to work with

[Intel-gfx] [PATCH i-g-t 8/9] gem_wsim: Busy stats balancers

From: Tvrtko Ursulin Add busy and busy-avg balancers which make balancing decisions by looking at engine busyness via the i915 PMU. And thus are able to make decisions on the actual instantaneous load of the system, and not use metrics that lag behind by a batch or

[Intel-gfx] [PATCH i-g-t 5/9] intel-gpu-overlay: Catch-up to new i915 PMU

From: Tvrtko Ursulin v2: Update for i915 changes. v3: Use 1eN for large numbers. (Chris Wilson) Signed-off-by: Tvrtko Ursulin Reviewed-by: Chris Wilson --- lib/igt_perf.h | 89

[Intel-gfx] [PATCH i-g-t 4/9] intel-gpu-overlay: Fix interrupts PMU readout

From: Tvrtko Ursulin Signed-off-by: Tvrtko Ursulin --- overlay/gem-interrupts.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/overlay/gem-interrupts.c b/overlay/gem-interrupts.c index a84aef0398a7..3eda24f4d7eb

[Intel-gfx] [PATCH i-g-t 2/9] intel-gpu-overlay: Consolidate perf PMU access to library

From: Tvrtko Ursulin Various tool modules implement their owm PMU open wrapper which can be replaced by calling the library one. v2: * Remove extra newline. (Chris Wilson) * Commit msg. Signed-off-by: Tvrtko Ursulin --- lib/igt_perf.c

[Intel-gfx] [PATCH i-g-t 3/9] lib/perf: Fix data types and general tidy

From: Tvrtko Ursulin Configuration and format are uint64_t in the perf API. Tidy some other details as well. Signed-off-by: Tvrtko Ursulin --- lib/igt_perf.c | 40 +++- lib/igt_perf.h | 4 ++-- 2 files

[Intel-gfx] [PATCH i-g-t 6/9] intel-gpu-overlay: Use RAPL PMU for power reading

From: Tvrtko Ursulin Wire up to the RAPL PMU for GPU energy readings. The only complication is that we have to add code to parse: # cat /sys/devices/power/events/energy-gpu.scale 2.3283064365386962890625e-10 Signed-off-by: Tvrtko Ursulin

[Intel-gfx] [PATCH i-g-t 1/9] intel-gpu-overlay: Move local perf implementation to a library

From: Tvrtko Ursulin Idea is to avoid duplication across multiple users in upcoming patches. v2: Commit message and use a separate library instead of piggy- backing to libintel_tools. (Chris Wilson) Signed-off-by: Tvrtko Ursulin ---

[Intel-gfx] [PATCH v4 i-g-t 0/7] IGT PMU support

From: Tvrtko Ursulin 1. Fixes for intel-gpu-overlay to work on top of the proposed i915 PMU perf API. 2. New test to exercise the same API. 3. Update to gem_wsim and media-bench.pl to be able to use engine busyness via PMU for making balancing decisions. v2: * Added

[Intel-gfx] [PATCH i-g-t 9/9] media-bench.pl: Add busy balancers to the list

From: Tvrtko Ursulin Signed-off-by: Tvrtko Ursulin --- scripts/media-bench.pl | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/scripts/media-bench.pl b/scripts/media-bench.pl index 0956ef0a0621..78f45199e95d 100755 ---

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Use rcu instead of stop_machine in set_wedged

Daniel Vetter writes: > stop_machine is not really a locking primitive we should use, except > when the hw folks tell us the hw is broken and that's the only way to > work around it. > > This patch tries to address the locking abuse of stop_machine() from > > commit

Re: [Intel-gfx] [PATCH 0/5] drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES

On Tue, 2017-10-10 at 02:20 -0700, Kenneth Graunke wrote: > On Thursday, September 14, 2017 12:53:01 AM PDT Mika Kahola wrote: > > > > This patch series introduces fixes to reduce dependency for > > I915_MAX_PIPES and minor optimizations to reduce hardcoding. > > > > Kahola, Mika (1): > >  

[Intel-gfx] [PATCH FOR CI 5/5] drm/i915: Cleanup South Error Interrupts

Cleanup and parametrize the handling of South Error Interrupts (SERR_INT). Reviewed-by: Ville Syrjälä Signed-off-by: Mika Kahola --- drivers/gpu/drm/i915/i915_irq.c | 12 drivers/gpu/drm/i915/i915_reg.h | 3 --- 2 files

[Intel-gfx] [PATCH FOR CI 2/5] drm/i915: Remove I915_MAX_PIPES dependency for DDB allocation

Remove dependency for I915_MAX_PIPES by replacing it with for_each_pipe() macro. v2: use 'enum pipe pipe' instead of 'i' Reviewed-by: Ville Syrjälä Signed-off-by: Ramalingam C Signed-off-by: Mika Kahola ---

[Intel-gfx] [PATCH FOR CI 3/5] drm/i915: Fold IRQ pipe masks

Fold IRQ pipe masks into one loop instead of hardcoding per pipe. Reviewed-by: Ville Syrjälä Signed-off-by: Mika Kahola --- drivers/gpu/drm/i915/i915_irq.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git

[Intel-gfx] [PATCH FOR CI 4/5] drm/i915: Favor for_each_pipe() macro

Favor for_each_pipe() macro when looping through pipes. v2: use 'enum pipe pipe' instead of 'i' Reviewed-by: Ville Syrjälä Signed-off-by: Mika Kahola --- drivers/gpu/drm/i915/intel_pipe_crc.c | 8 1 file changed, 4 insertions(+),

Re: [Intel-gfx] [PATCH v3 0/3] Various retries for LSPCON

Hi, > -Original Message- > From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf > Of Shashank Sharma > Sent: tiistai 10. lokakuuta 2017 13.08 > To: intel-gfx@lists.freedesktop.org > Subject: [Intel-gfx] [PATCH v3 0/3] Various retries for LSPCON > > This patch

[Intel-gfx] [PATCH FOR CI 0/5] drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES

This patch series introduces fixes to reduce dependency for I915_MAX_PIPES and minor optimizations to reduce hardcoding. Mika Kahola (5): drm/i915: Don't relay on I915_MAX_PIPES drm/i915: Remove I915_MAX_PIPES dependency for DDB allocation drm/i915: Fold IRQ pipe masks drm/i915: Favor

[Intel-gfx] [PATCH FOR CI 1/5] drm/i915: Don't relay on I915_MAX_PIPES

Let's remove the dependency on I915_MAX_PIPES. Instead, get the number of pipes from platform information. Reviewed-by: Ville Syrjälä Signed-off-by: Ramalingam C Signed-off-by: Mika Kahola ---

[Intel-gfx] [PATCH] i-g-t/tests: Drop gem_seqno_wrap, gem_pin, gem_hangcheck_forcewake

This improves the GEM tests section of I-G-T to make it more suitable for CI testing Cc: Joonas Lahtinen Signed-off-by: Abdiel Janulgue --- tests/Makefile.sources | 3 - tests/gem_hangcheck_forcewake.c | 123

Re: [Intel-gfx] [PATCH 4/5] drm/i915: Favor for_each_pipe() macro

On Thu, Sep 14, 2017 at 10:53:05AM +0300, Mika Kahola wrote: > Favor for_each_pipe() macro when looping through pipes. > > Signed-off-by: Mika Kahola > --- > drivers/gpu/drm/i915/intel_pipe_crc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [Intel-gfx] [PATCH 0/5] drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES

On Tue, 2017-10-10 at 12:19 +0300, Ville Syrjälä wrote: > On Thu, Sep 14, 2017 at 10:53:01AM +0300, Mika Kahola wrote: > > > > This patch series introduces fixes to reduce dependency for > > I915_MAX_PIPES and minor optimizations to reduce hardcoding. > > > > Kahola, Mika (1): > >   drm/i915:

Re: [Intel-gfx] [PATCH 1/8] drm/i915/bios: parse DDI ports also for CHV for HDMI DDC pin and DP AUX channel

On Mon, 09 Oct 2017, Ville Syrjälä wrote: > On Thu, Sep 28, 2017 at 11:21:57AM +0300, Jani Nikula wrote: >> While technically CHV isn't DDI, we do look at the VBT based DDI port >> info for HDMI DDC pin and DP AUX channel. (We call these "alternate", >> but they're

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Preallocate our mmu notifier workequeu to unbreak cpu hotplug deadlock

On Mon, Oct 09, 2017 at 06:44:00PM +0200, Daniel Vetter wrote: > 4.14-rc1 gained the fancy new cross-release support in lockdep, which > seems to have uncovered a few more rules about what is allowed and > isn't. > > This one here seems to indicate that allocating a work-queue while > holding

Re: [Intel-gfx] [PATCH v15 6/7] drm/i915: Introduce GEM proxy

On Tue, 2017-10-10 at 17:50 +0800, Tina Zhang wrote: > GEM proxy is a kind of GEM, whose backing physical memory is pinned > and produced by guest VM and is used by host as read only. With GEM > proxy, host is able to access guest physical memory through GEM object > interface. As GEM proxy is

Re: [Intel-gfx] [PATCH] drm/i915: Increase atomic update vblank evasion time with lockdep

On Tue, Oct 10, 2017 at 10:46:57AM +0200, Daniel Vetter wrote: > All our mmio writes take forever with lockdep due to the constant > lock acquire we do. Ville has some patches to only acquire > the mmio spinlocks once instead for every single mmio, but those > aren't ready yet. > > As an interim

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: Increase atomic update vblank evasion time with lockdep (rev2)

== Series Details == Series: drm/i915: Increase atomic update vblank evasion time with lockdep (rev2) URL : https://patchwork.freedesktop.org/series/31630/ State : failure == Summary == Test kms_flip: Subgroup modeset-vs-vblank-race-interruptible: pass -> FAIL

Re: [Intel-gfx] [PATCH] i-g-t/tests: Drop gem_seqno_wrap, gem_pin, gem_hangcheck_forcewake

+ Chris and Mika for R-b's as they're the test authors + Arek, Tomi for CI blacklist reduction On Tue, 2017-10-10 at 11:55 +0300, Abdiel Janulgue wrote: > This improves the GEM tests section of I-G-T to make it more > suitable for CI testing > > Cc: Joonas Lahtinen

Re: [Intel-gfx] [PATCH] drm/i915: Silently fallback to 4k scratch

On Tue, 2017-10-10 at 12:10 +0100, Chris Wilson wrote: > If we fail to allocate a 64k hugepage for scratch, we try again with a > normal 4k page (with some loss of efficiency at runtime). As we handle > this gracefully, we do not need a noisy allocation failure warning. > > Signed-off-by: Chris

[Intel-gfx] [PATCH i-g-t v2 6/9] intel-gpu-overlay: Use RAPL PMU for power reading

From: Tvrtko Ursulin Wire up to the RAPL PMU for GPU energy readings. The only complication is that we have to add code to parse: # cat /sys/devices/power/events/energy-gpu.scale 2.3283064365386962890625e-10 v2: Link with -lm. Signed-off-by: Tvrtko Ursulin

Re: [Intel-gfx] [PATCH 0/5] drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES

On Thursday, September 14, 2017 12:53:01 AM PDT Mika Kahola wrote: > This patch series introduces fixes to reduce dependency for > I915_MAX_PIPES and minor optimizations to reduce hardcoding. > > Kahola, Mika (1): > drm/i915: Remove I915_MAX_PIPES dependency for DDB allocation > > Mika Kahola

Re: [Intel-gfx] [PATCH] drm/i915: Allow null render state batchbuffers bigger than one page

Quoting Rodrigo Vivi (2017-10-05 05:34:02) > On Thu, Aug 24, 2017 at 11:00:27PM +, Rodrigo Vivi wrote: > > On Thu, Aug 24, 2017 at 3:39 PM, Oscar Mateo wrote: > > > > > > > > > On 08/23/2017 05:01 PM, Rodrigo Vivi wrote: > > >> > > >> On Tue, Jul 18, 2017 at 8:15 AM,

[Intel-gfx] ✓ Fi.CI.BAT: success for Various retries for LSPCON

== Series Details == Series: Various retries for LSPCON URL : https://patchwork.freedesktop.org/series/31639/ State : success == Summary == Series 31639v1 Various retries for LSPCON https://patchwork.freedesktop.org/api/1.0/series/31639/revisions/1/mbox/ Test gem_exec_suspend:

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES

== Series Details == Series: drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES URL : https://patchwork.freedesktop.org/series/30336/ State : success == Summary == Series 30336v1 drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES

Re: [Intel-gfx] [PATCH i-g-t 1/2] tests/kms_setmode: Request the intiial vbl count with RELATIVE instead of ABSOLUTE

On Mon, Oct 09, 2017 at 07:33:49PM +0300, Ville Syrjala wrote: > From: Ville Syrjälä > > Asking for the initial vblank count by specifying and absolute vblank count > of 0 > doesn't make much sense. Switch to a relative query instead. > > Signed-off-by: Ville

Re: [Intel-gfx] [PATCH 2/5] drm/i915: Remove I915_MAX_PIPES dependency for DDB allocation

On Thu, Sep 14, 2017 at 10:53:03AM +0300, Mika Kahola wrote: > From: "Kahola, Mika" > > Remove dependency for I915_MAX_PIPES by replacing it with > for_each_pipe() macro. > > Signed-off-by: Ramalingam C > Signed-off-by: Kahola, Mika

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gvt: Dma-buf support for GVT-g

== Series Details == Series: drm/i915/gvt: Dma-buf support for GVT-g URL : https://patchwork.freedesktop.org/series/31638/ State : failure == Summary == CHK include/config/kernel.release CHK include/generated/uapi/linux/version.h CHK include/generated/utsrelease.h CHK

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Increase atomic update vblank evasion time with lockdep

== Series Details == Series: drm/i915: Increase atomic update vblank evasion time with lockdep URL : https://patchwork.freedesktop.org/series/31630/ State : success == Summary == shard-hswtotal:2552 pass:1412 dwarn:24 dfail:0 fail:13 skip:1103 time:9576s == Logs == For more

Re: [Intel-gfx] [PATCH] drm/i915: Silently fallback to 4k scratch

On 10 October 2017 at 12:10, Chris Wilson wrote: > If we fail to allocate a 64k hugepage for scratch, we try again with a > normal 4k page (with some loss of efficiency at runtime). As we handle > this gracefully, we do not need a noisy allocation failure warning. > >

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: Increase atomic update vblank evasion time with lockdep (rev2)

== Series Details == Series: drm/i915: Increase atomic update vblank evasion time with lockdep (rev2) URL : https://patchwork.freedesktop.org/series/31630/ State : success == Summary == Series 31630v2 drm/i915: Increase atomic update vblank evasion time with lockdep

[Intel-gfx] [PATCH v15 0/7] drm/i915/gvt: Dma-buf support for GVT-g

v14->15: 1) Add VFIO_DEVICE_GET_GFX_DMABUF ABI. (Gerd) 2) Add intel_vgpu_dmabuf_cleanup() to clean up the vGPU's dmabuf. (Gerd) v13->v14: 1) add PROBE, DMABUF and REGION flags. (Alex) 2) return -ENXIO when gem proxy object is banned by ioctl. (Chris) (Daniel) 3) add some details about the

[Intel-gfx] [PATCH v15 7/7] drm/i915/gvt: Dmabuf support for GVT-g

This patch introduces a guest's framebuffer sharing mechanism based on dma-buf subsystem. With this sharing mechanism, guest's framebuffer can be shared between guest VM and host. v15: - Add VFIO_DEVICE_GET_GFX_DMABUF ABI. (Gerd) - Add intel_vgpu_dmabuf_cleanup() to clean up the vGPU's dmabuf.

[Intel-gfx] [PATCH v15 4/7] drm/i915/gvt: Add opregion support

Windows guest driver needs vbt in opregion, to configure the setting for display. Without opregion support, the display registers won't be set and this blocks display model to get the correct information of the guest display plane. This patch is to provide a virtual opregion for guest. Current

[Intel-gfx] [PATCH v15 6/7] drm/i915: Introduce GEM proxy

GEM proxy is a kind of GEM, whose backing physical memory is pinned and produced by guest VM and is used by host as read only. With GEM proxy, host is able to access guest physical memory through GEM object interface. As GEM proxy is such a special kind of GEM, a new flag I915_GEM_OBJECT_IS_PROXY

[Intel-gfx] [PATCH v15 3/7] drm/i915/gvt: Add RGB 64-bit 16:16:16:16 float format

The RGB 64-bit 16:16:16:16 float pixel format is needed by windows 10 guest VM. This patch is to add this pixel format support to gvt device model. Without this patch, some Apps, e.g. "DXGIGammaVM.exe", will crash and make guest screen black. Signed-off-by: Tina Zhang ---

[Intel-gfx] [PATCH v15 2/7] drm: Introduce RGB 64-bit 16:16:16:16 float format

The RGB 64-bit 16:16:16:16 float pixel format is needed by some Apps in windows. The float format in each component is 1:5:10 MSb-sign:exponent: fraction. This patch is to introduce the format to drm, so that the windows guest's framebuffer in this kind of format can be recognized and used by

[Intel-gfx] [PATCH v15 1/7] drm/i915/gvt: Add framebuffer decoder support

This patch is to introduce the framebuffer decoder which can decode guest OS's framebuffer information, including primary, cursor and sprite plane. v14: - refine pixel format table. (Zhenyu) v9: - move drm format change to a separate patch. (Xiaoguang) v8: - fix a bug in decoding primary plane.

[Intel-gfx] [PATCH v15 5/7] vfio: ABI for mdev display dma-buf operation

Add VFIO_DEVICE_QUERY_GFX_PLANE ioctl command to let user mode query and get the plan and its related information. This ioctl can be invoked with: 1) either flag DMABUF or REGION is set. Vendor driver returns success and the plane_info only when the specific kind of buffer is supported. 2) flag

Re: [Intel-gfx] [PATCH] drm/i915: Allow null render state batchbuffers bigger than one page

Quoting Chris Wilson (2017-10-10 11:25:38) > Quoting Rodrigo Vivi (2017-10-05 05:34:02) > > On Thu, Aug 24, 2017 at 11:00:27PM +, Rodrigo Vivi wrote: > > > On Thu, Aug 24, 2017 at 3:39 PM, Oscar Mateo > > > wrote: > > > > > > > > > > > > On 08/23/2017 05:01 PM, Rodrigo

Re: [Intel-gfx] [PATCH v3 0/3] Various retries for LSPCON

Regards Shashank On 10/10/2017 3:42 PM, Saarinen, Jani wrote: Hi, -Original Message- From: Intel-gfx [mailto:intel-gfx-boun...@lists.freedesktop.org] On Behalf Of Shashank Sharma Sent: tiistai 10. lokakuuta 2017 13.08 To: intel-gfx@lists.freedesktop.org Subject: [Intel-gfx] [PATCH

[Intel-gfx] [PATCH 13/14] drm/i915: Add NV12 as supported format for sprite plane

From: Chandra Konduru This patch adds NV12 to list of supported formats for sprite plane. v2: Rebased (me) v3: Review comments by Ville addressed - Removed skl_plane_formats_with_nv12 and added NV12 case in existing skl_plane_formats - Added

[Intel-gfx] [PATCH 12/14] drm/i915: Add NV12 as supported format for primary plane

From: Chandra Konduru This patch adds NV12 to list of supported formats for primary plane v2: Rebased (Chandra Konduru) v3: Rebased (me) v4: Review comments by Ville addressed Removed the skl_primary_formats_with_nv12 and added NV12 case in existing

[Intel-gfx] [PATCH 14/14] drm/i915: Add NV12 support to intel_framebuffer_init

From: Chandra Konduru This patch adds NV12 as supported format to intel_framebuffer_init and performs various checks. v2: -Fix an issue in checks added (Chandra Konduru) v3: rebased (me) v4: Review comments by Ville addressed Added platform check for NV12 in

Re: [Intel-gfx] [PATCH] drm/i915: Use execlists_num_ports instead of size of array

Quoting Mika Kuoppala (2017-10-10 12:48:57) > There is function to tell how many ports we have, so use it. > We still have direct relationship with array size and port count, > so no harm was done. > > Fixes: 76e70087d360 ("drm/i915: Make execlist port count variable") > Cc: Mika Kuoppala

Re: [Intel-gfx] [PATCH i-g-t 3/9] lib/perf: Fix data types and general tidy

Quoting Tvrtko Ursulin (2017-10-10 10:30:02) > From: Tvrtko Ursulin > > Configuration and format are uint64_t in the perf API. Planning for a busy few years? ;) > Tidy some other details as well. > > Signed-off-by: Tvrtko Ursulin

Re: [Intel-gfx] [PATCH i-g-t 4/9] intel-gpu-overlay: Fix interrupts PMU readout

Quoting Tvrtko Ursulin (2017-10-10 10:30:03) > From: Tvrtko Ursulin > > Signed-off-by: Tvrtko Ursulin > --- > overlay/gem-interrupts.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/overlay/gem-interrupts.c

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Use rcu instead of stop_machine in set_wedged

On Tue, Oct 10, 2017 at 12:50:45PM +0300, Mika Kuoppala wrote: > Daniel Vetter writes: > > > stop_machine is not really a locking primitive we should use, except > > when the hw folks tell us the hw is broken and that's the only way to > > work around it. > > > > This

Re: [Intel-gfx] [PATCH i-g-t 7/9] tests/perf_pmu: Tests for i915 PMU API

Quoting Tvrtko Ursulin (2017-10-10 10:30:06) > +static void > +event_wait(int gem_fd, const struct intel_execution_engine2 *e) > +{ > + struct drm_i915_gem_exec_object2 obj = { }; > + struct drm_i915_gem_execbuffer2 eb = { }; > + data_t data; > + igt_display_t *display = >

[Intel-gfx] ✓ Fi.CI.BAT: success for Adding NV12 support (rev3)

== Series Details == Series: Adding NV12 support (rev3) URL : https://patchwork.freedesktop.org/series/28103/ State : success == Summary == Series 28103v3 Adding NV12 support https://patchwork.freedesktop.org/api/1.0/series/28103/revisions/3/mbox/ Test chamelium: Subgroup

[Intel-gfx] [CI i-g-t] tests: Drop gem_bad_length as redundant

From: Daniele Ceraolo Spurio Most of the gem_bad_length code is compiled out because creating a zero-length object is not allowed anymore by i915 and thus it is not possible to execute it. The remaining part checks that creation of a zero-length object does

Re: [Intel-gfx] [PATCH v15 6/7] drm/i915: Introduce GEM proxy

On Tue, 2017-10-10 at 13:58 +0300, Joonas Lahtinen wrote: > On Tue, 2017-10-10 at 17:50 +0800, Tina Zhang wrote: > > GEM proxy is a kind of GEM, whose backing physical memory is pinned > > and produced by guest VM and is used by host as read only. With GEM > > proxy, host is able to access guest

Re: [Intel-gfx] [PATCH] drm/i915/selftests: ditch the kernel context

Quoting Matthew Auld (2017-10-10 14:30:30) > There's really no good reason to be using the kernel context for the > huge-page livetests. Also with the introduction of commit bef27bdb6cfb > ("drm/i915: Assert we do not try to expand VMA for hugepage inside GGTT") > we start hitting the bug on in

Re: [Intel-gfx] [PATCH 11/14] drm/i915: Upscale scaler max scale for NV12

On Tuesday 10 October 2017 05:47 PM, Vidya Srinivas wrote: From: Chandra Konduru This patch updates scaler max limit support for NV12 v2: Rebased (me) v3: Rebased (me) v4: Missed the Tested-by/Reviewed-by in the previous series Adding the same to commit

Re: [Intel-gfx] [PATCH v15 5/7] vfio: ABI for mdev display dma-buf operation

On Tue, 2017-10-10 at 17:50 +0800, Tina Zhang wrote: > Add VFIO_DEVICE_QUERY_GFX_PLANE ioctl command to let user mode query > and > get the plan and its related information. This ioctl can be invoked > with: > 1) either flag DMABUF or REGION is set. Vendor driver returns success > and > the

[Intel-gfx] [PATCH i-g-t v6 7/9] tests/perf_pmu: Tests for i915 PMU API

From: Tvrtko Ursulin A bunch of tests for the new i915 PMU feature. Parts of the code were initialy sketched by Dmitry Rogozhkin. v2: (Most suggestions by Chris Wilson) * Add new class/instance based engine list. * Add gem_has_engine/gem_require_engine to work with

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Silently fallback to 4k scratch

== Series Details == Series: drm/i915: Silently fallback to 4k scratch URL : https://patchwork.freedesktop.org/series/31646/ State : success == Summary == shard-hswtotal:2552 pass:1431 dwarn:5 dfail:0 fail:13 skip:1103 time:9667s == Logs == For more details see:

[Intel-gfx] [PATCH i-g-t v2 4/9] intel-gpu-overlay: Fix interrupts PMU readout

From: Tvrtko Ursulin v2: Use correct address of. (Chris Wilson) Signed-off-by: Tvrtko Ursulin Reviewed-by: Chris Wilson --- overlay/gem-interrupts.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff

Re: [Intel-gfx] [PATCH] drm/i915: remove redundant check on has_aliasing_ppgtt

On 10/10/17 15:35, Joonas Lahtinen wrote: > On Tue, 2017-10-10 at 14:47 +0100, Colin King wrote: >> From: Colin Ian King >> >> There is a previous check to on has_aliasing_ppgtt that returns >> 0 if it is false, so it is impossible for has_aliasing_ppgtt to >> be false

[Intel-gfx] [PATCH] drm/i915: Use execlists_num_ports instead of size of array

There is function to tell how many ports we have, so use it. We still have direct relationship with array size and port count, so no harm was done. Fixes: 76e70087d360 ("drm/i915: Make execlist port count variable") Cc: Mika Kuoppala Cc: Chris Wilson

[Intel-gfx] [PATCH 7/9] drm/i915: Split intel_disable_ddi() into DP vs. HDMI variants

From: Ville Syrjälä Untangle intel_disable_ddi() by splitting it into DP and HDMI specific variants. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_ddi.c | 51 1 file changed,

[Intel-gfx] [PATCH v2 0/9] drm/i915: Cleaner DDI DP vs. HDMI split

From: Ville Syrjälä Here's a small selection of patches I extracted from my monster DDI encoder->type drobbery series. These mostly split the encoder hooks along the DP vs. HDMI lines. Hopefully it'll be a bit easier to get these reviewed when they're not buried

[Intel-gfx] [PATCH 6/9] drm/i915: Remove useless eDP check from intel_ddi_pre_enable_dp()

From: Ville Syrjälä intel_edp_panel_on() will itself do the is_edp() check, so the caller doesn't have to bother. Pre-DDI code doesn't bother, so let's follow the same approach for DDI. Reviewed-by: Jani Nikula Signed-off-by: Ville Syrjälä

[Intel-gfx] [PATCH v2 8/9] drm/i915: Plumb crtc_state etc. directly to intel_ddi_pre_enable_{dp, hdmi}()

From: Ville Syrjälä Rather that plumb the link parameters separately to intel_ddi_pre_enable_dp() let's just pass the entire crtc state. intel_ddi_pre_enable_hdmi() already took the crtc state, but for some reason intel_ddi_pre_enable() still wanted to extract

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Use rcu instead of stop_machine in set_wedged

On Tue, Oct 10, 2017 at 10:21:45AM +0100, Chris Wilson wrote: > Quoting Daniel Vetter (2017-10-09 17:44:01) > > stop_machine is not really a locking primitive we should use, except > > when the hw folks tell us the hw is broken and that's the only way to > > work around it. > > > > This patch

[Intel-gfx] ✗ Fi.CI.IGT: warning for drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES (rev2)

== Series Details == Series: drm/i915: Miscellaneous fixes to reduce dependency for I915_MAX_PIPES (rev2) URL : https://patchwork.freedesktop.org/series/30336/ State : warning == Summary == Test kms_setmode: Subgroup basic: fail -> PASS (shard-hsw)

[Intel-gfx] [PATCH] drm/i915: Use enum pipe for PCH transcoders

From: Ville Syrjälä One more place where we've failed to switch to enum pipe when talking about PCH transcoders. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_display.c | 6 +++--- 1 file changed, 3 insertions(+), 3

Re: [Intel-gfx] [PATCH i-g-t 7/9] tests/perf_pmu: Tests for i915 PMU API

Quoting Tvrtko Ursulin (2017-10-10 14:38:10) > > On 10/10/2017 13:37, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2017-10-10 10:30:06) > >> +static void > >> +event_wait(int gem_fd, const struct intel_execution_engine2 *e) > >> +{ > >> + struct drm_i915_gem_exec_object2 obj = { }; > >>

Re: [Intel-gfx] [PATCH 2/2] drm/i915: Use rcu instead of stop_machine in set_wedged

On Tue, Oct 10, 2017 at 3:39 PM, Chris Wilson wrote: > Style nits: > > Quoting Daniel Vetter (2017-10-09 17:44:01) >> diff --git a/drivers/gpu/drm/i915/i915_gem.c >> b/drivers/gpu/drm/i915/i915_gem.c >> index 82a10036fb38..8d7d8d1f78db 100644 >> ---

[Intel-gfx] ✓ Fi.CI.BAT: success for IGT PMU support (rev8)

== Series Details == Series: IGT PMU support (rev8) URL : https://patchwork.freedesktop.org/series/28253/ State : success == Summary == IGT patchset tested on top of latest successful build d7c88290ab6a8393dc341b30c7fb5e27d2952901 syncobj: Add a test for SYNCOBJ_CREATE_SIGNALED with latest

[Intel-gfx] [PATCH i-g-t v3 6/9] intel-gpu-overlay: Use RAPL PMU for power reading

From: Tvrtko Ursulin Wire up to the RAPL PMU for GPU energy readings. The only complication is that we have to add code to parse: # cat /sys/devices/power/events/energy-gpu.scale 2.3283064365386962890625e-10 v2: Link with -lm. v3: strtod can handle scientific

[Intel-gfx] [PATCH i-g-t 0/2] Add support for NV12

This series adds NV12 support to IGT and a test case for checking NV12 functionality. The series is based on the initial version posted by Chandra Konduru during 2015 based on the IGT framework available then. Previous version reference links: https://patchwork.freedesktop.org/patch/57590/

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Parse DSI backlight/cabc ports.

> -Original Message- > From: Nikula, Jani > Sent: Tuesday, October 10, 2017 12:47 PM > To: Chauhan, Madhav ; intel- > g...@lists.freedesktop.org > Cc: Hiremath, Shashidhar ; Shankar, Uma > ; Chauhan, Madhav >

[Intel-gfx] [PATCH] drm/i915/selftests: ditch the kernel context

There's really no good reason to be using the kernel context for the huge-page livetests. Also with the introduction of commit bef27bdb6cfb ("drm/i915: Assert we do not try to expand VMA for hugepage inside GGTT") we start hitting the bug on in the selftests, since the kernel context will always

Re: [Intel-gfx] [PATCH i-g-t 7/9] tests/perf_pmu: Tests for i915 PMU API

On 10/10/2017 13:37, Chris Wilson wrote: Quoting Tvrtko Ursulin (2017-10-10 10:30:06) +static void +event_wait(int gem_fd, const struct intel_execution_engine2 *e) +{ + struct drm_i915_gem_exec_object2 obj = { }; + struct drm_i915_gem_execbuffer2 eb = { }; + data_t data; +

[Intel-gfx] [PATCH i-g-t 1/2] Prep work for adding NV12 testcase

From: chandra konduru This patch adds necessary prep work for nv12 testcase: - updated fb allocation functions to handle NV12 format - igt helper function to return png image size - igt helper function to calculate start of uv in a given NV12 buffer - igt helper

  1   2   3   >