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 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.