At Fri, 17 Dec 2010 15:25:13 -0800, Greg KH wrote: > > > This patch doesn't apply to the .36 stable tree > If someone wants it applied there, please email the backport > to [email protected]
This patch isn't needed for 2.6.36. The relevant change was introduced since 2.6.37-rc. thanks, Takashi > > thanks, > > greg k-h > > > commit: 136711be41ec97f7f1a9c3a5e8535eb7da5fea59 > > From: Takashi Iwai <[email protected]> > > Date: Sat, 4 Dec 2010 16:13:06 +0100 > > Subject: [PATCH] agp/intel: Fix wrong kunmap in i830_cleanup() > > > > Add a missing NULL check and fix the wrong address passed to kunmap() > > in i830_cleanup(). > > > > Cc: [email protected] > > Signed-off-by: Takashi Iwai <[email protected]> > > [danvet: added cc stable] > > Signed-off-by: Daniel Vetter <[email protected]> > > > > Signed-off-by: Chris Wilson <[email protected]> > > --- > > drivers/char/agp/intel-gtt.c | 6 ++++-- > > 1 files changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/char/agp/intel-gtt.c b/drivers/char/agp/intel-gtt.c > > index 9272c38..16a2847 100644 > > --- a/drivers/char/agp/intel-gtt.c > > +++ b/drivers/char/agp/intel-gtt.c > > @@ -812,8 +812,10 @@ static int intel_fake_agp_fetch_size(void) > > > > static void i830_cleanup(void) > > { > > - kunmap(intel_private.i8xx_page); > > - intel_private.i8xx_flush_page = NULL; > > + if (intel_private.i8xx_flush_page) { > > + kunmap(intel_private.i8xx_flush_page); > > + intel_private.i8xx_flush_page = NULL; > > + } > > > > __free_page(intel_private.i8xx_page); > > intel_private.i8xx_page = NULL; > > > > _______________________________________________ > > stable mailing list > > [email protected] > > http://linux.kernel.org/mailman/listinfo/stable > _______________________________________________ stable mailing list [email protected] http://linux.kernel.org/mailman/listinfo/stable
