Message-ID: <479BD1D1.8020503@gmail.com>
Date: Sat, 26 Jan 2008 20:35:29 -0400
From: Kevin Winchester <kjwinchester@gmail.com>
User-Agent: Thunderbird 2.0.0.6 (X11/20071022)
MIME-Version: 1.0
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
References: <479BCE03.6070603@gmail.com>
In-Reply-To: <479BCE03.6070603@gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 3372
Lines: 95

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


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/