All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Suresh Siddha <suresh.b.siddha@intel.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	"H. Peter Anvin" <hpa@zytor.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	Daniel J Blueman <daniel.blueman@gmail.com>
Subject: Re: [GIT PULL] more x86/apic changes for v2.6.40
Date: Tue, 24 May 2011 20:37:21 +0200	[thread overview]
Message-ID: <20110524183721.GA31824@elte.hu> (raw)
In-Reply-To: <BANLkTi=bh+P07UPzhv9v=0x78o9+PUrCnw@mail.gmail.com>


* Linus Torvalds <torvalds@linux-foundation.org> wrote:

> On Tue, May 24, 2011 at 10:45 AM, Suresh Siddha
> <suresh.b.siddha@intel.com> wrote:
> >
> > Patch appended(I am sorry, should have caught this earlier. Looks like
> > my system does keep the ioapic state intact during suspend/resume.)
> 
> Looks likely. My bisect just finished as I got this email, with
> 
> 57a6f74023c7fd943160d7635bbc8d9f66e2ab54 is the first bad commit
> commit 57a6f74023c7fd943160d7635bbc8d9f66e2ab54
> Author: Suresh Siddha <suresh.b.siddha@intel.com>
> Date:   Wed May 18 16:31:36 2011 -0700
> 
>     x86, ioapic: Consolidate ioapic_saved_data[] into 'struct ioapic'
> 
>     Signed-off-by: Suresh Siddha <suresh.b.siddha@intel.com>
>     Cc: daniel.blueman@gmail.com
>     Link: http://lkml.kernel.org/r/20110518233157.830697056@sbsiddha-MOBL3.sc.intel.com
>     Signed-off-by: Ingo Molnar <mingo@elte.hu>
> 
> :040000 040000 0ae77b7536a88d0ce4f789a7c870aa3103ccc1d5
> 3460a9ec2b6a017d91b8fa6465e8d7d7cd8be71d M	arch
> 
> and I'm now going back to top-of-git and applying your patch to verify
> that it fixes it. But it looks rather likely that it does.

Ok, great: i also just sent the pull request with Suresh's fix included so you 
can pull that if the fix works fine.

Thanks,

	Ingo

  reply	other threads:[~2011-05-24 18:37 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-23 19:30 [GIT PULL] more x86/apic changes for v2.6.40 Ingo Molnar
2011-05-24  5:58 ` Linus Torvalds
2011-05-24  6:03   ` Ingo Molnar
2011-05-24  6:09     ` Linus Torvalds
2011-05-24  6:15     ` Ingo Molnar
2011-05-24 17:45       ` Suresh Siddha
2011-05-24 18:34         ` Linus Torvalds
2011-05-24 18:37           ` Ingo Molnar [this message]
2011-05-24 18:36         ` [conditional GIT PULL] x86 fixes Ingo Molnar
2011-05-24 18:50           ` Linus Torvalds
2011-05-24 19:25         ` [tip:x86/urgent] x86, ioapic: Restore ioapic entries during resume properly tip-bot for Suresh Siddha

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=20110524183721.GA31824@elte.hu \
    --to=mingo@elte.hu \
    --cc=akpm@linux-foundation.org \
    --cc=daniel.blueman@gmail.com \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=suresh.b.siddha@intel.com \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.