Re: [PATCH] clean up exports in fs/{open,read_write}.c

Christoph Hellwig wrote:

On Fri, Aug 24, 2007 at 11:33:10AM +0800, Eugene Teo wrote:

Takashi-san fixed sound/isa/wavefront/wavefront_synth.c to use
request_firmware instead of sys_*. Since that is the last driver in the
kernel that uses sys_{read,close}, this patch kills these exports. sys_open
is left exported for sparc64 only.

I can't find any spar user of it, so please kill it.


it's in the solaris compat code which is modular

In the case a symbol
is needed only for a particular architecture it's better to do it in the
architecture-specific ksyms file than with an ifdef, btw.


agreed.

(well if it's only 1 or 2, if it's "half of them" it's obviously adifferent story)

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Follow-Ups:
- Re: [PATCH] clean up exports in fs/{open,read_write}.c
  - From: Christoph Hellwig <[email protected]>

References:
- [PATCH] clean up exports in fs/{open,read_write}.c
  - From: Eugene Teo <[email protected]>
- Re: [PATCH] clean up exports in fs/{open,read_write}.c
  - From: Christoph Hellwig <[email protected]>

Prev by Date: RE: [PATCH] mptsas: scan the logical volume at first
Next by Date: Re: [-mm patch] unexport sys_{open,read}
Previous by thread: Re: [PATCH] clean up exports in fs/{open,read_write}.c
Next by thread: Re: [PATCH] clean up exports in fs/{open,read_write}.c
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Newbies] [Netfilter] [Bugtraq] [Photo] [Stuff] [Gimp] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Video 4 Linux] [Linux for the blind] [Linux Resources]