From: Ben Greear <greearb@candelatech.com>
To: Chase Venters <chase.venters@clientec.com>
Cc: "Brian F. G. Bidulock" <bidulock@openss7.org>,
Daniel Phillips <phillips@google.com>,
Stephen Hemminger <shemminger@osdl.org>,
Sridhar Samudrala <sri@us.ibm.com>,
netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [RFC/PATCH 1/2] in-kernel sockets API
Date: Tue, 13 Jun 2006 17:19:41 -0700 [thread overview]
Message-ID: <448F561D.5080007@candelatech.com> (raw)
In-Reply-To: <200606131906.16683.chase.venters@clientec.com>
Chase Venters wrote:
> On Tuesday 13 June 2006 18:42, Ben Greear wrote:
>
>>Chase Venters wrote:
>>
>>>At least some of us feel like stable module APIs should be explicitly
>>>discouraged, because we don't want to offer comfort for code that
>>>refuses to live in the tree (since getting said code into the tree is
>>>often a goal).
>>
>>Some of us write modules for specific features that are not wanted in
>>the mainline kernel, even though they are pure GPL. Our life is hard
>>enough with out people setting out to deliberately make things more
>>difficult!
>
>
> Fair enough, but if you are doing out of tree, pure GPL modules,
> EXPORT_SYMBOL_GPL() isn't a bad thing, is it?
>
> Don't mistake me for actually having a big opinion specifically about this
> socket API's usage of EXPORT_SYMBOL()... just raising some points that I
> think apply to these decisions in general. I don't really see a compelling
> reason for EXPORT_SYMBOL() over EXPORT_SYMBOL_GPL() on the socket APIs
> though... I'm trying to imagine what kind of legitimate non-GPL modules might
> use them.
I got to the flame war late and only saw your comment that stable API should
be discouraged. That kind of thinking pisses me off because it assumes all
modules out of the tree are that way because the authors want them out of the
tree. I also understand that sometimes API needs to change, but please don't
encourage change just to punish other authors, be they proprietary or otherwise.
As for what type of EXPORT macro to use I surely don't have anything to
say that hasn't been said multiple times before.
Ben
--
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc http://www.candelatech.com
next prev parent reply other threads:[~2006-06-14 0:19 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-06-12 23:56 [RFC/PATCH 1/2] in-kernel sockets API Sridhar Samudrala
2006-06-13 5:07 ` Stephen Hemminger
2006-06-13 11:13 ` Arnaldo Carvalho de Melo
2006-06-13 11:22 ` Brian F. G. Bidulock
2006-06-13 21:12 ` Daniel Phillips
2006-06-13 21:40 ` Brian F. G. Bidulock
2006-06-13 22:00 ` Chase Venters
2006-06-13 22:30 ` Daniel Phillips
2006-06-13 22:47 ` Brian F. G. Bidulock
2006-06-13 23:59 ` Chase Venters
2006-06-14 0:31 ` Brian F. G. Bidulock
2006-06-14 0:53 ` Chase Venters
2006-06-14 6:07 ` Brian F. G. Bidulock
2006-06-14 7:58 ` Chase Venters
2006-06-14 9:28 ` Brian F. G. Bidulock
2006-06-14 10:43 ` Alan Cox
2006-06-14 10:54 ` Brian F. G. Bidulock
2006-06-14 10:36 ` Theodore Tso
2006-06-13 22:44 ` Brian F. G. Bidulock
2006-06-13 23:42 ` Ben Greear
2006-06-14 0:05 ` Chase Venters
2006-06-14 0:18 ` Brian F. G. Bidulock
2006-06-14 0:29 ` Chase Venters
2006-06-14 0:36 ` Brian F. G. Bidulock
2006-06-14 0:19 ` Ben Greear [this message]
2006-06-14 0:38 ` Brian F. G. Bidulock
2006-06-14 13:30 ` Harald Welte
2006-06-14 14:29 ` Erik Mouw
2006-06-14 15:26 ` Harald Welte
2006-06-14 17:48 ` Daniel Phillips
2006-06-14 18:03 ` Brian F. G. Bidulock
2006-06-14 20:52 ` Sridhar Samudrala
2006-06-13 14:58 ` Jan Engelhardt
2006-06-13 16:27 ` Sridhar Samudrala
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=448F561D.5080007@candelatech.com \
--to=greearb@candelatech.com \
--cc=bidulock@openss7.org \
--cc=chase.venters@clientec.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=phillips@google.com \
--cc=shemminger@osdl.org \
--cc=sri@us.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;
as well as URLs for NNTP newsgroup(s).