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
next prev parent 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