parisc: Drop HP-UX specifix fcntl and signal flags

PATCH In review Helge Deller 2 participants 3 messages Aug 29, 2020 PA-RISC architecture

Share & tools

Helge Deller

author maintainer

2020-08-29 10:51 UTC #0 lore raw

PATCH 1/1 parisc: Drop HP-UX specifix fcntl and signal flags

Those flags are nowhere used in the Linux kernel and were added when we still wanted to support HP-UX in a compat mode. Since we never will support HP-UX, drop those flags.

Signed-off-by Helge Deller <[email protected]>

arch/parisc/include/uapi/asm/fcntl.h +0 -3

index 03ce20e5ad7d..4289ee732c4c 100644
@@ -3,7 +3,6 @@
 #define _PARISC_FCNTL_H
 
 #define O_APPEND	000000010
-#define O_BLKSEEK	000000100 /* HPUX only */
 #define O_CREAT		000000400 /* not fcntl */
 #define O_EXCL		000002000 /* not fcntl */
 #define O_LARGEFILE	000004000

@@ -12,13 +11,11 @@
 #define O_NONBLOCK	000200004 /* HPUX has separate NDELAY & NONBLOCK */
 #define O_NOCTTY	000400000 /* not fcntl */
 #define O_DSYNC		001000000 /* HPUX only */
-#define O_RSYNC		002000000 /* HPUX only */
 #define O_NOATIME	004000000
 #define O_CLOEXEC	010000000 /* set close_on_exec */
 
 #define O_DIRECTORY	000010000 /* must be a directory */
 #define O_NOFOLLOW	000000200 /* don't follow links */
-#define O_INVISIBLE	004000000 /* invisible I/O, for DMAPI/XDSM */
 
 #define O_PATH		020000000
 #define __O_TMPFILE	040000000

arch/parisc/include/uapi/asm/signal.h +0 -1

index d38563a394f2..045fde6d2fac 100644
@@ -61,7 +61,6 @@
 #define SA_NODEFER	0x00000020
 #define SA_RESTART	0x00000040
 #define SA_NOCLDWAIT	0x00000080
-#define _SA_SIGGFAULT	0x00000100 /* HPUX */
 
 #define SA_NOMASK	SA_NODEFER
 #define SA_ONESHOT	SA_RESETHAND

Rolf Eike Beer

2020-08-29 11:30 UTC #1 lore raw

Am Samstag, 29. August 2020, 12:51:12 CEST schrieb Helge Deller:

Quoted 20 lines

Those flags are nowhere used in the Linux kernel and were added when we still wanted to support HP-UX in a compat mode. Since we never will support HP-UX, drop those flags.
Signed-off-by Helge Deller <[email protected]>
(quoted diff) +0 -0
b/arch/parisc/include/uapi/asm/fcntl.h index 03ce20e5ad7d..4289ee732c4c 100644
arch/parisc/include/uapi/asm/fcntl.h +0 -1
@@ -3,7 +3,6 @@
 #define _PARISC_FCNTL_H
 
 #define O_APPEND	000000010
-#define O_BLKSEEK	000000100 /* HPUX only */
 #define O_CREAT		000000400 /* not fcntl */
 #define O_EXCL		000002000 /* not fcntl */
 #define O_LARGEFILE	000004000

arch/parisc/include/uapi/asm/fcntl.h line 15

Quoted 4 lines

arch/parisc/include/uapi/asm/fcntl.h +0 -1

@@ -12,13 +11,11 @@
 #define O_NONBLOCK	000200004 /* HPUX has separate NDELAY & NONBLOCK */
 #define O_NOCTTY	000400000 /* not fcntl */
 #define O_DSYNC		001000000 /* HPUX only */
-#define O_RSYNC		002000000 /* HPUX only */

So, what about O_DSYNC? Either it should go away the same if it is HPUX only, or the comment needs to be fixed.

Eike

Helge Deller

author maintainer

2020-08-29 12:23 UTC #2 lore raw

Quoted 15 lines · Rolf Eike Beer

Am Samstag, 29. August 2020, 12:51:12 CEST schrieb Helge Deller:
Quoted 14 lines
Those flags are nowhere used in the Linux kernel and were added when we still wanted to support HP-UX in a compat mode. Since we never will support HP-UX, drop those flags.
Signed-off-by Helge Deller <[email protected]>
(quoted diff) +0 -0
b/arch/parisc/include/uapi/asm/fcntl.h index 03ce20e5ad7d..4289ee732c4c 100644
arch/parisc/include/uapi/asm/fcntl.h +0 -0
@@ -3,7 +3,6 @@
 #define _PARISC_FCNTL_H
 

Quoted 13 lines

arch/parisc/include/uapi/asm/fcntl.h line 15

Quoted 10 lines

arch/parisc/include/uapi/asm/fcntl.h +0 -2

 #define O_APPEND	000000010
-#define O_BLKSEEK	000000100 /* HPUX only */
 #define O_CREAT		000000400 /* not fcntl */
 #define O_EXCL		000002000 /* not fcntl */
 #define O_LARGEFILE	000004000

@@ -12,13 +11,11 @@
 #define O_NONBLOCK	000200004 /* HPUX has separate NDELAY & NONBLOCK */
 #define O_NOCTTY	000400000 /* not fcntl */
 #define O_DSYNC		001000000 /* HPUX only */
-#define O_RSYNC		002000000 /* HPUX only */

So, what about O_DSYNC? Either it should go away the same if it is HPUX only, or the comment needs to be fixed.

That's true.
I had it fixed already in my git tree:
https://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux.git/commit/?h=for-next&id=297f57686b3f634306b6a67a90d5a146392693c2

Helge