linux-um archives
 help / color / mirror / Atom feed
From: BlaisorBlade <blaisorblade_spam@yahoo.it>
To: Jeff Dike <jdike@addtoit.com>
Cc: user-mode-linux-devel@lists.sourceforge.net
Subject: Re: [uml-devel] [PATCH] Removing user_syms.c altogether
Date: Tue, 2 Mar 2004 20:00:12 +0100	[thread overview]
Message-ID: <200403022000.12906.blaisorblade_spam@yahoo.it> (raw)
In-Reply-To: <200403021840.i22IeBWo002559@ccure.user-mode-linux.org>

Alle 19:40, martedì 2 marzo 2004, Jeff Dike ha scritto:
> blaisorblade_spam@yahoo.it said:
> > Eeehm - yes! Note however the comment about genksyms: what I say is my
> > idea,  but possibly a wrong one.
>
> Yeah, this isn't right.  The userspace symbols are from libc, and have no
> place under kernel/.

> They should be exported from os/ somehow
No problem: a file inside os-* can actually *not* be a USER_OBJS.
> , and a
> different OS might export totally different symbols.

Yes, that is true (I thought to it). However, the need to avoid code 
duplication is still there.

So, avoid moving the symbols to ksyms.c, but change user_syms.c that way (i.e. 
make it not be an USER_OBJ), so we can avoid duplicating code. That is 
especially important because we cannot duplicate the __attribute_used__ 
macro, which is needed with newer Gcc, and which is defined differently for 
different compilers *in kernel headers*.

If you need the rebuilt patch, just let me know and I'll post it next week 
(I'm going to leave for a few days).
-- 
Paolo Giarrusso, aka Blaisorblade
Linux registered user n. 292729



-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id\x1356&alloc_id438&opÌk
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

  reply	other threads:[~2004-03-02 19:34 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-02-28 19:10 [uml-devel] [PATCH] Removing user_syms.c altogether BlaisorBlade
2004-02-28 22:33 ` Jeff Dike
2004-02-29 12:19   ` BlaisorBlade
2004-03-02 18:40     ` Jeff Dike
2004-03-02 19:00       ` BlaisorBlade [this message]
2004-03-02 23:24         ` Jeff Dike

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=200403022000.12906.blaisorblade_spam@yahoo.it \
    --to=blaisorblade_spam@yahoo.it \
    --cc=jdike@addtoit.com \
    --cc=user-mode-linux-devel@lists.sourceforge.net \
    /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