linux-kernel - RE: [ 67/89] drm/radeon: properly track the crtc not_enabled case evergreen_mc

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: <A3397C8B8B789E45844E7EC5DEAD89D02DC1B172@sausexdag04.amd.com>
Date:	Mon, 3 Dec 2012 23:40:37 +0000
From:	"Deucher, Alexander" <Alexander.Deucher@....com>
To:	Josh Boyer <jwboyer@...il.com>
CC:	Ben Hutchings <ben@...adent.org.uk>,
	Greg KH <gregkh@...uxfoundation.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"stable@...r.kernel.org" <stable@...r.kernel.org>,
	"akpm@...ux-foundation.org" <akpm@...ux-foundation.org>,
	"alan@...rguk.ukuu.org.uk" <alan@...rguk.ukuu.org.uk>
Subject: RE: [ 67/89] drm/radeon: properly track the crtc not_enabled case
 evergreen_mc_stop()

> -----Original Message-----
> From: Josh Boyer [mailto:jwboyer@...il.com]
> Sent: Monday, December 03, 2012 6:26 PM
> To: Deucher, Alexander
> Cc: Ben Hutchings; Greg KH; linux-kernel@...r.kernel.org;
> stable@...r.kernel.org; akpm@...ux-foundation.org;
> alan@...rguk.ukuu.org.uk
> Subject: Re: [ 67/89] drm/radeon: properly track the crtc not_enabled case
> evergreen_mc_stop()
> 
> On Mon, Dec 3, 2012 at 10:35 AM, Deucher, Alexander
> <Alexander.Deucher@....com> wrote:
> >> -----Original Message-----
> >> From: Josh Boyer [mailto:jwboyer@...il.com]
> >> Sent: Monday, December 03, 2012 10:25 AM
> >> To: Ben Hutchings; Greg KH
> >> Cc: linux-kernel@...r.kernel.org; stable@...r.kernel.org; akpm@...ux-
> >> foundation.org; alan@...rguk.ukuu.org.uk; Deucher, Alexander
> >> Subject: Re: [ 67/89] drm/radeon: properly track the crtc not_enabled
> case
> >> evergreen_mc_stop()
> >>
> >> On Mon, Dec 3, 2012 at 9:32 AM, Ben Hutchings <ben@...adent.org.uk>
> >> wrote:
> >> > 3.2-stable review patch.  If anyone has any objections, please let me
> know.
> >> >
> >> > ------------------
> >> >
> >> > From: Alex Deucher <alexander.deucher@....com>
> >> >
> >> > commit 804cc4a0ad3a896ca295f771a28c6eb36ced7903 upstream.
> >> >
> >> > The save struct is not initialized previously so explicitly
> >> > mark the crtcs as not used when they are not in use.
> >> >
> >> > Signed-off-by: Alex Deucher <alexander.deucher@....com>
> >> > Signed-off-by: Ben Hutchings <ben@...adent.org.uk>
> >>
> >> Hm.  If this is needed in 3.2, presumably it's needed in 3.6 as well.  I
> >> don't see it queued for 3.6.9, and the Cc: tag is there.
> >>
> >> Greg, Alex, was this just something that was missed, or am I wrong about
> >> it needing to go into 3.6?
> >
> > The original patches should go into 3.6 kernels as well:
> >
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commitdiff;h=4
> a15903db02026728d0cf2755c6fabae16b8db6a
> >
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commitdiff;h=6
> 2444b7462a2b98bc78d68736c03a7c4e66ba7e2
> >
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commitdiff;h=8
> 04cc4a0ad3a896ca295f771a28c6eb36ced7903
> >
> > I've been meaning to follow up on it, but I haven't had the time.  Do I need
> to send explicit patches to stable@...r or can I just ask the above commits
> be cherrypicked to 3.6?
> >
> 
> Normally the CC tag works.  Not entirely sure why it didn't for the one
> patch I asked about.  The other two commits you've highlighted here are
> lacking any sort of stable tag, so you'd have to pipe up here about them.

The patches were not initially a bug fix per se, so I didn't cc stable.  It was only later that I got reports of the patches fixing issues for some people. Normally I add the stable cc if the patch is a bug fix.

> 
> I went ahead and tried the cherry-pick myself on top of 3.6.9, in the
> order you specified above.  The 62444b7462a has a trivial conflict coming
> back.  I've attached an mbox with these three patches.  If you want to
> give them a glance over and OK them, that would be great.

They look good to me.  Thanks!

Reviewed-by: Alex Deucher <alexander.deucher@....com>

> 
> (Apologies for the attachment.  Gmail is going to mess it up otherwise.)
> 
> I have them building locally at the moment on top of 3.6.9 and don't
> expect many issues, but I can't personally test the fixes myself.
> 
> josh

--
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/