Re: [Intel-gfx] [Mesa-dev] gitlab.fd.o financial situation and impact on services

2020-02-29 Thread Marek Olšák
For Mesa, we could run CI only when Marge pushes, so that it's a strictly pre-merge CI. Marek On Sat., Feb. 29, 2020, 17:20 Nicolas Dufresne, wrote: > Le samedi 29 février 2020 à 15:54 -0600, Jason Ekstrand a écrit : > > On Sat, Feb 29, 2020 at 3:47 PM Timur Kristóf > wrote: > > > On Sat,

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/dmc: Use firmware v2.06 for TGL (rev3)

2020-02-29 Thread Patchwork
== Series Details == Series: drm/i915/dmc: Use firmware v2.06 for TGL (rev3) URL : https://patchwork.freedesktop.org/series/74048/ State : success == Summary == CI Bug Log - changes from CI_DRM_8026_full -> Patchwork_16760_full Summary

Re: [Intel-gfx] [Mesa-dev] gitlab.fd.o financial situation and impact on services

2020-02-29 Thread Jason Ekstrand
On Sat, Feb 29, 2020 at 3:47 PM Timur Kristóf wrote: > > On Sat, 2020-02-29 at 14:46 -0500, Nicolas Dufresne wrote: > > > > > > 1. I think we should completely disable running the CI on MRs which > > > are > > > marked WIP. Speaking from personal experience, I usually make a lot > > > of > > >

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/3] drm/i915: add i915_ioc32.h for compat (rev2)

2020-02-29 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915: add i915_ioc32.h for compat (rev2) URL : https://patchwork.freedesktop.org/series/74033/ State : success == Summary == CI Bug Log - changes from CI_DRM_8024_full -> Patchwork_16758_full

Re: [Intel-gfx] [PATCH i-g-t 3/5] i915: Exercise preemption timeout controls in sysfs

2020-02-29 Thread Chris Wilson
Quoting Chris Wilson (2020-02-29 18:34:49) > Quoting Andi Shyti (2020-02-29 12:45:27) > > > > > > > + char buf[512]; > > > > > > > + int len; > > > > > > > + > > > > > > > + lseek(engines, 0, SEEK_SET); > > > > > > > + while ((len = syscall(SYS_getdents64, engines, buf, > > > > >

Re: [Intel-gfx] [PATCH i-g-t 3/5] i915: Exercise preemption timeout controls in sysfs

2020-02-29 Thread Chris Wilson
Quoting Andi Shyti (2020-02-29 12:45:27) > > > > > > + char buf[512]; > > > > > > + int len; > > > > > > + > > > > > > + lseek(engines, 0, SEEK_SET); > > > > > > + while ((len = syscall(SYS_getdents64, engines, buf, > > > > > > sizeof(buf))) > 0) { > > > > > > + void

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/2] drm/i915: Handle all MCR ranges (rev2)

2020-02-29 Thread Patchwork
== Series Details == Series: series starting with [1/2] drm/i915: Handle all MCR ranges (rev2) URL : https://patchwork.freedesktop.org/series/74047/ State : success == Summary == CI Bug Log - changes from CI_DRM_8024_full -> Patchwork_16756_full

[Intel-gfx] ✓ Fi.CI.IGT: success for Re-org uC debugfs files and move them under GT

2020-02-29 Thread Patchwork
== Series Details == Series: Re-org uC debugfs files and move them under GT URL : https://patchwork.freedesktop.org/series/74051/ State : success == Summary == CI Bug Log - changes from CI_DRM_8024_full -> Patchwork_16755_full Summary

Re: [Intel-gfx] [Mesa-dev] gitlab.fd.o financial situation and impact on services

2020-02-29 Thread Jason Ekstrand
On Fri, Feb 28, 2020 at 11:00 AM Rob Clark wrote: > > On Fri, Feb 28, 2020 at 3:43 AM Michel Dänzer wrote: > > > > On 2020-02-28 10:28 a.m., Erik Faye-Lund wrote: > > > > > > We could also do stuff like reducing the amount of tests we run on each > > > commit, and punt some testing to a

[Intel-gfx] ✓ Fi.CI.IGT: success for series starting with [1/4] drm/i915/perf: Reintroduce wait on OA configuration completion (rev2)

2020-02-29 Thread Patchwork
== Series Details == Series: series starting with [1/4] drm/i915/perf: Reintroduce wait on OA configuration completion (rev2) URL : https://patchwork.freedesktop.org/series/74049/ State : success == Summary == CI Bug Log - changes from CI_DRM_8024_full -> Patchwork_16753_full

[Intel-gfx] ✓ Fi.CI.BAT: success for bitops: Always inline sign extension helpers

2020-02-29 Thread Patchwork
== Series Details == Series: bitops: Always inline sign extension helpers URL : https://patchwork.freedesktop.org/series/74108/ State : success == Summary == CI Bug Log - changes from CI_DRM_8036 -> Patchwork_16774 Summary ---

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/huc: update TGL HuC to v7.0.12

2020-02-29 Thread Patchwork
== Series Details == Series: drm/i915/huc: update TGL HuC to v7.0.12 URL : https://patchwork.freedesktop.org/series/74046/ State : success == Summary == CI Bug Log - changes from CI_DRM_8023_full -> Patchwork_16750_full Summary ---

[Intel-gfx] ✗ Fi.CI.BAT: failure for series starting with [1/3] drm/i915/perf: remove generated code

2020-02-29 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915/perf: remove generated code URL : https://patchwork.freedesktop.org/series/74107/ State : failure == Summary == CI Bug Log - changes from CI_DRM_8035 -> Patchwork_16773

[Intel-gfx] [PATCH] bitops: Always inline sign extension helpers

2020-02-29 Thread Josh Poimboeuf
With CONFIG_CC_OPTIMIZE_FOR_SIZE, objtool reports: drivers/gpu/drm/i915/gem/i915_gem_execbuffer.o: warning: objtool: i915_gem_execbuffer2_ioctl()+0x5b7: call to gen8_canonical_addr() with UACCESS enabled This means i915_gem_execbuffer2_ioctl() is calling gen8_canonical_addr() from the

[Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for series starting with [1/3] drm/i915/perf: remove generated code

2020-02-29 Thread Patchwork
== Series Details == Series: series starting with [1/3] drm/i915/perf: remove generated code URL : https://patchwork.freedesktop.org/series/74107/ State : warning == Summary == $ dim checkpatch origin/drm-tip 882c81c4e09c drm/i915/perf: remove generated code -:24: WARNING:UNKNOWN_COMMIT_ID:

Re: [Intel-gfx] [PATCH i-g-t 3/5] i915: Exercise preemption timeout controls in sysfs

2020-02-29 Thread Andi Shyti
> > > > > + char buf[512]; > > > > > + int len; > > > > > + > > > > > + lseek(engines, 0, SEEK_SET); > > > > > + while ((len = syscall(SYS_getdents64, engines, buf, > > > > > sizeof(buf))) > 0) { > > > > > + void *ptr = buf; > > > > > + > > > > > + while

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Minimize uaccess exposure in i915_gem_execbuffer2_ioctl()

2020-02-29 Thread Patchwork
== Series Details == Series: drm/i915: Minimize uaccess exposure in i915_gem_execbuffer2_ioctl() URL : https://patchwork.freedesktop.org/series/74045/ State : success == Summary == CI Bug Log - changes from CI_DRM_8023_full -> Patchwork_16749_full

[Intel-gfx] [PATCH 2/3] drm/i915/perf: reintroduce wait on OA configuration completion

2020-02-29 Thread Lionel Landwerlin
We still need to wait for the initial OA configuration to happen before we enable OA report writes to the OA buffer. Signed-off-by: Lionel Landwerlin Fixes: 15d0ace1f876 ("drm/i915/perf: execute OA configuration from command stream") Reviewed-by: Chris Wilson ---

[Intel-gfx] [PATCH 3/3] drm/i915/perf: introduce global sseu pinning

2020-02-29 Thread Lionel Landwerlin
On Gen11 powergating half the execution units is a functional requirement when using the VME samplers. Not fullfilling this requirement can lead to hangs. This unfortunately plays fairly poorly with the NOA requirements. NOA requires a stable power configuration to maintain its configuration. As

[Intel-gfx] [PATCH 1/3] drm/i915/perf: remove generated code

2020-02-29 Thread Lionel Landwerlin
A little bit of history : Back when i915-perf was introduced (4.13), there was no way to dynamically add new OA configurations to i915. Only the generated configs baked in at build time were allowed. It quickly became obvious that we would need to allow applications to upload

[Intel-gfx] ✗ Fi.CI.IGT: failure for series starting with [v3,01/11] drm/i915/tgl: Implement Wa_1409804808

2020-02-29 Thread Patchwork
== Series Details == Series: series starting with [v3,01/11] drm/i915/tgl: Implement Wa_1409804808 URL : https://patchwork.freedesktop.org/series/74044/ State : failure == Summary == CI Bug Log - changes from CI_DRM_8023_full -> Patchwork_16748_full

Re: [Intel-gfx] [PATCH 03/51] drm: add managed resources tied to drm_device

2020-02-29 Thread Daniel Vetter
On Sat, Feb 29, 2020 at 12:17 PM Sam Ravnborg wrote: > > > > + * > > > > + * Based on drivers/base/devres.c > > > > + */ > > > > + > > > > +#include > > > > + > > > > +#include > > > > +#include > > > > +#include > > > > + > > > > +#include > > > > +#include > > > > > > It is good practice

Re: [Intel-gfx] [PATCH 03/51] drm: add managed resources tied to drm_device

2020-02-29 Thread Sam Ravnborg
Hi Daniel. > > > diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c > > > index 9fcd6ab3c154..3e5627d6eba6 100644 > > > --- a/drivers/gpu/drm/drm_drv.c > > > +++ b/drivers/gpu/drm/drm_drv.c > > > @@ -629,6 +629,9 @@ int drm_dev_init(struct drm_device *dev, > > > dev->dev =

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915/gt: Check engine-is-awake on reset later

2020-02-29 Thread Patchwork
== Series Details == Series: drm/i915/gt: Check engine-is-awake on reset later URL : https://patchwork.freedesktop.org/series/74041/ State : success == Summary == CI Bug Log - changes from CI_DRM_8023_full -> Patchwork_16746_full Summary

Re: [Intel-gfx] [PATCH 26/51] drm: Manage drm_mode_config_init with drmm_

2020-02-29 Thread Sam Ravnborg
Hi Daniel. > > > Also with drmm_ explicit drm_driver->release hooks are kinda not the > > > best option, so deprecate that hook to discourage future users. > > The ->release hooks has others uses until everything is moved over to > > drmm_, or so I think. So deprecation seems a lttle too soon. >

Re: [Intel-gfx] [PATCH i-g-t 3/5] i915: Exercise preemption timeout controls in sysfs

2020-02-29 Thread Chris Wilson
Quoting Andi Shyti (2020-02-28 23:51:24) > > > > +void dyn_sysfs_engines(int i915, int engines, const char *file, > > > > +void (*test)(int, int)) > > > > +{ > > > > + char buf[512]; > > > > + int len; > > > > + > > > > + lseek(engines, 0, SEEK_SET); > > > > +

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Lock gmbus/aux mutexes while changing cdclk

2020-02-29 Thread Patchwork
== Series Details == Series: drm/i915: Lock gmbus/aux mutexes while changing cdclk URL : https://patchwork.freedesktop.org/series/74039/ State : success == Summary == CI Bug Log - changes from CI_DRM_8023_full -> Patchwork_16745_full

Re: [Intel-gfx] [PATCH v18 4/8] drm/i915: Introduce more *_state_changed indicators

2020-02-29 Thread Lisovskiy, Stanislav
>> But this patch is not about that - it is about how we signal/determine >> that some change has to be written at commit stage. >>As you remember when we were discussed offline, I just wanted to have >> some expicit way to mark if some global state subsystem had changed, >>without having to do

[Intel-gfx] ✗ Fi.CI.IGT: failure for Adding YUV444 packed format support for skl+ (rev4)

2020-02-29 Thread Patchwork
== Series Details == Series: Adding YUV444 packed format support for skl+ (rev4) URL : https://patchwork.freedesktop.org/series/73020/ State : failure == Summary == CI Bug Log - changes from CI_DRM_8023_full -> Patchwork_16744_full Summary

[Intel-gfx] ✓ Fi.CI.IGT: success for drm/i915: Print HDCP version info for all connectors

2020-02-29 Thread Patchwork
== Series Details == Series: drm/i915: Print HDCP version info for all connectors URL : https://patchwork.freedesktop.org/series/74037/ State : success == Summary == CI Bug Log - changes from CI_DRM_8023_full -> Patchwork_16743_full