linux-kernel - Re: [ 74/75] Revert "drm/i915: enable irqs earlier when resuming"

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1889932.BuDOlJCg10@photon>
Date:	Wed, 27 Mar 2013 19:45:42 +0400
From:	Ilya Tumaykin <itumaykin@...il.com>
To:	Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:	linux-kernel@...r.kernel.org, stable@...r.kernel.org,
	Chris Wilson <chris@...is-wilson.co.uk>,
	Mika Kuoppala <mika.kuoppala@...el.com>,
	Daniel Vetter <daniel.vetter@...ll.ch>
Subject: Re: [ 74/75] Revert "drm/i915: enable irqs earlier when resuming"

On 18.03.13 14:07:37 Greg Kroah-Hartman wrote:
> 3.8-stable review patch.  If anyone has any objections, please let me know.

Hello, Greg.

The original "drm/i915: enable irqs earlier when resuming" commit was 
introduced in order to fix bug appeared in 3.8 kernels. Reverting this commit 
you also bring back that bug. Bugzilla link:
https://bugzilla.kernel.org/show_bug.cgi?id=54691#c17

Sorry for late reply, I wasn't sure if this is simple revert or another 
mechanism was introduced by devs to avoid mentioned bug and that fix became 
unnecessary. Please undo this revert.

> ------------------
> 
> This reverts commit 31f14f4219d2a74b7a6d86c7798f49141b5eccbe which was
> commit 15239099d7a7a9ecdc1ccb5b187ae4cda5488ff9 upstream.
> 
> It caused problems in the 3.8-stable series, but 3.9-rc is just fine.
> 
> Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> Cc: Chris Wilson <chris@...is-wilson.co.uk>
> Cc: Mika Kuoppala <mika.kuoppala@...el.com>
> Cc: Ilya Tumaykin <itumaykin@...il.com>
> Cc: Daniel Vetter <daniel.vetter@...ll.ch>
> 
> ---
>  drivers/gpu/drm/i915/i915_drv.c |   13 +------------
>  1 file changed, 1 insertion(+), 12 deletions(-)
> 
> --- a/drivers/gpu/drm/i915/i915_drv.c
> +++ b/drivers/gpu/drm/i915/i915_drv.c
> @@ -486,7 +486,6 @@ static int i915_drm_freeze(struct drm_de
>  		intel_modeset_disable(dev);
> 
>  		drm_irq_uninstall(dev);
> -		dev_priv->enable_hotplug_processing = false;
>  	}
> 
>  	i915_save_state(dev);
> @@ -563,19 +562,9 @@ static int __i915_drm_thaw(struct drm_de
>  		error = i915_gem_init_hw(dev);
>  		mutex_unlock(&dev->struct_mutex);
> 
> -		/* We need working interrupts for modeset enabling ... */
> -		drm_irq_install(dev);
> -
>  		intel_modeset_init_hw(dev);
>  		intel_modeset_setup_hw_state(dev, false);
> -
> -		/*
> -		 * ... but also need to make sure that hotplug processing
> -		 * doesn't cause havoc. Like in the driver load code we don't
> -		 * bother with the tiny race here where we might loose hotplug
> -		 * notifications.
> -		 * */
> -		dev_priv->enable_hotplug_processing = true;
> +		drm_irq_install(dev);
>  	}
> 
>  	intel_opregion_init(dev);
-- 
Best regards.
Tumaykin Ilya.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/