public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Zhenyu Wang <zhenyuw@linux.intel.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	stable@kernel.org, stern@rowland.harvard.edu
Subject: Re: [PATCH 1/3] agp/intel: remove restore in resume
Date: Fri, 11 Sep 2009 09:03:42 +0800	[thread overview]
Message-ID: <20090911010342.GA6819@zhen-devel.sh.intel.com> (raw)
In-Reply-To: <20090910134252.5fc54ce2.akpm@linux-foundation.org>

[-- Attachment #1: Type: text/plain, Size: 1342 bytes --]

On 2009.09.10 13:42:52 -0700, Andrew Morton wrote:
> On Fri,  4 Sep 2009 09:43:34 +0800
> Zhenyu Wang <zhenyuw@linux.intel.com> wrote:
> 
> > As early pci resume has already restored config for host
> > bridge and graphics device, don't need to restore it again,
> > which might cause problem on some chips, like 845G tested by
> > Alan Stern.
> > 
> > Cc: Stable Team <stable@kernel.org>
> > Cc: Alan Stern <stern@rowland.harvard.edu>
> > Signed-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>
> 
> Why were these patches cc'ed to stable@kernel.org?  There's nothing in
> the changelog which explains why the problem which is being fixed is
> sufficiently serious to warrant backporting the patch.

Sorry, Andrew. I wasn't awared that an earlier version of this patch
has already been merged, and this patch fixed resume issue on Alan's
845G, which was discussed on linux-pm list, I should have made more
clear note about that. We've also seen other resume failure bugs, which
might be relate to this one. So I think it should be fine for stable.

> 
> Was there a [patch 3/3]?  I didn't receive it.

sorry, my mistake, the 3/3 commit on my tree is not ready to send, 
forget to make git format-patch happy.

-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

  reply	other threads:[~2009-09-11  1:04 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-04  1:43 [PATCH 1/3] agp/intel: remove restore in resume Zhenyu Wang
2009-09-04  1:43 ` [PATCH 2/3] drm/i915: " Zhenyu Wang
2009-09-10 20:42 ` [PATCH 1/3] agp/intel: " Andrew Morton
2009-09-11  1:03   ` Zhenyu Wang [this message]
2009-09-11  1:22     ` Andrew Morton
2009-09-11  1:49       ` Zhenyu Wang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090911010342.GA6819@zhen-devel.sh.intel.com \
    --to=zhenyuw@linux.intel.com \
    --cc=akpm@linux-foundation.org \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=stable@kernel.org \
    --cc=stern@rowland.harvard.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox