From: Pete Zaitcev <zaitcev@redhat.com>
To: Andrew Morton <akpm@osdl.org>
Cc: mpm@selenic.com, linux-kernel@vger.kernel.org
Subject: Re: [patch] arch-specific cond_syscall usage issues
Date: Fri, 9 Jan 2004 21:21:08 -0800 [thread overview]
Message-ID: <20040109212108.3d8f5e54.zaitcev@redhat.com> (raw)
In-Reply-To: <20040109193753.3c158b3b.akpm@osdl.org>
On Fri, 9 Jan 2004 19:37:53 -0800
Andrew Morton <akpm@osdl.org> wrote:
> > Experimenting with trying to use cond_syscall for a few arch-specific
> > syscalls, I discovered that it can't actually be used outside the file
> > in which sys_ni_syscall is declared because the assembler doesn't feel
> > obliged to output the symbol in that case:
> > One arch (PPC) is apparently trying to use cond_syscall this way
> > anyway, though it's probably never been actually tested as the above
> > test was done on a PPC.
>
> So why does the PPC kernel successfully link?
Perhaps it never was tested right when the change went in.
I saw the same failure when hch did it to sparc, I later did the same:
moved the syscall into kernel/sys.c. Mine wasn't arch specific.
Maybe 3 arches used it (pci_config_read_word or such).
I saw the way ppc did it at that time (2.5.75),
and thought that their toolchain must have been smarter than mine.
The patch is easy. The hard road would be to take it to binutils people
like H.J.Lu and see what they say.
-- Pete
next prev parent reply other threads:[~2004-01-10 5:21 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-01-10 3:29 [patch] arch-specific cond_syscall usage issues Matt Mackall
2004-01-10 3:37 ` Andrew Morton
2004-01-10 3:53 ` Matt Mackall
2004-01-10 5:21 ` Pete Zaitcev [this message]
2004-01-10 6:03 ` Matt Mackall
2004-01-14 16:13 ` Tom Rini
2004-01-14 19:31 ` Andrew Morton
2004-01-14 19:47 ` Matt Mackall
2004-01-14 19:49 ` Tom Rini
2004-04-23 19:14 ` [PATCH 2.6] include/asm-ppc/dma-mapping.h: dma_unmap_page() Arthur Othieno
2004-04-23 23:50 ` Tom Rini
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=20040109212108.3d8f5e54.zaitcev@redhat.com \
--to=zaitcev@redhat.com \
--cc=akpm@osdl.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mpm@selenic.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