All of lore.kernel.org
 help / color / mirror / Atom feed
From: Robert Love <rml@tech9.net>
To: Kent Borg <kentborg@borg.org>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>, LMKL <linux-kernel@vger.kernel.org>
Subject: Re: Linux 2.4.9-ac15
Date: 24 Sep 2001 15:08:22 -0400	[thread overview]
Message-ID: <1001358505.5382.4.camel@phantasy> (raw)
In-Reply-To: <20010924145944.A23589@borg.org>
In-Reply-To: <200109241610.f8OGAUk19897@adsl-209-76-109-63.dsl.snfc21.pacbell.net> <E15lYYQ-00032z-00@the-village.bc.nu>  <20010924145944.A23589@borg.org>

On Mon, 2001-09-24 at 14:59, Kent Borg wrote:
> Does CONFIG_APM depend upon CONFIG_MAGIC_SYSRQ?  Is it supposed to?

No, it shouldn't.  There must of been a bug in the new sysrq code merged
in.  Personally, I would just enable sysrq support for now.

> I tried doing a build of 2.4.9-ac15 plus
> patch-rml-2.4.10-pre12-preempt-kernel-1 (which did apply nicely with
> only one hunk moved only one line), but my build failed with:

Btw, this isn't the preemption patches fault, although I would recommend
you use the patches I make explicitly for Alan's tree. 2.4.9-ac14 will
apply cleanly and is at http://tech9.net/rml/linux

>   arch/i386/kernel/kernel.o: In function `apm':
>   arch/i386/kernel/kernel.o(.text+0xbd35): undefined reference to `__sysrq_lock_table'
>   arch/i386/kernel/kernel.o(.text+0xbd3c): undefined reference to `__sysrq_get_key_op'
>   arch/i386/kernel/kernel.o(.text+0xbd4d): undefined reference to `__sysrq_put_key_op'
>   arch/i386/kernel/kernel.o(.text+0xbd54): undefined reference to `__sysrq_unlock_table'
> 
> Which is apm.c failing on the likes of: 
> 
>   register_sysrq_key('o',&sysrq_poweroff_op);
> 
> and:
> 
>   unregister_sysrq_key('o',&sysrq_poweroff_op);
> 
> If I turn on CONFIG_MAGIC_SYSRQ, it builds happily.

I am not familar with the code, but either apm.c needs to wrap its sysrq
calls with ifdefs or the sysrq code needs to optimize away via defines
its various sysrq functions if it is not enabled.

The maintainer (Crutcher?) should see this.

> Previously 2.4.9-ac12 with the same rml preemption patch was happy to
> compile for me.  I have been using it on my Sony Vaio PCG-Z505LE
> happily.
> 
> 2.4.10 is also happy to compile without CONFIG_MAGIC_SYSRQ.

-- 
Robert M. Love
rml at ufl.edu
rml at tech9.net


  reply	other threads:[~2001-09-24 19:08 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-09-24 15:41 Linux 2.4.9-ac15 Alan Cox
2001-09-24 16:10 ` Wayne Whitney
2001-09-24 16:25   ` Alan Cox
2001-09-24 18:59     ` Kent Borg
2001-09-24 19:08       ` Robert Love [this message]
2001-09-24 19:00 ` Stefan Fleiter
2001-09-25 12:28 ` Hirling Endre
2001-09-25 16:49 ` Matthias Andree

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=1001358505.5382.4.camel@phantasy \
    --to=rml@tech9.net \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=kentborg@borg.org \
    --cc=linux-kernel@vger.kernel.org \
    /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.