Consolidate include/asm/fcntl.h into include/linux/fcntl.h

Consolidate include/asm/fcntl.h into include/linux/fcntl.h

Post by Stephen Rothwel » Mon, 26 Aug 2002 00:20:06



Hi All,

This patch consolidates as much as possible of the fcntl.h files into
include/linux/fcntl.h.  It builds fine on i386 but may have broken some
other architectures (but should not have) - I have been as careful as
I can.

Patch available at http://www.canb.auug.org.au/~sfr/31-fcntl.1.diff.gz

Comments welcome.
--
Cheers,

http://www.canb.auug.org.au/~sfr/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

 
 
 

Consolidate include/asm/fcntl.h into include/linux/fcntl.h

Post by Matthew Wilco » Mon, 26 Aug 2002 05:40:06


I think this is a bad idea -- when doing a new port, it's easier to fill
in the bits with the current scheme rather than with your proposed scheme.
There is one part which I like:

-/* for old implementation of bsd flock () */
-#define F_EXLCK                4       /* or 3 */
-#define F_SHLCK                8       /* or 4 */

All of these can go, nobody's been using them since kernel 2.0.  I took
out the last vestiges of them from locks.c earlier in 2.5.  None of the
BSDs have them either.

--
Revolutions do not require corporate support.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

 
 
 

Consolidate include/asm/fcntl.h into include/linux/fcntl.h

Post by Stephen Rothwel » Mon, 26 Aug 2002 19:00:15


Hi Willy,


Quote:

> I think this is a bad idea -- when doing a new port, it's easier to fill
> in the bits with the current scheme rather than with your proposed scheme.

The point about nay new port is that it is likely to be exactly the same as
one of the current ports as far as these defines and structures are
concerned.  WIth the current scheme, you get exactly what happened with
PPC and PPC64 - i.e. they are identical with most of the other ports except
for a couple of defines which are only different because of an error.
There is no existing ABI that would be an excuse for them to be different.

With the new scheme, asm/fcntl.h consists of only the necessary
differences from the "norm" as defined by linux/fcntl.h.

Unjustified differences between the ports are a bad thing.  For one,
it make the library maintainers job much harder ...

The longer term intent of my consolidation efforts is to lessen the
problems of all the port maintainers trying to keep up with each other
when bugs are fixed in (more or less generic) code in one port (usually
i386 ...).  There are several bugs like this currently and I suspect
many that have not been discovered yet.

Quote:> There is one part which I like:

> -/* for old implementation of bsd flock () */
> -#define F_EXLCK           4       /* or 3 */
> -#define F_SHLCK           8       /* or 4 */

> All of these can go, nobody's been using them since kernel 2.0.  I took
> out the last vestiges of them from locks.c earlier in 2.5.  None of the
> BSDs have them either.

Wonderful, I will update my patch to remove them completely.

--
Cheers,

http://www.canb.auug.org.au/~sfr/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

 
 
 

1. [TRIVIAL] cdrom.c is the only file to include asm/fcntl.h

Hi Linus,

drivers/cdrom/cdrom.c is the only file (apart from include/linux/fcntl.h)
that includes asm/fcntl.h.  This changes that and should have no affect.

I need to do this before I consolidate the asm/fcntl.h files into
linux/fcntl.h (coming next - again).

--
Cheers,

http://www.canb.auug.org.au/~sfr/

diff -ruN 2.5.34/drivers/cdrom/cdrom.c 2.5.34-fcntl.1/drivers/cdrom/cdrom.c
--- 2.5.34/drivers/cdrom/cdrom.c        2002-09-10 11:52:23.000000000 +1000

 #include <linux/proc_fs.h>
 #include <linux/blkpg.h>
 #include <linux/init.h>
+#include <linux/fcntl.h>

-#include <asm/fcntl.h>
 #include <asm/uaccess.h>

 /* used to tell the module to turn on full debugging messages */
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

2. Looking for A Sparc 10/20

3. One more time: /usr/include/linux, /usr/include/asm

4. What does this IPTABLE entry mean?

5. #include <asm/bitops.h> -> #include <linux/bitops.h>

6. date problems

7. Consolidate include/asm/signal.h

8. 3dlabs VX1

9. SSI includes including relative includes?

10. ANSIfy include/linux/a.out.h and include/linux/nls.h

11. [2.5.41] create asm-generic/fcntl.h

12. create asm-generic/fcntl.h

13. #include <linux/errno.h> WHERE are these includes?