Re: Changes to fcntl.h and F_* numbers?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/20/2009 11:57 PM, Carlos O'Donell wrote:
On Thu, Nov 19, 2009 at 12:23 PM, James Bottomley
<James.Bottomley@xxxxxxxxxxxxxxxxxxxxx>  wrote:
On Thu, 2009-11-19 at 12:00 -0500, Carlos O'Donell wrote:
On Wed, Nov 18, 2009 at 10:22 PM, Kyle McMartin<kyle@xxxxxxxxxxx>  wrote:
On Wed, Nov 18, 2009 at 03:22:15PM -0500, Carlos O'Donell wrote:
Kyle, Helge,

I've seen some recent churn regarding the kernel fcntl.h and F_* numbers.

If our numbers change please keep me in the loop so I can update the C
library side.


?

http://marc.info/?l=linux-api&m=125688056905828&w=2

But this isn't parisc specific ... it's a bug CodeSourcery reported in
the ABI assignment in asm-generic.

Yes, that's correct, the original issue doesn't apply to parisc.

I pull from kyle's tree here:
git://git.kernel.org/pub/scm/linux/kernel/git/kyle/parisc-2.6.git

In kyle's tree I see:
"arch/parisc/include/asm/fcntl.h"
#define F_GETOWN_EX     15
#define F_SETOWN_EX     16

The original message that Joseph Myers at CodeSourcery posted here:
http://marc.info/?l=linux-api&m=125466028819859&w=2

Does not apply to us.

We already look to have this sorted out locally in our fcntl.h ... so
we're not going to be changing anything, are we?  Or are you saying
(because we could get into trouble in future) that we should adopt the
asm generic ones?

I don't think we have to change anything.

It would probably be less maintenance for Kyle in the future if we
adopted the generic ones for future flags.

The message I referenced here:
http://marc.info/?l=linux-api&m=125688056905828&w=2

by Stephen Rothwell<sfr@xxxxxxxxxxxxxxxx>, has a parisc specific piece here:

~~~
diff --git a/arch/parisc/include/asm/fcntl.h b/arch/parisc/include/asm/fcntl.h
index 5f39d55..1e1c824 100644
--- a/arch/parisc/include/asm/fcntl.h
+++ b/arch/parisc/include/asm/fcntl.h
@@ -28,8 +28,6 @@
  #define F_SETOWN	12	/*  for sockets. */
  #define F_SETSIG	13	/*  for sockets. */
  #define F_GETSIG	14	/*  for sockets. */
-#define F_GETOWN_EX	15
-#define F_SETOWN_EX	16

  /* for posix fcntl() and lockf() */
  #define F_RDLCK		01
~~~

I don't know why he wants to remove bits from our fcntl.h.

The intent of my original email was to say "Hi! I'm the libc
maintainer, I'll fix things if you notice a problem."

I'm pretty sure I'm on linux-api, but I missed this issue.

Cheres,
Carlos.

Hi Carlos,

FWIW, Stephen's patch has been applied to mainline (2.6.32-rc8) in the meantime...

Helge
--
To unsubscribe from this list: send the line "unsubscribe linux-parisc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux SoC]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux