From: Sam Ravnborg <sam@ravnborg.org>
To: Ingo Molnar <mingo@elte.hu>
Cc: "Rafael J. Wysocki" <rjw@sisk.pl>,
Roland McGrath <roland@redhat.com>, Pavel Machek <pavel@suse.cz>,
linux-kernel@vger.kernel.org,
Thomas Gleixner <tglx@linutronix.de>,
"H. Peter Anvin" <hpa@zytor.com>
Subject: Re: [PATCH] fix rebuild of wakeup.bin
Date: Sun, 4 May 2008 09:21:14 +0200 [thread overview]
Message-ID: <20080504072114.GB15315@uranus.ravnborg.org> (raw)
In-Reply-To: <20080503194149.GC25097@elte.hu>
On Sat, May 03, 2008 at 09:41:49PM +0200, Ingo Molnar wrote:
>
> * Sam Ravnborg <sam@ravnborg.org> wrote:
>
> > On Sat, May 03, 2008 at 08:06:12PM +0200, Ingo Molnar wrote:
> > >
> > > * Rafael J. Wysocki <rjw@sisk.pl> wrote:
> > >
> > > > > Add wakeup.lds to list af targets to avoid rebuilding wakeup.bin -
> > > > > from Roland McGrath.
> > > >
> > > > Thanks for fixing this!
> > > >
> > > > Ingo, please merge.
> > >
> > > applied - thanks all!
> >
> > Just a note - I do not think this solves the problem Daniel has
> > reported with respect to first build of -next. I am awaiting more data
> > before I can analyse it further.
>
> yeah, i suspected that - that appears to be a race of some sort, i.e. a
> missing dependency coupled with parallelizable rules, not an over-eager
> dependency as fixed by this patch, right?
In the end it should be fixed by this patch.
It turned out to be a user rights issue where I assume
wakeup-lds were owned by root because we created it
despite have a fully build kernel.
This also explains why we did not see more reports on the subject.
So please push this fix towards linus soonish.
Thanks,
Sam
next prev parent reply other threads:[~2008-05-04 7:21 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-01 1:05 arch/x86/kernel/acpi/realmode/Makefile Roland McGrath
2008-05-01 9:31 ` [PATCH] fix rebuild of wakeup.bin Sam Ravnborg
2008-05-01 10:48 ` Rafael J. Wysocki
2008-05-03 18:06 ` Ingo Molnar
2008-05-03 18:15 ` Sam Ravnborg
2008-05-03 19:41 ` Ingo Molnar
2008-05-04 7:21 ` Sam Ravnborg [this message]
2008-05-01 9:39 ` arch/x86/kernel/acpi/realmode/Makefile Sam Ravnborg
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=20080504072114.GB15315@uranus.ravnborg.org \
--to=sam@ravnborg.org \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=pavel@suse.cz \
--cc=rjw@sisk.pl \
--cc=roland@redhat.com \
--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
all inboxes | Powered by JetHome®