* [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
@ 2019-03-27 9:20 Mika Kuoppala
2019-03-27 9:25 ` Chris Wilson
2019-03-27 13:24 ` ✗ Fi.CI.BAT: failure for drm/i915/selftests: Fix an IS_ERR() vs NULL check (rev3) Patchwork
0 siblings, 2 replies; 14+ messages in thread
From: Mika Kuoppala @ 2019-03-27 9:20 UTC (permalink / raw)
To: intel-gfx; +Cc: dan.carpenter
The live_context() function returns error pointers. It never returns
NULL.
Fixes: 52c0fdb25c7c ("drm/i915: Replace global breadcrumbs with per-context interrupt tracking")
Signed-off-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
---
drivers/gpu/drm/i915/selftests/i915_request.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/i915/selftests/i915_request.c b/drivers/gpu/drm/i915/selftests/i915_request.c
index e6ffe2240126..7c5154dba6d6 100644
--- a/drivers/gpu/drm/i915/selftests/i915_request.c
+++ b/drivers/gpu/drm/i915/selftests/i915_request.c
@@ -1146,11 +1146,15 @@ static int live_breadcrumbs_smoketest(void *arg)
mutex_lock(&i915->drm.struct_mutex);
for (n = 0; n < t[0].ncontexts; n++) {
- t[0].contexts[n] = live_context(i915, file);
- if (!t[0].contexts[n]) {
+ struct i915_gem_context *ctx;
+
+ ctx = live_context(i915, file);
+ if (IS_ERR(ctx)) {
ret = -ENOMEM;
goto out_contexts;
}
+
+ t[0].contexts[n] = ctx;
}
ret = igt_live_test_begin(&live, i915, __func__, "");
--
2.17.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
2019-03-27 9:20 [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check Mika Kuoppala
@ 2019-03-27 9:25 ` Chris Wilson
2019-03-27 9:32 ` Mika Kuoppala
2019-03-27 13:24 ` ✗ Fi.CI.BAT: failure for drm/i915/selftests: Fix an IS_ERR() vs NULL check (rev3) Patchwork
1 sibling, 1 reply; 14+ messages in thread
From: Chris Wilson @ 2019-03-27 9:25 UTC (permalink / raw)
To: Mika Kuoppala, intel-gfx; +Cc: dan.carpenter
Quoting Mika Kuoppala (2019-03-27 09:20:05)
> The live_context() function returns error pointers. It never returns
> NULL.
>
> Fixes: 52c0fdb25c7c ("drm/i915: Replace global breadcrumbs with per-context interrupt tracking")
> Signed-off-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> ---
> drivers/gpu/drm/i915/selftests/i915_request.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/selftests/i915_request.c b/drivers/gpu/drm/i915/selftests/i915_request.c
> index e6ffe2240126..7c5154dba6d6 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_request.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_request.c
> @@ -1146,11 +1146,15 @@ static int live_breadcrumbs_smoketest(void *arg)
>
> mutex_lock(&i915->drm.struct_mutex);
> for (n = 0; n < t[0].ncontexts; n++) {
> - t[0].contexts[n] = live_context(i915, file);
> - if (!t[0].contexts[n]) {
> + struct i915_gem_context *ctx;
> +
> + ctx = live_context(i915, file);
> + if (IS_ERR(ctx)) {
> ret = -ENOMEM;
ret = PTR_ERR(ctx);
Might be interesting if it's not ENOMEM.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread* [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
2019-03-27 9:25 ` Chris Wilson
@ 2019-03-27 9:32 ` Mika Kuoppala
2019-03-27 9:52 ` Chris Wilson
0 siblings, 1 reply; 14+ messages in thread
From: Mika Kuoppala @ 2019-03-27 9:32 UTC (permalink / raw)
To: intel-gfx; +Cc: dan.carpenter
The live_context() function returns error pointers. It never returns
NULL.
v2: don't clamp the err (Chris)
Fixes: 52c0fdb25c7c ("drm/i915: Replace global breadcrumbs with per-context interrupt tracking")
Signed-off-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
---
drivers/gpu/drm/i915/selftests/i915_request.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/selftests/i915_request.c b/drivers/gpu/drm/i915/selftests/i915_request.c
index e6ffe2240126..b0409353c5a4 100644
--- a/drivers/gpu/drm/i915/selftests/i915_request.c
+++ b/drivers/gpu/drm/i915/selftests/i915_request.c
@@ -1146,11 +1146,15 @@ static int live_breadcrumbs_smoketest(void *arg)
mutex_lock(&i915->drm.struct_mutex);
for (n = 0; n < t[0].ncontexts; n++) {
- t[0].contexts[n] = live_context(i915, file);
- if (!t[0].contexts[n]) {
- ret = -ENOMEM;
+ struct i915_gem_context *ctx;
+
+ ctx = live_context(i915, file);
+ if (IS_ERR(ctx)) {
+ ret = PTR_ERR(ctx);
goto out_contexts;
}
+
+ t[0].contexts[n] = ctx;
}
ret = igt_live_test_begin(&live, i915, __func__, "");
--
2.17.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 14+ messages in thread
* ✗ Fi.CI.BAT: failure for drm/i915/selftests: Fix an IS_ERR() vs NULL check (rev3)
2019-03-27 9:20 [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check Mika Kuoppala
2019-03-27 9:25 ` Chris Wilson
@ 2019-03-27 13:24 ` Patchwork
2019-03-28 11:56 ` Mika Kuoppala
1 sibling, 1 reply; 14+ messages in thread
From: Patchwork @ 2019-03-27 13:24 UTC (permalink / raw)
To: Mika Kuoppala; +Cc: intel-gfx
== Series Details ==
Series: drm/i915/selftests: Fix an IS_ERR() vs NULL check (rev3)
URL : https://patchwork.freedesktop.org/series/58557/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_5824 -> Patchwork_12612
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_12612 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_12612, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/58557/revisions/3/mbox/
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_12612:
### IGT changes ###
#### Possible regressions ####
* igt@i915_selftest@live_uncore:
- fi-kbl-8809g: PASS -> DMESG-FAIL
Known issues
------------
Here are the changes found in Patchwork_12612 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_basic@readonly-bsd2:
- fi-pnv-d510: NOTRUN -> SKIP [fdo#109271] +76
* igt@i915_selftest@live_contexts:
- fi-skl-gvtdvm: PASS -> DMESG-FAIL [fdo#110235 ]
* igt@kms_busy@basic-flip-c:
- fi-pnv-d510: NOTRUN -> SKIP [fdo#109271] / [fdo#109278]
* igt@kms_frontbuffer_tracking@basic:
- fi-byt-clapper: PASS -> FAIL [fdo#103167]
* igt@prime_vgem@basic-fence-flip:
- fi-gdg-551: PASS -> FAIL [fdo#103182]
#### Possible fixes ####
* igt@kms_busy@basic-flip-a:
- fi-gdg-551: FAIL [fdo#103182] -> PASS
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103182]: https://bugs.freedesktop.org/show_bug.cgi?id=103182
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#110235 ]: https://bugs.freedesktop.org/show_bug.cgi?id=110235
Participating hosts (42 -> 37)
------------------------------
Additional (1): fi-pnv-d510
Missing (6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-bdw-samus
Build changes
-------------
* Linux: CI_DRM_5824 -> Patchwork_12612
CI_DRM_5824: d6a239f8bd502261971e5a2fb0ec71ae0b30b298 @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4907: 7c8f2616fa0fd3ddb16e050c5b7ea9ce707abbe4 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_12612: 545ca65d2548ce9a6d6c965abb1ce061d2c7eb18 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
545ca65d2548 drm/i915/selftests: Fix an IS_ERR() vs NULL check
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12612/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: ✗ Fi.CI.BAT: failure for drm/i915/selftests: Fix an IS_ERR() vs NULL check (rev3)
2019-03-27 13:24 ` ✗ Fi.CI.BAT: failure for drm/i915/selftests: Fix an IS_ERR() vs NULL check (rev3) Patchwork
@ 2019-03-28 11:56 ` Mika Kuoppala
0 siblings, 0 replies; 14+ messages in thread
From: Mika Kuoppala @ 2019-03-28 11:56 UTC (permalink / raw)
To: Patchwork; +Cc: intel-gfx
Patchwork <patchwork@emeril.freedesktop.org> writes:
> == Series Details ==
>
> Series: drm/i915/selftests: Fix an IS_ERR() vs NULL check (rev3)
Ok, never ever send a patch with identical subject line.
> URL : https://patchwork.freedesktop.org/series/58557/
> State : failure
>
> == Summary ==
>
> CI Bug Log - changes from CI_DRM_5824 -> Patchwork_12612
> ====================================================
>
> Summary
> -------
>
> **FAILURE**
>
> Serious unknown changes coming with Patchwork_12612 absolutely need to be
> verified manually.
>
> If you think the reported changes have nothing to do with the changes
> introduced in Patchwork_12612, please notify your bug team to allow them
> to document this new failure mode, which will reduce false positives in CI.
>
> External URL: https://patchwork.freedesktop.org/api/1.0/series/58557/revisions/3/mbox/
>
> Possible new issues
> -------------------
>
> Here are the unknown changes that may have been introduced in Patchwork_12612:
>
> ### IGT changes ###
>
> #### Possible regressions ####
>
> * igt@i915_selftest@live_uncore:
> - fi-kbl-8809g: PASS -> DMESG-FAIL
<3> [320.015575] bcs0:RING_MI_MODE=200, fw_domains 0x2 still up after 100ms!
-Mika
>
>
> Known issues
> ------------
>
> Here are the changes found in Patchwork_12612 that come from known issues:
>
> ### IGT changes ###
>
> #### Issues hit ####
>
> * igt@gem_exec_basic@readonly-bsd2:
> - fi-pnv-d510: NOTRUN -> SKIP [fdo#109271] +76
>
> * igt@i915_selftest@live_contexts:
> - fi-skl-gvtdvm: PASS -> DMESG-FAIL [fdo#110235 ]
>
> * igt@kms_busy@basic-flip-c:
> - fi-pnv-d510: NOTRUN -> SKIP [fdo#109271] / [fdo#109278]
>
> * igt@kms_frontbuffer_tracking@basic:
> - fi-byt-clapper: PASS -> FAIL [fdo#103167]
>
> * igt@prime_vgem@basic-fence-flip:
> - fi-gdg-551: PASS -> FAIL [fdo#103182]
>
>
> #### Possible fixes ####
>
> * igt@kms_busy@basic-flip-a:
> - fi-gdg-551: FAIL [fdo#103182] -> PASS
>
>
> [fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
> [fdo#103182]: https://bugs.freedesktop.org/show_bug.cgi?id=103182
> [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
> [fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
> [fdo#110235 ]: https://bugs.freedesktop.org/show_bug.cgi?id=110235
>
>
> Participating hosts (42 -> 37)
> ------------------------------
>
> Additional (1): fi-pnv-d510
> Missing (6): fi-kbl-soraka fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-bdw-samus
>
>
> Build changes
> -------------
>
> * Linux: CI_DRM_5824 -> Patchwork_12612
>
> CI_DRM_5824: d6a239f8bd502261971e5a2fb0ec71ae0b30b298 @ git://anongit.freedesktop.org/gfx-ci/linux
> IGT_4907: 7c8f2616fa0fd3ddb16e050c5b7ea9ce707abbe4 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
> Patchwork_12612: 545ca65d2548ce9a6d6c965abb1ce061d2c7eb18 @ git://anongit.freedesktop.org/gfx-ci/linux
>
>
> == Linux commits ==
>
> 545ca65d2548 drm/i915/selftests: Fix an IS_ERR() vs NULL check
>
> == Logs ==
>
> For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_12612/
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
@ 2019-03-26 5:08 ` Dan Carpenter
0 siblings, 0 replies; 14+ messages in thread
From: Dan Carpenter @ 2019-03-26 5:08 UTC (permalink / raw)
To: Jani Nikula, Chris Wilson
Cc: kernel-janitors, David Airlie, intel-gfx, dri-devel, Thomas Zimmermann
The live_context() function returns error pointers. It never returns
NULL.
Fixes: 9c1477e83e62 ("drm/i915/selftests: Exercise adding requests to a full GGTT")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
---
drivers/gpu/drm/i915/selftests/i915_gem_evict.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_evict.c b/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
index 9a9451846b33..89766688e420 100644
--- a/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
+++ b/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
@@ -455,7 +455,7 @@ static int igt_evict_contexts(void *arg)
struct i915_gem_context *ctx;
ctx = live_context(i915, file);
- if (!ctx)
+ if (IS_ERR(ctx))
break;
/* We will need some GGTT space for the rq's context */
--
2.17.1
^ permalink raw reply related [flat|nested] 14+ messages in thread* [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
@ 2019-03-26 5:08 ` Dan Carpenter
0 siblings, 0 replies; 14+ messages in thread
From: Dan Carpenter @ 2019-03-26 5:08 UTC (permalink / raw)
To: Jani Nikula, Chris Wilson
Cc: kernel-janitors, David Airlie, intel-gfx, dri-devel, Thomas Zimmermann
The live_context() function returns error pointers. It never returns
NULL.
Fixes: 9c1477e83e62 ("drm/i915/selftests: Exercise adding requests to a full GGTT")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
---
drivers/gpu/drm/i915/selftests/i915_gem_evict.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_evict.c b/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
index 9a9451846b33..89766688e420 100644
--- a/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
+++ b/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
@@ -455,7 +455,7 @@ static int igt_evict_contexts(void *arg)
struct i915_gem_context *ctx;
ctx = live_context(i915, file);
- if (!ctx)
+ if (IS_ERR(ctx))
break;
/* We will need some GGTT space for the rq's context */
--
2.17.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 14+ messages in thread* Re: [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
2019-03-26 5:08 ` Dan Carpenter
@ 2019-03-26 9:30 ` Mika Kuoppala
-1 siblings, 0 replies; 14+ messages in thread
From: Mika Kuoppala @ 2019-03-26 9:30 UTC (permalink / raw)
To: Dan Carpenter, Jani Nikula, Chris Wilson
Cc: David Airlie, kernel-janitors, intel-gfx, dri-devel, Thomas Zimmermann
Dan Carpenter <dan.carpenter@oracle.com> writes:
> The live_context() function returns error pointers. It never returns
> NULL.
>
> Fixes: 9c1477e83e62 ("drm/i915/selftests: Exercise adding requests to a full GGTT")
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
i915_request.c has another :)
-Mika
> ---
> drivers/gpu/drm/i915/selftests/i915_gem_evict.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_evict.c b/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
> index 9a9451846b33..89766688e420 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
> @@ -455,7 +455,7 @@ static int igt_evict_contexts(void *arg)
> struct i915_gem_context *ctx;
>
> ctx = live_context(i915, file);
> - if (!ctx)
> + if (IS_ERR(ctx))
> break;
>
> /* We will need some GGTT space for the rq's context */
> --
> 2.17.1
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
@ 2019-03-26 9:30 ` Mika Kuoppala
0 siblings, 0 replies; 14+ messages in thread
From: Mika Kuoppala @ 2019-03-26 9:30 UTC (permalink / raw)
To: Dan Carpenter, Jani Nikula, Chris Wilson
Cc: David Airlie, kernel-janitors, intel-gfx, dri-devel, Thomas Zimmermann
Dan Carpenter <dan.carpenter@oracle.com> writes:
> The live_context() function returns error pointers. It never returns
> NULL.
>
> Fixes: 9c1477e83e62 ("drm/i915/selftests: Exercise adding requests to a full GGTT")
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
i915_request.c has another :)
-Mika
> ---
> drivers/gpu/drm/i915/selftests/i915_gem_evict.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/selftests/i915_gem_evict.c b/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
> index 9a9451846b33..89766688e420 100644
> --- a/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
> +++ b/drivers/gpu/drm/i915/selftests/i915_gem_evict.c
> @@ -455,7 +455,7 @@ static int igt_evict_contexts(void *arg)
> struct i915_gem_context *ctx;
>
> ctx = live_context(i915, file);
> - if (!ctx)
> + if (IS_ERR(ctx))
> break;
>
> /* We will need some GGTT space for the rq's context */
> --
> 2.17.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
2019-03-26 9:30 ` Mika Kuoppala
@ 2019-03-26 14:53 ` Chris Wilson
-1 siblings, 0 replies; 14+ messages in thread
From: Chris Wilson @ 2019-03-26 14:53 UTC (permalink / raw)
To: Dan Carpenter, Jani Nikula, Mika Kuoppala
Cc: Tvrtko Ursulin, David Airlie, kernel-janitors, intel-gfx,
Matthew Auld, dri-devel, Thomas Zimmermann, Rodrigo Vivi
Quoting Mika Kuoppala (2019-03-26 09:30:57)
> Dan Carpenter <dan.carpenter@oracle.com> writes:
>
> > The live_context() function returns error pointers. It never returns
> > NULL.
> >
> > Fixes: 9c1477e83e62 ("drm/i915/selftests: Exercise adding requests to a full GGTT")
> > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
>
> Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
>
> i915_request.c has another :)
Having found it, you could have typed up the patch :-p
Thanks for the fix, pushed.
-Chris
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
@ 2019-03-26 14:53 ` Chris Wilson
0 siblings, 0 replies; 14+ messages in thread
From: Chris Wilson @ 2019-03-26 14:53 UTC (permalink / raw)
To: Dan Carpenter, Jani Nikula, Mika Kuoppala
Cc: Tvrtko Ursulin, David Airlie, kernel-janitors, intel-gfx,
Matthew Auld, dri-devel, Thomas Zimmermann, Rodrigo Vivi
Quoting Mika Kuoppala (2019-03-26 09:30:57)
> Dan Carpenter <dan.carpenter@oracle.com> writes:
>
> > The live_context() function returns error pointers. It never returns
> > NULL.
> >
> > Fixes: 9c1477e83e62 ("drm/i915/selftests: Exercise adding requests to a full GGTT")
> > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
>
> Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
>
> i915_request.c has another :)
Having found it, you could have typed up the patch :-p
Thanks for the fix, pushed.
-Chris
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
2019-03-26 14:53 ` Chris Wilson
@ 2019-03-26 15:35 ` Dan Carpenter
-1 siblings, 0 replies; 14+ messages in thread
From: Dan Carpenter @ 2019-03-26 15:35 UTC (permalink / raw)
To: Chris Wilson
Cc: kernel-janitors, David Airlie, intel-gfx, Thomas Zimmermann, dri-devel
On Tue, Mar 26, 2019 at 02:53:49PM +0000, Chris Wilson wrote:
> Quoting Mika Kuoppala (2019-03-26 09:30:57)
> > Dan Carpenter <dan.carpenter@oracle.com> writes:
> >
> > > The live_context() function returns error pointers. It never returns
> > > NULL.
> > >
> > > Fixes: 9c1477e83e62 ("drm/i915/selftests: Exercise adding requests to a full GGTT")
> > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> >
> > Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> >
> > i915_request.c has another :)
>
> Having found it, you could have typed up the patch :-p
>
Smatch didn't warn about that one because we didn't dereference it right
away... I should make a check which complains about the condition
instead of the dereference.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 14+ messages in thread* Re: [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check
@ 2019-03-26 15:35 ` Dan Carpenter
0 siblings, 0 replies; 14+ messages in thread
From: Dan Carpenter @ 2019-03-26 15:35 UTC (permalink / raw)
To: Chris Wilson
Cc: kernel-janitors, David Airlie, intel-gfx, Thomas Zimmermann, dri-devel
On Tue, Mar 26, 2019 at 02:53:49PM +0000, Chris Wilson wrote:
> Quoting Mika Kuoppala (2019-03-26 09:30:57)
> > Dan Carpenter <dan.carpenter@oracle.com> writes:
> >
> > > The live_context() function returns error pointers. It never returns
> > > NULL.
> > >
> > > Fixes: 9c1477e83e62 ("drm/i915/selftests: Exercise adding requests to a full GGTT")
> > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
> >
> > Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com>
> >
> > i915_request.c has another :)
>
> Having found it, you could have typed up the patch :-p
>
Smatch didn't warn about that one because we didn't dereference it right
away... I should make a check which complains about the condition
instead of the dereference.
regards,
dan carpenter
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2019-03-28 11:56 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-27 9:20 [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check Mika Kuoppala
2019-03-27 9:25 ` Chris Wilson
2019-03-27 9:32 ` Mika Kuoppala
2019-03-27 9:52 ` Chris Wilson
2019-03-27 13:24 ` ✗ Fi.CI.BAT: failure for drm/i915/selftests: Fix an IS_ERR() vs NULL check (rev3) Patchwork
2019-03-28 11:56 ` Mika Kuoppala
-- strict thread matches above, loose matches on Subject: below --
2019-03-26 5:08 [PATCH] drm/i915/selftests: Fix an IS_ERR() vs NULL check Dan Carpenter
2019-03-26 5:08 ` Dan Carpenter
2019-03-26 9:30 ` Mika Kuoppala
2019-03-26 9:30 ` Mika Kuoppala
2019-03-26 14:53 ` Chris Wilson
2019-03-26 14:53 ` Chris Wilson
2019-03-26 15:35 ` Dan Carpenter
2019-03-26 15:35 ` Dan Carpenter
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.