All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pete Zaitcev <zaitcev@redhat.com>
To: Kai Germaschewski <kai@tp1.ruhr-uni-bochum.de>
Cc: Pete Zaitcev <zaitcev@redhat.com>,
	Martin Schwidefsky <schwidefsky@de.ibm.com>,
	linux-kernel@vger.kernel.org
Subject: Re: Strange s390 code in 2.4.19-pre8
Date: Mon, 13 May 2002 12:23:30 -0400	[thread overview]
Message-ID: <20020513122330.B6208@devserv.devel.redhat.com> (raw)
In-Reply-To: <20020513121008.B29935@devserv.devel.redhat.com> <Pine.LNX.4.44.0205131116440.19498-100000@chaos.physics.uiowa.edu>

> Date: Mon, 13 May 2002 11:18:22 -0500 (CDT)
> From: Kai Germaschewski <kai@tp1.ruhr-uni-bochum.de>

> > The old Makefile was correct, and the only failing was a namespace
> > conflict between your fsm.c and fsm.c in ISDN, when CONFIG_MODVERSIONS
> > are used. The right fix is to rename your fsm.c or to create
> > fsm_s390_ksyms.c with all EXPORT_SYMBOL's sitting in there.
> > It is fixed in 2.5 with $(MODPREFIX) in Rules.make.
> 
> It's easily possible to backport the 2.5 Rules.make change to 2.4, so the 
> file doesn't have to be renamed - if you want me to do that, let me know.

Keith said to rename such things. Another easy thing to do
would be to move EXPORT_SYMBOL into s390_ksyms.c, under
#ifdef CONFIG_FSM_something. I understand that renaming
creates a diff with 100% change, so damage can easily
be slipped in, and then it's a hell to investigate.
[Let's see if a well known persona uses it as an excuse
for more commercial advertising of a certain software product].

-- Pete

  reply	other threads:[~2002-05-13 16:24 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-05-13  7:54 Strange s390 code in 2.4.19-pre8 Martin Schwidefsky
2002-05-13 16:10 ` Pete Zaitcev
2002-05-13 16:18   ` Kai Germaschewski
2002-05-13 16:23     ` Pete Zaitcev [this message]
2002-05-13 16:30       ` Kai Germaschewski
2002-05-13 16:32         ` Pete Zaitcev
2002-05-13 17:39           ` Kai Germaschewski
2002-05-13 16:16 ` Kai Germaschewski
2002-05-13 16:19   ` Pete Zaitcev
2002-05-13 16:25     ` Kai Germaschewski
2002-05-14  1:22     ` Keith Owens
  -- strict thread matches above, loose matches on Subject: below --
2002-05-13 22:42 Ulrich Weigand
2002-05-13 22:50 ` Kai Germaschewski
2002-05-10 16:47 Pete Zaitcev

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=20020513122330.B6208@devserv.devel.redhat.com \
    --to=zaitcev@redhat.com \
    --cc=kai@tp1.ruhr-uni-bochum.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=schwidefsky@de.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 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.