On Thu, Mar 17, 2016 at 04:42:12PM +0200, Imre Deak wrote:
> Atm, in case failure injection forces an error a subsequent error
> messages will make automated tests (CI) report this event as a breakage,
> even though the event is expected. To fix this print the error message
> with debug log level in this case.
>
> While at it print the error message for any init failure for
> consistency.
>
> CC: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Signed-off-by: Imre Deak <imre.deak@xxxxxxxxx>
> ---
> drivers/gpu/drm/i915/i915_dma.c | 14 +++++++++++---
> 1 file changed, 11 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c
> index 68592b0..d107871 100644
> --- a/drivers/gpu/drm/i915/i915_dma.c
> +++ b/drivers/gpu/drm/i915/i915_dma.c
> @@ -66,6 +66,14 @@ bool __i915_inject_load_failure(const char *func, int line)
> return false;
> }
>
> +#define i915_load_error(fmt, ...) do { \
> + if (i915.inject_load_failure && \
> + i915_load_fail_count == i915.inject_load_failure) \
> + DRM_DEBUG_DRIVER(fmt, ##__VA_ARGS__); \
> + else \
> + DRM_ERROR(fmt, ##__VA_ARGS__); \
> +} while(0)
> +
> static int i915_getparam(struct drm_device *dev, void *data,
> struct drm_file *file_priv)
> {
> @@ -1332,10 +1340,8 @@ int i915_driver_load(struct drm_device *dev, unsigned long flags)
> }
>
> ret = i915_load_modeset_init(dev);
> - if (ret < 0) {
> - DRM_ERROR("failed to init modeset\n");
> + if (ret < 0)
> goto out_cleanup_vblank;
> - }
>
> i915_driver_register(dev_priv);
>
> @@ -1357,6 +1363,8 @@ out_runtime_pm_put:
> out_free_priv:
> kfree(dev_priv);
>
> + i915_load_error("device initialization failed (%d)\n", ret);
Please put a representative sample output into the changelog.
I think we should include a request to file bug with full drm.debug=0xf.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
