From: Oleg Verych <olecom@flower.upol.cz>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: Andi Kleen <ak@suse.de>, LKML <linux-kernel@vger.kernel.org>,
vgoyal@in.ibm.com, akpm@osdl.org, rjw@sisk.pl,
ebiederm@xmission.com,
Reloc Kernel List <fastboot@lists.osdl.org>,
pavel@suse.cz, magnus.damm@gmail.com
Subject: reboot, not loop forever (Re: [PATCH 20/20] x86_64: Move CPU verification code to common file)
Date: Sat, 18 Nov 2006 08:10:24 +0000 [thread overview]
Message-ID: <20061118081024.GE14673@flower.upol.cz> (raw)
In-Reply-To: <455EB72B.1010103@zytor.com>
On Fri, Nov 17, 2006 at 11:32:59PM -0800, H. Peter Anvin wrote:
> Oleg Verych wrote:
> >On Fri, Nov 17, 2006 at 10:59:32PM -0800, H. Peter Anvin wrote:
> >>Oleg Verych wrote:
> >>>It will burn CPU, until power cycle will be done (my AMD64 laptop and
> >>>Intel's amd64 destop PC require that). In case of reboot timeout (or
> >>>just reboot with jump to BIOS), i will just choose another image to boot
> >>>or will press F8 to have another boot device.
> >>>
> >>That's a fairly stupid argument, since it assumes operator intervention,
> >>at which point you have access to the machine anyway.
> >
> >I would never call *power cycle* stupid, just because from physics
> >point of veiw.
> >
> >Example. I have my flower.upol.cz many kilometers far away from me.
> >I used to boot it from that flash (new hardware, sata problems, etc).
> >
> >When something goes wrong with rc kernel or power source, bum.
> >And i had to move my ass there, just to press reset. Because.
>
> Yes, and you would have to do that to press F8 too.
That peace of code used in many places, thus, i've mentioned F8, if
wrong kernel on wrong computer was launched.
> >While i have "power on, on AC failures" in BIOS, *sometimes* flash
> >will not boot (i don't know why, maybe it's GRUB+flash-read,
> >or BIOS usb hdd implementation specific).
>
> I was making the point that there is unattended recovery possible. That
> makes it a significant argument. That a user on a laptop has to wait
> four seconds pushing the power button is not.
As additional note to Andi and many of you, who will say, that it's
a couple of asm instructions, just send patch.
I'm see many kinds of reboot functions in include/linux/reboot.h.
There even reboot_fixup.h. Some of them may be copy/pasted in place of
that while(1) loop, who knows which exactly? What problems it may cause?
I used to write PC bootloaders with tasm, when i was a child
(10 years ago). Nothing major, ok with that.
But i bet, i will spot whitespace and tabification issues in files,
i will visit with emacs and eventually making patches with that. First
ever try with top Makefile failed, this makefile have something after
tabification, that Andrew Morton's makefile (from FC5) doesn't like.
Funny. Don't call me bureaucrat, but this is (my) mater of *not* being
kind of dumb.
> -hpa
Thanks.
--
-o--=O`C info emacs : not found /. .\ (is there any reason to live?)
#oo'L O info make : not found o ( R.I.P )
<___=E M man gcc : not found .-- ( Debian Operating System )
next prev parent reply other threads:[~2006-11-18 8:03 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-17 22:34 [RFC][PATCH 0/20] x86_64: Relocatable bzImage (V3) Vivek Goyal
2006-11-17 22:36 ` [PATCH 1/20] x86_64: Align data segment to PAGE_SIZE boundary Vivek Goyal
2006-11-17 22:37 ` [PATCH 2/20] x86_64: Assembly safe page.h and pgtable.h Vivek Goyal
2006-11-18 8:49 ` Andi Kleen
2006-11-18 13:19 ` Vivek Goyal
2006-11-17 22:38 ` [PATCH 3/20] x86_64: Kill temp_boot_pmds Vivek Goyal
2006-11-17 22:39 ` [PATCH 4/20] x86_64: Cleanup the early boot page table Vivek Goyal
2006-11-17 22:40 ` [PATCH 5/20] x86_64: Fix early printk to use standard ISA mapping Vivek Goyal
2006-11-17 22:41 ` [PATCH 6/20] x86_64: Modify copy bootdata to use virtual addresses Vivek Goyal
2006-11-17 22:42 ` [PATCH 7/20] x86_64: cleanup segments Vivek Goyal
2006-11-17 22:44 ` [PATCH 8/20] x86_64: Add EFER to the set registers saved by save_processor_state Vivek Goyal
2006-11-18 0:11 ` Pavel Machek
2006-11-17 22:45 ` [PATCH 9/20] x86_64: 64bit PIC SMP trampoline Vivek Goyal
2006-11-18 0:27 ` Pavel Machek
2006-11-18 0:33 ` Vivek Goyal
2006-11-18 0:38 ` Pavel Machek
2006-11-17 22:47 ` [PATCH 10/20] x86_64: wakeup.S Remove dead code Vivek Goyal
2006-11-18 0:14 ` Pavel Machek
2006-11-17 22:48 ` [PATCH 11/20] x86_64: wakeup.S Rename labels to reflect right register names Vivek Goyal
2006-11-18 0:15 ` Pavel Machek
2006-11-17 22:49 ` [PATCH 12/20] x86_64: wakeup.S Misc cleanup Vivek Goyal
2006-11-18 0:19 ` Pavel Machek
2006-11-18 1:25 ` Vivek Goyal
2006-11-17 22:51 ` [PATCH 13/20] x86_64: 64bit PIC ACPI wakeup trampoline Vivek Goyal
2006-11-18 0:20 ` Pavel Machek
2006-11-17 22:52 ` [PATCH 14/20] x86_64: Modify discover_ebda to use virtual address Vivek Goyal
2006-11-17 22:54 ` [PATCH 15/20] x86_64: Remove the identity mapping as early as possible Vivek Goyal
2006-11-17 22:55 ` [PATCH 16/20] x86_64: __pa and __pa_symbol address space separation Vivek Goyal
2006-11-17 22:56 ` [PATCH 17/20] x86_64: Remove CONFIG_PHYSICAL_START Vivek Goyal
2006-11-18 1:14 ` Magnus Damm
2006-11-18 2:45 ` Vivek Goyal
2006-11-20 10:02 ` Magnus Damm
2006-11-17 22:57 ` [PATCH 18/20] x86_64: Relocatable kernel support Vivek Goyal
2006-11-18 5:49 ` Oleg Verych
2006-11-18 6:49 ` Andi Kleen
2006-11-17 22:58 ` [PATCH 19/20] x86_64: Extend bzImage protocol for relocatable kernel Vivek Goyal
2006-11-18 0:30 ` H. Peter Anvin
2006-11-18 0:37 ` Vivek Goyal
2006-11-18 0:45 ` H. Peter Anvin
2006-11-18 1:47 ` Vivek Goyal
2006-11-17 22:59 ` [PATCH 20/20] x86_64: Move CPU verification code to common file Vivek Goyal
2006-11-18 5:21 ` Oleg Verych
2006-11-18 6:38 ` Andi Kleen
2006-11-18 6:41 ` H. Peter Anvin
[not found] ` <20061118070101.GA14673@flower.upol.cz>
2006-11-18 6:59 ` H. Peter Anvin
2006-11-18 7:22 ` Oleg Verych
2006-11-18 7:32 ` H. Peter Anvin
2006-11-18 8:10 ` Oleg Verych [this message]
2006-11-18 8:06 ` Andi Kleen
2006-11-18 8:16 ` H. Peter Anvin
2006-11-18 8:29 ` Andi Kleen
2006-11-18 10:55 ` Paul Mackerras
2006-11-18 10:58 ` Andi Kleen
2006-11-18 12:59 ` Vivek Goyal
2006-11-18 17:46 ` Pavel Machek
2006-11-18 8:52 ` [RFC][PATCH 0/20] x86_64: Relocatable bzImage (V3) Andi Kleen
2006-11-18 13:14 ` Vivek Goyal
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=20061118081024.GE14673@flower.upol.cz \
--to=olecom@flower.upol.cz \
--cc=ak@suse.de \
--cc=akpm@osdl.org \
--cc=ebiederm@xmission.com \
--cc=fastboot@lists.osdl.org \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=pavel@suse.cz \
--cc=rjw@sisk.pl \
--cc=vgoyal@in.ibm.com \
/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®