[Intel-gfx] ✗ Fi.CI.IGT: failure for drm-buf: Add debug option (rev6)

2021-01-15 Thread Patchwork
== Series Details == Series: drm-buf: Add debug option (rev6) URL : https://patchwork.freedesktop.org/series/85813/ State : failure == Summary == CI Bug Log - changes from CI_DRM_9625_full -> Patchwork_19377_full Summary ---

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/userptr: detect un-GUP-able pages early

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/userptr: detect un-GUP-able pages early URL : https://patchwork.freedesktop.org/series/85925/ State : failure == Summary == CI Bug Log - changes from CI_DRM_9624_full -> Patchwork_19376_full Summary

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [CI,1/5] drm/i915/gt: Drop atomic for engine->fw_active tracking

2021-01-15 Thread Patchwork
== Series Details == Series: series starting with [CI,1/5] drm/i915/gt: Drop atomic for engine->fw_active tracking URL : https://patchwork.freedesktop.org/series/85918/ State : failure == Summary == CI Bug Log - changes from CI_DRM_9624_full -> Patchwork_19374_full

[Intel-gfx] ✓ Fi.CI.IGT: success for HDCP misc fixes

2021-01-15 Thread Patchwork
== Series Details == Series: HDCP misc fixes URL : https://patchwork.freedesktop.org/series/85917/ State : success == Summary == CI Bug Log - changes from CI_DRM_9624_full -> Patchwork_19373_full Summary --- **SUCCESS** No

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/gem: Use GEM context tracking for i915_gem_objects (rev2)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/gem: Use GEM context tracking for i915_gem_objects (rev2) URL : https://patchwork.freedesktop.org/series/85907/ State : success == Summary == CI Bug Log - changes from CI_DRM_9624_full -> Patchwork_19372_full

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/debugfs : PM_REQ and PM_RES registers (rev3)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/debugfs : PM_REQ and PM_RES registers (rev3) URL : https://patchwork.freedesktop.org/series/85437/ State : failure == Summary == CI Bug Log - changes from CI_DRM_9623_full -> Patchwork_19371_full

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gen12: Add display render clear color decompression support (rev4)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/gen12: Add display render clear color decompression support (rev4) URL : https://patchwork.freedesktop.org/series/85877/ State : success == Summary == CI Bug Log - changes from CI_DRM_9626 -> Patchwork_19378

[Intel-gfx] ✗ Fi.CI.BUILD: failure for sna: Probe userptr storage on construction

2021-01-15 Thread Patchwork
== Series Details == Series: sna: Probe userptr storage on construction URL : https://patchwork.freedesktop.org/series/85934/ State : failure == Summary == Applying: sna: Probe userptr storage on construction error: sha1 information is lacking or useless (src/sna/kgem.c). error: could not

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/gen12: Add display render clear color decompression support (rev4)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/gen12: Add display render clear color decompression support (rev4) URL : https://patchwork.freedesktop.org/series/85877/ State : warning == Summary == $ dim sparse --fast origin/drm-tip Sparse version: v0.6.2 Fast mode used, each commit won't be checked

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/gen12: Add display render clear color decompression support (rev4)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/gen12: Add display render clear color decompression support (rev4) URL : https://patchwork.freedesktop.org/series/85877/ State : warning == Summary == $ dim checkpatch origin/drm-tip b3cc19e97e22 drm/framebuffer: Format modifier for Intel Gen 12 render

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/display: Set error value before returning in intel_hdcp*_disable() (rev2)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/display: Set error value before returning in intel_hdcp*_disable() (rev2) URL : https://patchwork.freedesktop.org/series/85895/ State : success == Summary == CI Bug Log - changes from CI_DRM_9623_full -> Patchwork_19370_full

[Intel-gfx] ✓ Fi.CI.BAT: success for drm-buf: Add debug option (rev6)

2021-01-15 Thread Patchwork
== Series Details == Series: drm-buf: Add debug option (rev6) URL : https://patchwork.freedesktop.org/series/85813/ State : success == Summary == CI Bug Log - changes from CI_DRM_9625 -> Patchwork_19377 Summary --- **SUCCESS** No

Re: [Intel-gfx] [PATCH 03/11] drm/i915: split fdi code out from intel_display.c

2021-01-15 Thread Jani Nikula
On Thu, 14 Jan 2021, Jani Nikula wrote: > From: Dave Airlie > > This just refactors out the fdi code to a separate file. > > Signed-off-by: Dave Airlie > [Jani: cleaned up intel_fdi.h a bit.] > Signed-off-by: Jani Nikula > Reviewed-by: Ville Syrjälä Thanks for the patches and review, pushed

[Intel-gfx] [PATCH xf86-video-intel] sna: Probe userptr storage on construction

2021-01-15 Thread Chris Wilson
Not all user address ranges are valid for userptr, as we can only utilize struct pages (and not indirect pfnmaps). Currently we use a set-domain call after userptr to validate the range by populating the userptr, but this has the drawback of forcing the population ever for userptr that are never

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm-buf: Add debug option (rev6)

2021-01-15 Thread Patchwork
== Series Details == Series: drm-buf: Add debug option (rev6) URL : https://patchwork.freedesktop.org/series/85813/ State : warning == Summary == $ dim checkpatch origin/drm-tip cd675d2b9e93 dma-buf: Add debug option -:85: WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a

[Intel-gfx] [PATCH v9 3/3] drm/i915/tgl: Add Clear Color support for TGL Render Decompression

2021-01-15 Thread Imre Deak
From: Radhakrishna Sripada Render Decompression is supported with Y-Tiled main surface. The CCS is linear and has 4 bits of data for each main surface cache line pair, a ratio of 1:256. Additional Clear Color information is passed from the user-space through an offset in the GEM BO. Add a new

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/userptr: detect un-GUP-able pages early

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/userptr: detect un-GUP-able pages early URL : https://patchwork.freedesktop.org/series/85925/ State : success == Summary == CI Bug Log - changes from CI_DRM_9624 -> Patchwork_19376 Summary ---

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/dp_helper: tweak kerneldoc to address warning

2021-01-15 Thread Patchwork
== Series Details == Series: drm/dp_helper: tweak kerneldoc to address warning URL : https://patchwork.freedesktop.org/series/85906/ State : failure == Summary == CI Bug Log - changes from CI_DRM_9622_full -> Patchwork_19369_full Summary

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915/userptr: detect un-GUP-able pages early

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/userptr: detect un-GUP-able pages early URL : https://patchwork.freedesktop.org/series/85925/ State : warning == Summary == $ dim checkpatch origin/drm-tip 84aaa6e33a39 drm/i915/userptr: detect un-GUP-able pages early -:106: CHECK:PARENTHESIS_ALIGNMENT:

[Intel-gfx] ✗ Fi.CI.BAT: failure for drm/i915/gem: Mark private i915_gem_stolen functions as static

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/gem: Mark private i915_gem_stolen functions as static URL : https://patchwork.freedesktop.org/series/85919/ State : failure == Summary == CI Bug Log - changes from CI_DRM_9624 -> Patchwork_19375

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915: add back static declaration

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915: add back static declaration URL : https://patchwork.freedesktop.org/series/85902/ State : failure == Summary == CI Bug Log - changes from CI_DRM_9622_full -> Patchwork_19368_full Summary ---

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915/gem: Mark private i915_gem_stolen functions as static

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/gem: Mark private i915_gem_stolen functions as static URL : https://patchwork.freedesktop.org/series/85919/ State : warning == Summary == $ dim sparse --fast origin/drm-tip Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

Re: [Intel-gfx] [PATCH] drm-buf: Add debug option

2021-01-15 Thread John Stultz
On Thu, Jan 14, 2021 at 1:03 AM Daniel Vetter wrote: > > On Wed, Jan 13, 2021 at 10:08 PM Chris Wilson > wrote: > > Quoting Daniel Vetter (2021-01-13 20:50:11) > > > On Wed, Jan 13, 2021 at 4:43 PM Chris Wilson > > > wrote: > > > > > > > > Quoting Daniel Vetter (2021-01-13 14:06:04) > > > > >

[Intel-gfx] ✓ Fi.CI.BAT: success for series starting with [CI,1/5] drm/i915/gt: Drop atomic for engine->fw_active tracking

2021-01-15 Thread Patchwork
== Series Details == Series: series starting with [CI,1/5] drm/i915/gt: Drop atomic for engine->fw_active tracking URL : https://patchwork.freedesktop.org/series/85918/ State : success == Summary == CI Bug Log - changes from CI_DRM_9624 -> Patchwork_19374

[Intel-gfx] [PATCH v8 3/3] drm/i915/tgl: Add Clear Color support for TGL Render Decompression

2021-01-15 Thread Imre Deak
From: Radhakrishna Sripada Render Decompression is supported with Y-Tiled main surface. The CCS is linear and has 4 bits of data for each main surface cache line pair, a ratio of 1:256. Additional Clear Color information is passed from the user-space through an offset in the GEM BO. Add a new

[Intel-gfx] [PATCH v8 2/3] drm/i915/gem: Add a helper to read data from a GEM object page

2021-01-15 Thread Imre Deak
Add a simple helper to read data with the CPU from the page of a GEM object. Do the read either via a kmap if the object has struct pages or an iomap otherwise. This is needed by the next patch, reading a u64 value from the object (w/o requiring the obj to be mapped to the GPU). Suggested by

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for series starting with [CI,1/5] drm/i915/gt: Drop atomic for engine->fw_active tracking

2021-01-15 Thread Patchwork
== Series Details == Series: series starting with [CI,1/5] drm/i915/gt: Drop atomic for engine->fw_active tracking URL : https://patchwork.freedesktop.org/series/85918/ State : warning == Summary == $ dim sparse --fast origin/drm-tip Sparse version: v0.6.2 Fast mode used, each commit won't

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [CI,1/5] drm/i915/gt: Drop atomic for engine->fw_active tracking

2021-01-15 Thread Patchwork
== Series Details == Series: series starting with [CI,1/5] drm/i915/gt: Drop atomic for engine->fw_active tracking URL : https://patchwork.freedesktop.org/series/85918/ State : warning == Summary == $ dim checkpatch origin/drm-tip 1b415ca0d734 drm/i915/gt: Drop atomic for engine->fw_active

[Intel-gfx] ✓ Fi.CI.BAT: success for HDCP misc fixes

2021-01-15 Thread Patchwork
== Series Details == Series: HDCP misc fixes URL : https://patchwork.freedesktop.org/series/85917/ State : success == Summary == CI Bug Log - changes from CI_DRM_9624 -> Patchwork_19373 Summary --- **SUCCESS** No regressions

[Intel-gfx] ✓ Fi.CI.IGT: success for rebased refactor of intel_display (rev2)

2021-01-15 Thread Patchwork
== Series Details == Series: rebased refactor of intel_display (rev2) URL : https://patchwork.freedesktop.org/series/85867/ State : success == Summary == CI Bug Log - changes from CI_DRM_9622_full -> Patchwork_19367_full Summary ---

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/gem: Use GEM context tracking for i915_gem_objects (rev2)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/gem: Use GEM context tracking for i915_gem_objects (rev2) URL : https://patchwork.freedesktop.org/series/85907/ State : success == Summary == CI Bug Log - changes from CI_DRM_9624 -> Patchwork_19372

Re: [Intel-gfx] [PATCH] dma-buf: Add debug option

2021-01-15 Thread Christian König
Am 15.01.21 um 17:47 schrieb Daniel Vetter: We have too many people abusing the struct page they can get at but really shouldn't in importers. Aside from that the backing page might simply not exist (for dynamic p2p mappings) looking at it and using it e.g. for mmap can also wreak the page

Re: [Intel-gfx] [PATCH v7 1/3] drm/framebuffer: Format modifier for Intel Gen 12 render compression with Clear Color

2021-01-15 Thread Chery, Nanley G
> -Original Message- > From: Imre Deak > Sent: Thursday, January 14, 2021 12:13 PM > To: intel-gfx@lists.freedesktop.org > Cc: Sripada, Radhakrishna ; Ville Syrjala > ; Pandiyan, Dhinakaran > ; Kondapally, Kalyan > ; Rafael Antognolli > ; Chery, Nanley G > ; Daniel Vetter ; Nikula, >

Re: [Intel-gfx] [PATCH 00/29] [Set 15] Finally rid W=1 warnings from GPU!

2021-01-15 Thread Zack Rusin
> On Jan 15, 2021, at 13:15, Lee Jones wrote: > > This set is part of a larger effort attempting to clean-up W=1 > kernel builds, which are currently overwhelmingly riddled with > niggly little warnings. > > Last set! All clean after this for; Arm, Arm64, PPC, MIPS and x86. Thanks! For all

[Intel-gfx] [PATCH 24/29] drm/i915/gvt/handlers: Remove kernel-doc formatting from non-kernel-doc header

2021-01-15 Thread Lee Jones
Fixes the following W=1 kernel build warning(s): drivers/gpu/drm/i915/gvt/handlers.c:1666: warning: Function parameter or member 'vgpu' not described in 'bxt_ppat_low_write' drivers/gpu/drm/i915/gvt/handlers.c:1666: warning: Function parameter or member 'offset' not described in

[Intel-gfx] [PATCH 00/29] [Set 15] Finally rid W=1 warnings from GPU!

2021-01-15 Thread Lee Jones
This set is part of a larger effort attempting to clean-up W=1 kernel builds, which are currently overwhelmingly riddled with niggly little warnings. Last set! All clean after this for; Arm, Arm64, PPC, MIPS and x86. Lee Jones (29): drm/vmwgfx/vmwgfx_cmdbuf: Fix misnaming of 'headers' should

[Intel-gfx] [PATCH i-g-t] i915/gem_userptr_blits: Check set-domain on userptr for validation

2021-01-15 Thread Chris Wilson
We use set-domain on userptr to validate that get-user-pages is applicable to the address range. Let's perform a very basic test to exercise this combination. Signed-off-by: Chris Wilson --- tests/i915/gem_userptr_blits.c | 26 ++ 1 file changed, 26 insertions(+) diff

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/dp: fix spelling contraction "couldnt" -> "couldn't"

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/dp: fix spelling contraction "couldnt" -> "couldn't" URL : https://patchwork.freedesktop.org/series/85901/ State : failure == Summary == CI Bug Log - changes from CI_DRM_9621_full -> Patchwork_19366_full

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/debugfs : PM_REQ and PM_RES registers (rev3)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/debugfs : PM_REQ and PM_RES registers (rev3) URL : https://patchwork.freedesktop.org/series/85437/ State : success == Summary == CI Bug Log - changes from CI_DRM_9623 -> Patchwork_19371 Summary ---

Re: [Intel-gfx] [PATCH] drm/i915/userptr: detect un-GUP-able pages early

2021-01-15 Thread Chris Wilson
Quoting Chris Wilson (2021-01-15 16:56:42) > Quoting Jinoh Kang (2021-01-15 16:23:31) > > If GUP-ineligible pages are passed to a GEM userptr object, -EFAULT is > > returned only when the object is actually bound. > > > > The xf86-video-intel userspace driver cannot differentiate this > >

Re: [Intel-gfx] [PATCH v7 2/3] drm/i915/gem: Add a helper to read data from a GEM object page

2021-01-15 Thread Imre Deak
On Thu, Jan 14, 2021 at 09:23:51PM +, Chris Wilson wrote: > Quoting Imre Deak (2021-01-14 20:13:13) > > Add a simple helper to read data with the CPU from the page of a GEM > > object. Do the read either via a kmap if the object has struct pages > > or an iomap otherwise. This is needed by the

Re: [Intel-gfx] [PATCH] drm/i915/userptr: detect un-GUP-able pages early

2021-01-15 Thread Chris Wilson
Quoting Jinoh Kang (2021-01-15 16:23:31) > If GUP-ineligible pages are passed to a GEM userptr object, -EFAULT is > returned only when the object is actually bound. > > The xf86-video-intel userspace driver cannot differentiate this > condition, and marks the GPU as wedged. The idea was to call

[Intel-gfx] [PATCH] dma-buf: Add debug option

2021-01-15 Thread Daniel Vetter
We have too many people abusing the struct page they can get at but really shouldn't in importers. Aside from that the backing page might simply not exist (for dynamic p2p mappings) looking at it and using it e.g. for mmap can also wreak the page handling of the exporter completely. Importers

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/display: Set error value before returning in intel_hdcp*_disable() (rev2)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/display: Set error value before returning in intel_hdcp*_disable() (rev2) URL : https://patchwork.freedesktop.org/series/85895/ State : success == Summary == CI Bug Log - changes from CI_DRM_9623 -> Patchwork_19370

[Intel-gfx] [PATCH] drm/i915/userptr: detect un-GUP-able pages early

2021-01-15 Thread Jinoh Kang
If GUP-ineligible pages are passed to a GEM userptr object, -EFAULT is returned only when the object is actually bound. The xf86-video-intel userspace driver cannot differentiate this condition, and marks the GPU as wedged. This not only disables graphics acceleration but may also cripple other

Re: [Intel-gfx] [PATCH] dma-buf: Add debug option

2021-01-15 Thread Christian König
Am 15.01.21 um 16:52 schrieb Daniel Vetter: We have too many people abusing the struct page they can get at but really shouldn't in importers. Aside from that the backing page might simply not exist (for dynamic p2p mappings) looking at it and using it e.g. for mmap can also wreak the page

Re: [Intel-gfx] [BUG] on reboot: bisected to: drm/i915: Shut down displays gracefully on reboot

2021-01-15 Thread Steven Rostedt
On Fri, 15 Jan 2021 09:50:25 +0200 Jani Nikula wrote: > >> fe0f1e3bfdfeb53e18f1206aea4f40b9bd1f291c > >> ("drm/i915: Shut down displays gracefully on reboot") > >> > >> Which makes sense, as it happens on shutdown. > > Please try this pull, heading to -rc4, which cointains "drm/i915: >

Re: [Intel-gfx] [BUG] on reboot: bisected to: drm/i915: Shut down displays gracefully on reboot

2021-01-15 Thread Steven Rostedt
On Thu, 14 Jan 2021 20:15:45 -0800 Linus Torvalds wrote: > On Thu, Jan 14, 2021 at 2:01 PM Steven Rostedt wrote: > > > > Thanks, I take it, it will be going into mainline soon. > > Just got merged - it might be a good idea to verify that your problem is > solved. > Yep, I just tested

Re: [Intel-gfx] [PATCH v2] drm/i915: support two CSC module on gen11 and later

2021-01-15 Thread Ville Syrjälä
On Fri, Jan 15, 2021 at 01:46:04PM +0800, Lee Shawn C wrote: > There are two CSC on pipeline on gen11 and later platform. > User space application is allowed to enable CTM and RGB > to YCbCr coversion at the same time now. > > v2: check csc capability in {}_color_check function. > > Cc: Ville

Re: [Intel-gfx] [PATCH] dma-buf: Add debug option

2021-01-15 Thread Chris Wilson
Quoting Daniel Vetter (2021-01-15 15:52:26) > +static void mangle_sg_table(struct sg_table *sg_table) > +{ > +#ifdef CONFIG_DMABUF_DEBUG > + int i; > + struct scatterlist *sg; > + > + if (!sg_table) if (!IS_ENABLED(CONFIG_DMABUF_DEBUG) || IS_ERR_OR_NULL(sg_table)) > +

Re: [Intel-gfx] [Linaro-mm-sig] [PATCH] drm-buf: Add debug option

2021-01-15 Thread Christian König
Am 15.01.21 um 14:02 schrieb Daniel Vetter: We have too many people abusing the struct page they can get at but really shouldn't in importers. Aside from that the backing page might simply not exist (for dynamic p2p mappings) looking at it and using it e.g. for mmap can also wreak the page

Re: [Intel-gfx] linux-next: build warnings after merge of the drm-misc tree

2021-01-15 Thread Nirmoy
Hi Stephen, On 1/15/21 2:23 AM, Stephen Rothwell wrote: Hi all, After merging the drm-misc tree, today's linux-next build (x86_64 allmodconfig) produced this warning: drivers/gpu/drm/amd/amdgpu/amdgpu_display.c: In function 'amdgpu_display_user_framebuffer_create':

Re: [Intel-gfx] [PATCH] drm/i915/gem: Mark private i915_gem_stolen functions as static

2021-01-15 Thread Rodrigo Vivi
On Fri, Jan 15, 2021 at 02:26:23PM +, Chris Wilson wrote: > drivers/gpu/drm/i915/gem/i915_gem_stolen.c:624:5: error: no previous > prototype for ‘__i915_gem_object_create_stolen’ [-Werror=missing-prototypes] > int __i915_gem_object_create_stolen(struct intel_memory_region *mem, >

[Intel-gfx] [PATCH] dma-buf: Add debug option

2021-01-15 Thread Daniel Vetter
We have too many people abusing the struct page they can get at but really shouldn't in importers. Aside from that the backing page might simply not exist (for dynamic p2p mappings) looking at it and using it e.g. for mmap can also wreak the page handling of the exporter completely. Importers

[Intel-gfx] ✗ Fi.CI.IGT: failure for drm/i915/display: Set error value before returning in intel_hdcp*_disable()

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/display: Set error value before returning in intel_hdcp*_disable() URL : https://patchwork.freedesktop.org/series/85895/ State : failure == Summary == CI Bug Log - changes from CI_DRM_9620_full -> Patchwork_19365_full

[Intel-gfx] [PATCH] dma-buf: Add debug option

2021-01-15 Thread Daniel Vetter
We have too many people abusing the struct page they can get at but really shouldn't in importers. Aside from that the backing page might simply not exist (for dynamic p2p mappings) looking at it and using it e.g. for mmap can also wreak the page handling of the exporter completely. Importers

Re: [Intel-gfx] [PATCH] drm-buf: Add debug option

2021-01-15 Thread kernel test robot
Hi Daniel, I love your patch! Perhaps something to improve: [auto build test WARNING on next-20210115] [also build test WARNING on v5.11-rc3] [cannot apply to tegra-drm/drm/tegra/for-next linus/master v5.11-rc3 v5.11-rc2 v5.11-rc1] [If your patch is applied to the wrong git tree, kindly drop us

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/dp_helper: tweak kerneldoc to address warning

2021-01-15 Thread Patchwork
== Series Details == Series: drm/dp_helper: tweak kerneldoc to address warning URL : https://patchwork.freedesktop.org/series/85906/ State : success == Summary == CI Bug Log - changes from CI_DRM_9622 -> Patchwork_19369 Summary ---

Re: [Intel-gfx] [PATCH 1/2] drm/i915/hdcp: Fix WARN_ON(data->k > INTEL_NUM_PIPES)

2021-01-15 Thread Jani Nikula
On Fri, 15 Jan 2021, Anshuman Gupta wrote: > Initialize no. of streams transmitted on a port to zero > such that intel_hdcp_required_content_stream() can > prepared the content stream after subsequemt attmept to > enable hdcp after a HDCP failure. > > Cc: Ramalingam C > Signed-off-by: Anshuman

Re: [Intel-gfx] [PATCH v7 1/5] drm/i915: Pass port to intel_panel_bl_funcs.get()

2021-01-15 Thread Jani Nikula
On Fri, 15 Jan 2021, Jani Nikula wrote: > On Thu, 14 Jan 2021, Lyude Paul wrote: >> In the next commit where we split PWM related backlight functions from >> higher-level backlight functions, we'll want to be able to retrieve the >> backlight level for the current display panel from the >>

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915: add back static declaration

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915: add back static declaration URL : https://patchwork.freedesktop.org/series/85902/ State : success == Summary == CI Bug Log - changes from CI_DRM_9622 -> Patchwork_19368 Summary --- **SUCCESS**

Re: [Intel-gfx] [PATCH] drm/i915: add back static declaration

2021-01-15 Thread Chris Wilson
Quoting Matthew Auld (2021-01-15 10:13:29) > warning: symbol '__i915_gem_object_create_stolen' was not declared. Should it > be static? > warning: symbol '_i915_gem_object_stolen_init' was not declared. Should it be > static? > > Signed-off-by: Matthew Auld Reviewed-by: Chris Wilson -Chris

[Intel-gfx] [PATCH] drm/i915/gem: Mark private i915_gem_stolen functions as static

2021-01-15 Thread Chris Wilson
drivers/gpu/drm/i915/gem/i915_gem_stolen.c:624:5: error: no previous prototype for ‘__i915_gem_object_create_stolen’ [-Werror=missing-prototypes] int __i915_gem_object_create_stolen(struct intel_memory_region *mem, ^~~

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for drm/i915: add back static declaration

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915: add back static declaration URL : https://patchwork.freedesktop.org/series/85902/ State : warning == Summary == $ dim sparse --fast origin/drm-tip Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately.

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for drm/i915: add back static declaration

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915: add back static declaration URL : https://patchwork.freedesktop.org/series/85902/ State : warning == Summary == $ dim checkpatch origin/drm-tip fb6fb58c7198 drm/i915: add back static declaration -:6: WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped

[Intel-gfx] [CI 4/5] drm/i915/gt: Reduce engine runtime stats from seqlock to a latch

2021-01-15 Thread Chris Wilson
Since we can compute the elapsed time to add to the total, during the PMU sample we only need to have a consistent view of the (start, total, active) tuple to be able to locally determine the runtime. That can be arrange by a pair of memory bariiers and carefully sequencing of the writes and

[Intel-gfx] [CI 3/5] drm/i915/gt: Convert stats.active to plain unsigned int

2021-01-15 Thread Chris Wilson
As context-in/out is now always serialised, we do not have to worry about concurrent enabling/disable of the busy-stats and can reduce the atomic_t active to a plain unsigned int, and the seqlock to a seqcount. Signed-off-by: Chris Wilson Reviewed-by: Andi Shyti ---

[Intel-gfx] [CI 5/5] drm/i915/gt: Reduce GT runtime stats from seqlock to a latch

2021-01-15 Thread Chris Wilson
Since we can compute the elapsed time to add to the total, during the PMU sample we only need to have a consistent view of the (start, total, active) tuple to be able to locally determine the runtime. That can be arrange by a pair of memory bariiers and carefully sequencing of the writes and

[Intel-gfx] [CI 1/5] drm/i915/gt: Drop atomic for engine->fw_active tracking

2021-01-15 Thread Chris Wilson
Since schedule-in/out is now entirely serialised by the tasklet bitlock, we do not need to worry about concurrent in/out operations and so reduce the atomic operations to plain instructions. Signed-off-by: Chris Wilson Reviewed-by: Andi Shyti --- drivers/gpu/drm/i915/gt/intel_engine_cs.c

[Intel-gfx] [CI 2/5] drm/i915/gt: Extract busy-stats for ring-scheduler

2021-01-15 Thread Chris Wilson
Lift the busy-stats context-in/out implementation out of intel_lrc, so that we can reuse it for other scheduler implementations. Signed-off-by: Chris Wilson Reviewed-by: Andi Shyti --- drivers/gpu/drm/i915/gt/intel_engine_stats.h | 49 +++

[Intel-gfx] ✓ Fi.CI.BAT: success for rebased refactor of intel_display (rev2)

2021-01-15 Thread Patchwork
== Series Details == Series: rebased refactor of intel_display (rev2) URL : https://patchwork.freedesktop.org/series/85867/ State : success == Summary == CI Bug Log - changes from CI_DRM_9622 -> Patchwork_19367 Summary ---

Re: [Intel-gfx] [PATCH] drm/i915/display: Set error value before returning in intel_hdcp*_disable()

2021-01-15 Thread Gupta, Anshuman
> -Original Message- > From: Gupta, Anshuman > Sent: Friday, January 15, 2021 5:39 PM > To: 'Jani Nikula' ; Chris Wilson wilson.co.uk>; intel-gfx@lists.freedesktop.org > Cc: Chris Wilson ; C, Ramalingam > ; Shankar, Uma ; > Sean Paul > Subject: RE: [PATCH] drm/i915/display: Set error

[Intel-gfx] [PATCH 2/2] drm/i915/hdcp: Fix error: uninitialized symbol 'ret'.

2021-01-15 Thread Anshuman Gupta
Move (num_hdcp_streams > 0) condition to stream_encryption() code block, where it actually belongs. Cc: Ramalingam C Signed-off-by: Anshuman Gupta --- drivers/gpu/drm/i915/display/intel_hdcp.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git

[Intel-gfx] [PATCH 1/2] drm/i915/hdcp: Fix WARN_ON(data->k > INTEL_NUM_PIPES)

2021-01-15 Thread Anshuman Gupta
Initialize no. of streams transmitted on a port to zero such that intel_hdcp_required_content_stream() can prepared the content stream after subsequemt attmept to enable hdcp after a HDCP failure. Cc: Ramalingam C Signed-off-by: Anshuman Gupta --- drivers/gpu/drm/i915/display/intel_hdcp.c | 2

[Intel-gfx] [PATCH 0/2] HDCP misc fixes

2021-01-15 Thread Anshuman Gupta
HDCP misc fixes. Anshuman Gupta (2): drm/i915/hdcp: Fix WARN_ON(data->k > INTEL_NUM_PIPES) drm/i915/hdcp: Fix error: uninitialized symbol 'ret'. drivers/gpu/drm/i915/display/intel_hdcp.c | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) -- 2.26.2

[Intel-gfx] ✗ Fi.CI.SPARSE: warning for rebased refactor of intel_display (rev2)

2021-01-15 Thread Patchwork
== Series Details == Series: rebased refactor of intel_display (rev2) URL : https://patchwork.freedesktop.org/series/85867/ State : warning == Summary == $ dim sparse --fast origin/drm-tip Sparse version: v0.6.2 Fast mode used, each commit won't be checked separately. -

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for rebased refactor of intel_display (rev2)

2021-01-15 Thread Patchwork
== Series Details == Series: rebased refactor of intel_display (rev2) URL : https://patchwork.freedesktop.org/series/85867/ State : warning == Summary == $ dim checkpatch origin/drm-tip 0142c754f498 drm/i915: refactor some crtc code out of intel display. (v2) -:32: WARNING:FILE_PATH_CHANGES:

Re: [Intel-gfx] [PATCH v7 1/5] drm/i915: Pass port to intel_panel_bl_funcs.get()

2021-01-15 Thread Jani Nikula
On Thu, 14 Jan 2021, Lyude Paul wrote: > In the next commit where we split PWM related backlight functions from > higher-level backlight functions, we'll want to be able to retrieve the > backlight level for the current display panel from the > intel_panel_bl_funcs->setup() function using

Re: [Intel-gfx] [PATCH v7 2/5] drm/i915: Keep track of pwm-related backlight hooks separately

2021-01-15 Thread Jani Nikula
On Thu, 14 Jan 2021, Lyude Paul wrote: > Currently, every different type of backlight hook that i915 supports is > pretty straight forward - you have a backlight, probably through PWM > (but maybe DPCD), with a single set of platform-specific hooks that are > used for controlling it. > > HDR

Re: [Intel-gfx] [Linaro-mm-sig] [PATCH] drm-buf: Add debug option

2021-01-15 Thread Christian König
Am 15.01.21 um 14:22 schrieb Daniel Vetter: On Fri, Jan 15, 2021 at 2:09 PM Christian König wrote: Am 15.01.21 um 14:02 schrieb Daniel Vetter: have too many people abusing the struct page they can get at but really shouldn't in importers. Aside from that the backing page might simply not

Re: [Intel-gfx] [Linaro-mm-sig] [PATCH] drm-buf: Add debug option

2021-01-15 Thread Daniel Vetter
On Fri, Jan 15, 2021 at 2:09 PM Christian König wrote: > > Am 15.01.21 um 14:02 schrieb Daniel Vetter: > > have too many people abusing the struct page they can get at but > > really shouldn't in importers. Aside from that the backing page might > > simply not exist (for dynamic p2p mappings)

Re: [Intel-gfx] [PATCH v4] drm/i915/debugfs : PM_REQ and PM_RES registers

2021-01-15 Thread Chris Wilson
Quoting Saichandana S (2021-01-15 13:01:10) > +static int i915_pm_req_res_info(struct seq_file *m, void *unused) > +{ > + struct drm_i915_private *i915 = node_to_i915(m->private); > + struct intel_csr *csr = >csr; > + u32 DC9_status; > + > + if (!HAS_CSR(i915)) > +

[Intel-gfx] [PATCH v2] drm/i915/gem: Use GEM context tracking for i915_gem_objects

2021-01-15 Thread Chris Wilson
Rather than take an indirect jump to the drm midlayer (that introduces a use-after-free in reading the ctx->file backpointer) to find all the vma on objects associated with the ctx->file, walk the LUT table stored in the context that tracks all the objects in use with the context. The improper

[Intel-gfx] [PATCH] drm-buf: Add debug option

2021-01-15 Thread Daniel Vetter
We have too many people abusing the struct page they can get at but really shouldn't in importers. Aside from that the backing page might simply not exist (for dynamic p2p mappings) looking at it and using it e.g. for mmap can also wreak the page handling of the exporter completely. Importers

[Intel-gfx] [PATCH v4] drm/i915/debugfs : PM_REQ and PM_RES registers

2021-01-15 Thread Saichandana S
PM_REQ register provides the value of the last PM request, response from PCU to PM_DBG_{REQ,RSP} . This debugfs provides DC9 Ready status but will also be used by DC9 IGT test . It will also print the useful debug information from Display Engine to PCU mailbox register. B.Spec : 49501, 49502 V2:

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: support two CSC module on gen11 and later (rev2)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915: support two CSC module on gen11 and later (rev2) URL : https://patchwork.freedesktop.org/series/85847/ State : success == Summary == CI Bug Log - changes from CI_DRM_9617_full -> Patchwork_19364_full

[Intel-gfx] [PATCH v2] drm/i915/display: Set return value before returning in intel_hdcp*_disable()

2021-01-15 Thread Chris Wilson
drivers/gpu/drm/i915/display/intel_hdcp.c:886 _intel_hdcp_disable() error: uninitialized symbol 'ret'. drivers/gpu/drm/i915/display/intel_hdcp.c:1951 _intel_hdcp2_disable() error: uninitialized symbol 'ret'. For MST, it is expected to always return early before disabling the stream, and this is

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/dp: fix spelling contraction "couldnt" -> "couldn't"

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/dp: fix spelling contraction "couldnt" -> "couldn't" URL : https://patchwork.freedesktop.org/series/85901/ State : success == Summary == CI Bug Log - changes from CI_DRM_9621 -> Patchwork_19366 Summary

Re: [Intel-gfx] [PATCH] drm/i915/display: Set error value before returning in intel_hdcp*_disable()

2021-01-15 Thread Gupta, Anshuman
> -Original Message- > From: Jani Nikula > Sent: Friday, January 15, 2021 4:05 PM > To: Chris Wilson ; intel-gfx@lists.freedesktop.org > Cc: Chris Wilson ; C, Ramalingam > ; Shankar, Uma ; > Gupta, Anshuman ; Sean Paul > > Subject: Re: [PATCH] drm/i915/display: Set error value before

[Intel-gfx] [PATCH] drm/i915/gem: Use GEM context tracking for i915_gem_objects

2021-01-15 Thread Chris Wilson
Rather than take an indirect jump to the drm midlayer (that introduces a use-after-free in reading the ctx->file backpointer) to find all the vma on objects associated with the ctx->file, walk the LUT table stored in the context that tracks all the objects in use with the context. Signed-off-by:

[Intel-gfx] [PATCH] drm/dp_helper: tweak kerneldoc to address warning

2021-01-15 Thread Lukas Bulwahn
warning: Function parameter or member 'color_spc' not described Tweak the kerneldoc for drm_dp_downstream_rgb_to_ycbcr_conversion(). Fixes: 07c9b8634fb6 ("drm/dp_helper: Add helpers to configure PCONs RGB-YCbCr Conversion") Signed-off-by: Lukas Bulwahn --- applies cleanly on next-2021011

Re: [Intel-gfx] [PATCH v2 06/11] drm/i915: Move the async_flip bit setup into the .async_flip() hook

2021-01-15 Thread Karthik B S
On 1/11/2021 10:07 PM, Ville Syrjala wrote: From: Ville Syrjälä Set up the async flip PLANE_CTL bit directly in the .async_flip() hook. Neither .update_plane() nor .disable_plane() ever need to set this so having it done by skl_plane_ctl_crtc() is rather pointless. Cc: Karthik B S Cc:

Re: [Intel-gfx] [PATCH v2 05/11] drm/i915: Add plane vfuncs to enable/disable flip_done interrupt

2021-01-15 Thread Karthik B S
On 1/11/2021 10:07 PM, Ville Syrjala wrote: From: Ville Syrjälä Prepare for more platforms with async flip support by turning the flip_done interrupt enable/disable into plane vfuncs. Cc: Karthik B S Cc: Vandita Kulkarni Signed-off-by: Ville Syrjälä Looks good to me. Reviewed-by:

Re: [Intel-gfx] [PATCH v4] drm/i915: Try to guess PCH type even without ISA bridge

2021-01-15 Thread Joonas Lahtinen
Quoting Jani Nikula (2021-01-15 12:50:54) > On Thu, 14 Jan 2021, Zhenyu Wang wrote: > > On 2021.01.14 08:58:19 +0800, Xiong Zhang wrote: > >> From: Zhenyu Wang > >> > >> Some vmm like hyperv and crosvm don't supply any ISA bridge to their guest, > >> when igd passthrough is equipped on these

[Intel-gfx] ✓ Fi.CI.BAT: success for drm/i915/display: Set error value before returning in intel_hdcp*_disable()

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915/display: Set error value before returning in intel_hdcp*_disable() URL : https://patchwork.freedesktop.org/series/85895/ State : success == Summary == CI Bug Log - changes from CI_DRM_9620 -> Patchwork_19365

Re: [Intel-gfx] [PATCH v4] drm/i915: Try to guess PCH type even without ISA bridge

2021-01-15 Thread Jani Nikula
On Thu, 14 Jan 2021, Zhenyu Wang wrote: > On 2021.01.14 08:58:19 +0800, Xiong Zhang wrote: >> From: Zhenyu Wang >> >> Some vmm like hyperv and crosvm don't supply any ISA bridge to their guest, >> when igd passthrough is equipped on these vmm, guest i915 display may >> couldn't work as guest

Re: [Intel-gfx] [PATCH] drm/i915/display: Set error value before returning in intel_hdcp*_disable()

2021-01-15 Thread Jani Nikula
On Fri, 15 Jan 2021, Chris Wilson wrote: > drivers/gpu/drm/i915/display/intel_hdcp.c:886 _intel_hdcp_disable() error: > uninitialized symbol 'ret'. > drivers/gpu/drm/i915/display/intel_hdcp.c:1951 _intel_hdcp2_disable() error: > uninitialized symbol 'ret'. > > Given the comment that this error

Re: [Intel-gfx] [PATCH v2 04/11] drm/i915: Generalize the async flip capability check

2021-01-15 Thread Karthik B S
On 1/11/2021 10:07 PM, Ville Syrjala wrote: From: Ville Syrjälä Only assign the plane->async_flip() vfunc when the plane supports async flips. For now we keep this artificially limited to the primary plane since thats the only thing the legacy page flip uapi can target and there is no async

Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the QSES check for HDCP 1.4 over MST

2021-01-15 Thread Gupta, Anshuman
> -Original Message- > From: Sean Paul > Sent: Thursday, January 14, 2021 12:04 AM > To: Daniel Vetter > Cc: Gupta, Anshuman ; David Airlie > ; intel-gfx@lists.freedesktop.org; Sean Paul > ; dri-de...@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH] drm/i915/hdcp: Disable the

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Add support for Intel's eDP backlight controls (rev10)

2021-01-15 Thread Patchwork
== Series Details == Series: drm/i915: Add support for Intel's eDP backlight controls (rev10) URL : https://patchwork.freedesktop.org/series/81702/ State : success == Summary == CI Bug Log - changes from CI_DRM_9612_full -> Patchwork_19363_full

  1   2   >