* [PATCH] drm/crc: Actually allow to change the crc source
@ 2019-08-21 20:38 Daniel Vetter
2019-08-21 21:03 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
` (3 more replies)
0 siblings, 4 replies; 7+ messages in thread
From: Daniel Vetter @ 2019-08-21 20:38 UTC (permalink / raw)
To: DRI Development
Cc: Tomeu Vizoso, Daniel Vetter, Intel Graphics Development,
Daniel Vetter, Emil Velikov
Oops.
Fixes: 9edbf1fa600a ("drm: Add API for capturing frame CRCs")
Cc: Tomeu Vizoso <tomeu.vizoso@collabora.com>
Cc: Emil Velikov <emil.velikov@collabora.com>
Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
---
drivers/gpu/drm/drm_debugfs_crc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_debugfs_crc.c b/drivers/gpu/drm/drm_debugfs_crc.c
index be1b7ba92ffe..6fe693ee15f9 100644
--- a/drivers/gpu/drm/drm_debugfs_crc.c
+++ b/drivers/gpu/drm/drm_debugfs_crc.c
@@ -369,7 +369,7 @@ void drm_debugfs_crtc_crc_add(struct drm_crtc *crtc)
crc_ent = debugfs_create_dir("crc", crtc->debugfs_entry);
- debugfs_create_file("control", S_IRUGO, crc_ent, crtc,
+ debugfs_create_file("control", S_IRUGO | S_IWUSR, crc_ent, crtc,
&drm_crtc_crc_control_fops);
debugfs_create_file("data", S_IRUGO, crc_ent, crtc,
&drm_crtc_crc_data_fops);
--
2.23.0.rc1
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply related [flat|nested] 7+ messages in thread
* ✗ Fi.CI.CHECKPATCH: warning for drm/crc: Actually allow to change the crc source
2019-08-21 20:38 [PATCH] drm/crc: Actually allow to change the crc source Daniel Vetter
@ 2019-08-21 21:03 ` Patchwork
2019-08-21 21:25 ` ✓ Fi.CI.BAT: success " Patchwork
` (2 subsequent siblings)
3 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2019-08-21 21:03 UTC (permalink / raw)
To: Daniel Vetter; +Cc: intel-gfx
== Series Details ==
Series: drm/crc: Actually allow to change the crc source
URL : https://patchwork.freedesktop.org/series/65572/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
3300dfdef52f drm/crc: Actually allow to change the crc source
-:23: WARNING:SYMBOLIC_PERMS: Symbolic permissions 'S_IRUGO | S_IWUSR' are not preferred. Consider using octal permissions '0644'.
#23: FILE: drivers/gpu/drm/drm_debugfs_crc.c:372:
+ debugfs_create_file("control", S_IRUGO | S_IWUSR, crc_ent, crtc,
-:26: WARNING:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author 'Daniel Vetter <daniel.vetter@ffwll.ch>'
total: 0 errors, 2 warnings, 0 checks, 8 lines checked
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
* ✓ Fi.CI.BAT: success for drm/crc: Actually allow to change the crc source
2019-08-21 20:38 [PATCH] drm/crc: Actually allow to change the crc source Daniel Vetter
2019-08-21 21:03 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
@ 2019-08-21 21:25 ` Patchwork
2019-08-22 12:20 ` [PATCH] " Ville Syrjälä
2019-08-22 12:44 ` ✓ Fi.CI.IGT: success for " Patchwork
3 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2019-08-21 21:25 UTC (permalink / raw)
To: Daniel Vetter; +Cc: intel-gfx
== Series Details ==
Series: drm/crc: Actually allow to change the crc source
URL : https://patchwork.freedesktop.org/series/65572/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6758 -> Patchwork_14131
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/
Known issues
------------
Here are the changes found in Patchwork_14131 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_basic@create-fd-close:
- fi-icl-u3: [PASS][1] -> [DMESG-WARN][2] ([fdo#107724]) +1 similar issue
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/fi-icl-u3/igt@gem_basic@create-fd-close.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/fi-icl-u3/igt@gem_basic@create-fd-close.html
#### Possible fixes ####
* igt@gem_exec_suspend@basic-s3:
- fi-blb-e6850: [INCOMPLETE][3] ([fdo#107718]) -> [PASS][4]
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/fi-blb-e6850/igt@gem_exec_suspend@basic-s3.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/fi-blb-e6850/igt@gem_exec_suspend@basic-s3.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
[fdo#107724]: https://bugs.freedesktop.org/show_bug.cgi?id=107724
[fdo#111045]: https://bugs.freedesktop.org/show_bug.cgi?id=111045
[fdo#111096]: https://bugs.freedesktop.org/show_bug.cgi?id=111096
Participating hosts (55 -> 47)
------------------------------
Additional (1): fi-pnv-d510
Missing (9): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-hsw-peppy fi-byt-squawks fi-bsw-cyan fi-icl-y fi-byt-clapper fi-bdw-samus
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_6758 -> Patchwork_14131
CI-20190529: 20190529
CI_DRM_6758: 8616440ca224c1fa1775b6b2d21bad7ddf66442a @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5146: 357dbe1869d88a2f08bcee4eebceff4ee9014424 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_14131: 3300dfdef52fef736629069d972d77680df763a3 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
3300dfdef52f drm/crc: Actually allow to change the crc source
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/crc: Actually allow to change the crc source
2019-08-21 20:38 [PATCH] drm/crc: Actually allow to change the crc source Daniel Vetter
2019-08-21 21:03 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2019-08-21 21:25 ` ✓ Fi.CI.BAT: success " Patchwork
@ 2019-08-22 12:20 ` Ville Syrjälä
2020-01-28 15:49 ` [Intel-gfx] " Daniel Vetter
2019-08-22 12:44 ` ✓ Fi.CI.IGT: success for " Patchwork
3 siblings, 1 reply; 7+ messages in thread
From: Ville Syrjälä @ 2019-08-22 12:20 UTC (permalink / raw)
To: Daniel Vetter
Cc: Daniel Vetter, Intel Graphics Development, Tomeu Vizoso,
DRI Development, Emil Velikov
On Wed, Aug 21, 2019 at 10:38:35PM +0200, Daniel Vetter wrote:
> Oops.
>
> Fixes: 9edbf1fa600a ("drm: Add API for capturing frame CRCs")
> Cc: Tomeu Vizoso <tomeu.vizoso@collabora.com>
> Cc: Emil Velikov <emil.velikov@collabora.com>
> Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org>
> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> ---
> drivers/gpu/drm/drm_debugfs_crc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_debugfs_crc.c b/drivers/gpu/drm/drm_debugfs_crc.c
> index be1b7ba92ffe..6fe693ee15f9 100644
> --- a/drivers/gpu/drm/drm_debugfs_crc.c
> +++ b/drivers/gpu/drm/drm_debugfs_crc.c
> @@ -369,7 +369,7 @@ void drm_debugfs_crtc_crc_add(struct drm_crtc *crtc)
>
> crc_ent = debugfs_create_dir("crc", crtc->debugfs_entry);
>
> - debugfs_create_file("control", S_IRUGO, crc_ent, crtc,
> + debugfs_create_file("control", S_IRUGO | S_IWUSR, crc_ent, crtc,
> &drm_crtc_crc_control_fops);
Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
But could we please go octal? My brain can't decode that alphabet soup.
> debugfs_create_file("data", S_IRUGO, crc_ent, crtc,
> &drm_crtc_crc_data_fops);
> --
> 2.23.0.rc1
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
--
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
* ✓ Fi.CI.IGT: success for drm/crc: Actually allow to change the crc source
2019-08-21 20:38 [PATCH] drm/crc: Actually allow to change the crc source Daniel Vetter
` (2 preceding siblings ...)
2019-08-22 12:20 ` [PATCH] " Ville Syrjälä
@ 2019-08-22 12:44 ` Patchwork
3 siblings, 0 replies; 7+ messages in thread
From: Patchwork @ 2019-08-22 12:44 UTC (permalink / raw)
To: Daniel Vetter; +Cc: intel-gfx
== Series Details ==
Series: drm/crc: Actually allow to change the crc source
URL : https://patchwork.freedesktop.org/series/65572/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6758_full -> Patchwork_14131_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Known issues
------------
Here are the changes found in Patchwork_14131_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_ctx_shared@exec-single-timeline-bsd:
- shard-iclb: [PASS][1] -> [SKIP][2] ([fdo#110841])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb6/igt@gem_ctx_shared@exec-single-timeline-bsd.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb4/igt@gem_ctx_shared@exec-single-timeline-bsd.html
* igt@gem_exec_balancer@smoke:
- shard-iclb: [PASS][3] -> [SKIP][4] ([fdo#110854])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb1/igt@gem_exec_balancer@smoke.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb7/igt@gem_exec_balancer@smoke.html
* igt@gem_exec_schedule@deep-bsd:
- shard-iclb: [PASS][5] -> [SKIP][6] ([fdo#111325]) +1 similar issue
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb3/igt@gem_exec_schedule@deep-bsd.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb2/igt@gem_exec_schedule@deep-bsd.html
* igt@gem_exec_schedule@fifo-bsd1:
- shard-iclb: [PASS][7] -> [SKIP][8] ([fdo#109276]) +14 similar issues
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb2/igt@gem_exec_schedule@fifo-bsd1.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb6/igt@gem_exec_schedule@fifo-bsd1.html
* igt@gem_tiled_swapping@non-threaded:
- shard-glk: [PASS][9] -> [DMESG-WARN][10] ([fdo#108686])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-glk3/igt@gem_tiled_swapping@non-threaded.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-glk1/igt@gem_tiled_swapping@non-threaded.html
* igt@gem_workarounds@suspend-resume-context:
- shard-skl: [PASS][11] -> [INCOMPLETE][12] ([fdo#104108])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-skl10/igt@gem_workarounds@suspend-resume-context.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-skl4/igt@gem_workarounds@suspend-resume-context.html
* igt@i915_pm_rps@min-max-config-loaded:
- shard-iclb: [PASS][13] -> [FAIL][14] ([fdo#111409])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb4/igt@i915_pm_rps@min-max-config-loaded.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb2/igt@i915_pm_rps@min-max-config-loaded.html
* igt@i915_suspend@fence-restore-tiled2untiled:
- shard-apl: [PASS][15] -> [DMESG-WARN][16] ([fdo#108566]) +6 similar issues
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-apl4/igt@i915_suspend@fence-restore-tiled2untiled.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-apl3/igt@i915_suspend@fence-restore-tiled2untiled.html
* igt@kms_cursor_legacy@pipe-c-single-move:
- shard-apl: [PASS][17] -> [INCOMPLETE][18] ([fdo#103927]) +1 similar issue
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-apl2/igt@kms_cursor_legacy@pipe-c-single-move.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-apl4/igt@kms_cursor_legacy@pipe-c-single-move.html
* igt@kms_frontbuffer_tracking@fbc-1p-pri-indfb-multidraw:
- shard-iclb: [PASS][19] -> [FAIL][20] ([fdo#103167]) +6 similar issues
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb1/igt@kms_frontbuffer_tracking@fbc-1p-pri-indfb-multidraw.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-1p-pri-indfb-multidraw.html
* igt@kms_psr2_su@frontbuffer:
- shard-iclb: [PASS][21] -> [SKIP][22] ([fdo#109642] / [fdo#111068])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb2/igt@kms_psr2_su@frontbuffer.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb6/igt@kms_psr2_su@frontbuffer.html
* igt@kms_psr@psr2_sprite_render:
- shard-iclb: [PASS][23] -> [SKIP][24] ([fdo#109441])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb2/igt@kms_psr@psr2_sprite_render.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb4/igt@kms_psr@psr2_sprite_render.html
* igt@kms_setmode@basic:
- shard-kbl: [PASS][25] -> [FAIL][26] ([fdo#99912])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-kbl7/igt@kms_setmode@basic.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-kbl6/igt@kms_setmode@basic.html
#### Possible fixes ####
* igt@gem_exec_schedule@preempt-queue-bsd1:
- shard-iclb: [SKIP][27] ([fdo#109276]) -> [PASS][28] +10 similar issues
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb6/igt@gem_exec_schedule@preempt-queue-bsd1.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb4/igt@gem_exec_schedule@preempt-queue-bsd1.html
* igt@gem_exec_schedule@wide-bsd:
- shard-iclb: [SKIP][29] ([fdo#111325]) -> [PASS][30] +6 similar issues
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb4/igt@gem_exec_schedule@wide-bsd.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb7/igt@gem_exec_schedule@wide-bsd.html
* igt@i915_suspend@sysfs-reader:
- shard-apl: [DMESG-WARN][31] ([fdo#108566]) -> [PASS][32] +3 similar issues
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-apl3/igt@i915_suspend@sysfs-reader.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-apl3/igt@i915_suspend@sysfs-reader.html
* igt@kms_flip@flip-vs-dpms-off-vs-modeset:
- shard-iclb: [INCOMPLETE][33] ([fdo#107713]) -> [PASS][34]
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb7/igt@kms_flip@flip-vs-dpms-off-vs-modeset.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb5/igt@kms_flip@flip-vs-dpms-off-vs-modeset.html
* igt@kms_flip@flip-vs-expired-vblank:
- shard-skl: [FAIL][35] ([fdo#105363]) -> [PASS][36]
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-skl9/igt@kms_flip@flip-vs-expired-vblank.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-skl3/igt@kms_flip@flip-vs-expired-vblank.html
* igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-pwrite:
- shard-iclb: [FAIL][37] ([fdo#103167]) -> [PASS][38] +4 similar issues
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-pwrite.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb4/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-shrfb-draw-pwrite.html
* igt@kms_psr@psr2_dpms:
- shard-iclb: [SKIP][39] ([fdo#109441]) -> [PASS][40]
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb3/igt@kms_psr@psr2_dpms.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb2/igt@kms_psr@psr2_dpms.html
* igt@kms_setmode@basic:
- shard-hsw: [FAIL][41] ([fdo#99912]) -> [PASS][42]
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-hsw5/igt@kms_setmode@basic.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-hsw7/igt@kms_setmode@basic.html
* igt@kms_vblank@pipe-a-wait-forked-hang:
- shard-snb: [INCOMPLETE][43] ([fdo#105411]) -> [PASS][44]
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-snb1/igt@kms_vblank@pipe-a-wait-forked-hang.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-snb1/igt@kms_vblank@pipe-a-wait-forked-hang.html
* igt@perf_pmu@rc6:
- shard-kbl: [SKIP][45] ([fdo#109271]) -> [PASS][46]
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-kbl1/igt@perf_pmu@rc6.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-kbl1/igt@perf_pmu@rc6.html
#### Warnings ####
* igt@kms_atomic_transition@5x-modeset-transitions:
- shard-snb: [SKIP][47] ([fdo#109271] / [fdo#109278]) -> [SKIP][48] ([fdo#109271])
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-snb7/igt@kms_atomic_transition@5x-modeset-transitions.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-snb6/igt@kms_atomic_transition@5x-modeset-transitions.html
* igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-blt:
- shard-skl: [FAIL][49] ([fdo#108040]) -> [FAIL][50] ([fdo#103167])
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-skl8/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-blt.html
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-skl5/igt@kms_frontbuffer_tracking@fbc-1p-offscren-pri-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-indfb-draw-pwrite:
- shard-iclb: [INCOMPLETE][51] ([fdo#107713]) -> [FAIL][52] ([fdo#103167])
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6758/shard-iclb7/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-indfb-draw-pwrite.html
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/shard-iclb2/igt@kms_frontbuffer_tracking@fbc-1p-primscrn-pri-indfb-draw-pwrite.html
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#104108]: https://bugs.freedesktop.org/show_bug.cgi?id=104108
[fdo#105363]: https://bugs.freedesktop.org/show_bug.cgi?id=105363
[fdo#105411]: https://bugs.freedesktop.org/show_bug.cgi?id=105411
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#108040]: https://bugs.freedesktop.org/show_bug.cgi?id=108040
[fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
[fdo#108686]: https://bugs.freedesktop.org/show_bug.cgi?id=108686
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109276]: https://bugs.freedesktop.org/show_bug.cgi?id=109276
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642
[fdo#110841]: https://bugs.freedesktop.org/show_bug.cgi?id=110841
[fdo#110854]: https://bugs.freedesktop.org/show_bug.cgi?id=110854
[fdo#111068]: https://bugs.freedesktop.org/show_bug.cgi?id=111068
[fdo#111325]: https://bugs.freedesktop.org/show_bug.cgi?id=111325
[fdo#111409]: https://bugs.freedesktop.org/show_bug.cgi?id=111409
[fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
Participating hosts (10 -> 10)
------------------------------
No changes in participating hosts
Build changes
-------------
* CI: CI-20190529 -> None
* Linux: CI_DRM_6758 -> Patchwork_14131
CI-20190529: 20190529
CI_DRM_6758: 8616440ca224c1fa1775b6b2d21bad7ddf66442a @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_5146: 357dbe1869d88a2f08bcee4eebceff4ee9014424 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_14131: 3300dfdef52fef736629069d972d77680df763a3 @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_14131/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] drm/crc: Actually allow to change the crc source
2019-08-22 12:20 ` [PATCH] " Ville Syrjälä
@ 2020-01-28 15:49 ` Daniel Vetter
0 siblings, 0 replies; 7+ messages in thread
From: Daniel Vetter @ 2020-01-28 15:49 UTC (permalink / raw)
To: Ville Syrjälä
Cc: Daniel Vetter, Intel Graphics Development, Tomeu Vizoso,
DRI Development, Emil Velikov
On Thu, Aug 22, 2019 at 2:20 PM Ville Syrjälä
<ville.syrjala@linux.intel.com> wrote:
>
> On Wed, Aug 21, 2019 at 10:38:35PM +0200, Daniel Vetter wrote:
> > Oops.
> >
> > Fixes: 9edbf1fa600a ("drm: Add API for capturing frame CRCs")
> > Cc: Tomeu Vizoso <tomeu.vizoso@collabora.com>
> > Cc: Emil Velikov <emil.velikov@collabora.com>
> > Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org>
> > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> > ---
> > drivers/gpu/drm/drm_debugfs_crc.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/drm_debugfs_crc.c b/drivers/gpu/drm/drm_debugfs_crc.c
> > index be1b7ba92ffe..6fe693ee15f9 100644
> > --- a/drivers/gpu/drm/drm_debugfs_crc.c
> > +++ b/drivers/gpu/drm/drm_debugfs_crc.c
> > @@ -369,7 +369,7 @@ void drm_debugfs_crtc_crc_add(struct drm_crtc *crtc)
> >
> > crc_ent = debugfs_create_dir("crc", crtc->debugfs_entry);
> >
> > - debugfs_create_file("control", S_IRUGO, crc_ent, crtc,
> > + debugfs_create_file("control", S_IRUGO | S_IWUSR, crc_ent, crtc,
> > &drm_crtc_crc_control_fops);
>
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Found this old patch of mine, finally applied it. Thanks for your review.
-Daniel
>
> But could we please go octal? My brain can't decode that alphabet soup.
>
> > debugfs_create_file("data", S_IRUGO, crc_ent, crtc,
> > &drm_crtc_crc_data_fops);
> > --
> > 2.23.0.rc1
> >
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
> --
> Ville Syrjälä
> Intel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [Intel-gfx] [PATCH] drm/crc: Actually allow to change the crc source
@ 2020-01-28 15:49 ` Daniel Vetter
0 siblings, 0 replies; 7+ messages in thread
From: Daniel Vetter @ 2020-01-28 15:49 UTC (permalink / raw)
To: Ville Syrjälä
Cc: Daniel Vetter, Intel Graphics Development, Tomeu Vizoso,
DRI Development, Emil Velikov
On Thu, Aug 22, 2019 at 2:20 PM Ville Syrjälä
<ville.syrjala@linux.intel.com> wrote:
>
> On Wed, Aug 21, 2019 at 10:38:35PM +0200, Daniel Vetter wrote:
> > Oops.
> >
> > Fixes: 9edbf1fa600a ("drm: Add API for capturing frame CRCs")
> > Cc: Tomeu Vizoso <tomeu.vizoso@collabora.com>
> > Cc: Emil Velikov <emil.velikov@collabora.com>
> > Cc: Benjamin Gaignard <benjamin.gaignard@linaro.org>
> > Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
> > ---
> > drivers/gpu/drm/drm_debugfs_crc.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/drm_debugfs_crc.c b/drivers/gpu/drm/drm_debugfs_crc.c
> > index be1b7ba92ffe..6fe693ee15f9 100644
> > --- a/drivers/gpu/drm/drm_debugfs_crc.c
> > +++ b/drivers/gpu/drm/drm_debugfs_crc.c
> > @@ -369,7 +369,7 @@ void drm_debugfs_crtc_crc_add(struct drm_crtc *crtc)
> >
> > crc_ent = debugfs_create_dir("crc", crtc->debugfs_entry);
> >
> > - debugfs_create_file("control", S_IRUGO, crc_ent, crtc,
> > + debugfs_create_file("control", S_IRUGO | S_IWUSR, crc_ent, crtc,
> > &drm_crtc_crc_control_fops);
>
> Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Found this old patch of mine, finally applied it. Thanks for your review.
-Daniel
>
> But could we please go octal? My brain can't decode that alphabet soup.
>
> > debugfs_create_file("data", S_IRUGO, crc_ent, crtc,
> > &drm_crtc_crc_data_fops);
> > --
> > 2.23.0.rc1
> >
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
>
> --
> Ville Syrjälä
> Intel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2020-01-28 15:50 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-21 20:38 [PATCH] drm/crc: Actually allow to change the crc source Daniel Vetter
2019-08-21 21:03 ` ✗ Fi.CI.CHECKPATCH: warning for " Patchwork
2019-08-21 21:25 ` ✓ Fi.CI.BAT: success " Patchwork
2019-08-22 12:20 ` [PATCH] " Ville Syrjälä
2020-01-28 15:49 ` Daniel Vetter
2020-01-28 15:49 ` [Intel-gfx] " Daniel Vetter
2019-08-22 12:44 ` ✓ Fi.CI.IGT: success for " Patchwork
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.