2001-04-26 09:11:12

by Martin Dalecki

[permalink] [raw]
Subject: PATCH: 2.4.3 tinny module interface cleanum

diff -ur linux/fs/super.c new/fs/super.c
--- linux/fs/super.c Wed Apr 18 20:41:17 2001
+++ new/fs/super.c Thu Apr 26 01:08:48 2001
@@ -691,7 +691,7 @@
* the request.
*/

-struct super_block *get_empty_super(void)
+static struct super_block *get_empty_super(void)
{
struct super_block *s;

diff -ur linux/include/linux/fs.h new/include/linux/fs.h
--- linux/include/linux/fs.h Wed Apr 18 20:41:18 2001
+++ new/include/linux/fs.h Thu Apr 26 01:03:03 2001
@@ -1291,7 +1285,6 @@

extern struct file_system_type *get_fs_type(const char *name);
extern struct super_block *get_super(kdev_t);
-struct super_block *get_empty_super(void);
extern void put_super(kdev_t);
unsigned long generate_cluster(kdev_t, int b[], int);
unsigned long generate_cluster_swab32(kdev_t, int b[], int);
diff -ur linux/kernel/ksyms.c new/kernel/ksyms.c
--- linux/kernel/ksyms.c Wed Apr 18 20:41:19 2001
+++ new/kernel/ksyms.c Thu Apr 26 00:40:48 2001
@@ -129,7 +129,6 @@
EXPORT_SYMBOL(update_atime);
EXPORT_SYMBOL(get_fs_type);
EXPORT_SYMBOL(get_super);
-EXPORT_SYMBOL(get_empty_super);
EXPORT_SYMBOL(getname);
EXPORT_SYMBOL(names_cachep);
EXPORT_SYMBOL(fput);


Attachments:
get_emty_super.diff (1.13 kB)

2001-04-26 09:47:32

by Ingo Oeser

[permalink] [raw]
Subject: Re: PATCH: 2.4.3 tinny module interface cleanum

On Thu, Apr 26, 2001 at 10:58:46AM +0200, Martin Dalecki wrote:
> 1. Help making the module interface cleaner by a tinny margin :-).

You only help changing the API during a stable[1] series. Wait until 2.5
for this.

API cannot change during stable series. (ABI can, BTW)

So lets just forget about this, ok ;-)

Regards

Ingo Oeser

[1] By stable I mean "marked as the stable branch" not the actual
behavior >;)
--
10.+11.03.2001 - 3. Chemnitzer LinuxTag <http://www.tu-chemnitz.de/linux/tag>
<<<<<<<<<<<< been there and had much fun >>>>>>>>>>>>

2001-04-26 09:54:42

by Martin Dalecki

[permalink] [raw]
Subject: Re: PATCH: 2.4.3 tinny module interface cleanum

Ingo Oeser wrote:
>
> On Thu, Apr 26, 2001 at 10:58:46AM +0200, Martin Dalecki wrote:
> > 1. Help making the module interface cleaner by a tinny margin :-).
>
> You only help changing the API during a stable[1] series. Wait until 2.5
> for this.
>
> API cannot change during stable series. (ABI can, BTW)
> So lets just forget about this, ok ;-)

So just show me one module using this function in a non-broken way!

2001-04-26 11:50:25

by Hugh Dickins

[permalink] [raw]
Subject: Re: PATCH: 2.4.3 tinny module interface cleanum

On Thu, 26 Apr 2001, Ingo Oeser wrote:
> On Thu, Apr 26, 2001 at 10:58:46AM +0200, Martin Dalecki wrote:
> > 1. Help making the module interface cleaner by a tinny margin :-).
>
> You only help changing the API during a stable[1] series. Wait until 2.5
> for this.
>
> API cannot change during stable series. (ABI can, BTW)
>
> So lets just forget about this, ok ;-)

Wrong answer: AV already did it (static get_empty_super) in 2.4.4-pre4.

Hugh