public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Kevin Winchester <kjwinchester@gmail.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: Thomas Gleixner <tglx@linutronix.de>,
	"H. Peter Anvin" <hpa@zytor.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: X fails to start with latest Linus git
Date: Sat, 26 Jan 2008 20:35:29 -0400	[thread overview]
Message-ID: <479BD1D1.8020503@gmail.com> (raw)
In-Reply-To: <479BCE03.6070603@gmail.com>

Kevin Winchester wrote:
> Hi Ingo,
> 
> Starting X (autostarted with GDM) manages to lock up my system
> (requiring a hard reset) after the first few tree merges of the window,
> and bisection shows:
> 
> -----------------
> 
> 19ef9309273d26cb005cb23e6a370353dca91099 is first bad commit
> commit 19ef9309273d26cb005cb23e6a370353dca91099
> Author: Ingo Molnar <mingo@elte.hu>
> Date:   Fri Jan 25 21:08:34 2008 +0100
> 
>     printk: use ktime_get()
> 
>     printk timestamps: use ktime_get().
> 
>     Some platforms have a functioning clocksource function only after
>     they are done with early bootup, so delay this until out of
>     SYSTEM_BOOTING state.
> 
>     it's also inherently safe now, as any bugs in this area will be
>     caught by the printk recursion checks.
> 
>     Signed-off-by: Ingo Molnar <mingo@elte.hu>
> 
> -----------------
> 
> as the first bad commit.  I have no idea why that would cause problems,
> so it could also be some timing-related problem that just happened to
> bisect to this patch.
> 
> I will try advancing to the head of Linus' tree and then reverting this
> patch to make sure it fixes the problem, but I figured I'd send this
> first to see if it is obvious to anyone.
> 

Yes, with this as my "git log" :

------------

commit 1814180093994543e8fbfd8d91f383081a9bc283
Author: Kevin Winchester <kevin@alekhine.(none)>
Date:   Sat Jan 26 20:21:44 2008 -0400

    Revert "printk: use ktime_get()"

    This reverts commit 19ef9309273d26cb005cb23e6a370353dca91099.

commit 9b73e76f3cf63379dcf45fcd4f112f5812418d0a
Merge: 50d9a12... 23c3e29...
Author: Linus Torvalds <torvalds@linux-foundation.org>
Date:   Fri Jan 25 17:19:08 2008 -0800

    Merge git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6

    * git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6:
(200 commits)
      [SCSI] usbstorage: use last_sector_bug flag universally
      [SCSI] libsas: abstract STP task status into a function
      [SCSI] ultrastor: clean up inline asm warnings
      [SCSI] aic7xxx: fix firmware build
      [SCSI] aacraid: fib context lock for management ioctls
      [SCSI] ch: remove forward declarations
      [SCSI] ch: fix device minor number management bug
      [SCSI] ch: handle class_device_create failure properly
      [SCSI] NCR5380: fix section mismatch
      [SCSI] sg: fix /proc/scsi/sg/devices when no SCSI devices
      [SCSI] IB/iSER: add logical unit reset support
      [SCSI] don't use __GFP_DMA for sense buffers if not required
      [SCSI] use dynamically allocated sense buffer
      [SCSI] scsi.h: add macro for enclosure bit of inquiry data
      [SCSI] sd: add fix for devices with last sector access problems
      [SCSI] fix pcmcia compile problem
      [SCSI] aacraid: add Voodoo Lite class of cards.
      [SCSI] aacraid: add new driver features flags
      [SCSI] qla2xxx: Update version number to 8.02.00-k7.
      [SCSI] qla2xxx: Issue correct MBC_INITIALIZE_FIRMWARE command.
      ...

-------------

The machine boots normally, but without that revert, X locks up.  Does
that make sense to anyone?

-- 
Kevin Winchester



  reply	other threads:[~2008-01-27  0:35 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-27  0:19 X fails to start with latest Linus git Kevin Winchester
2008-01-27  0:35 ` Kevin Winchester [this message]
2008-01-27  6:27   ` Ingo Molnar
2008-01-27 10:42     ` Jan Engelhardt
2008-01-27 12:31     ` Kevin Winchester
2008-01-27 12:35       ` Ingo Molnar
2008-01-27 23:08         ` Kevin Winchester
2008-01-27 12:37       ` Ingo Molnar
2008-01-27 12:41         ` Ingo Molnar
2008-01-27 12:42           ` Ingo Molnar
2008-01-27 23:12             ` Kevin Winchester
2008-01-27 23:35             ` Kevin Winchester
2008-01-27 23:10         ` Kevin Winchester
2008-01-27  7:12 ` [git pull] printk fix (was: Re: X fails to start with latest Linus git) Ingo Molnar

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=479BD1D1.8020503@gmail.com \
    --to=kjwinchester@gmail.com \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=tglx@linutronix.de \
    /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