public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Khem Raj <kraj@mvista.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: David Singleton <dsingleton@mvista.com>,
	robustmutexes@lists.osdl.org, linux-kernel@vger.kernel.org
Subject: Re: Robust Futex update
Date: Fri, 14 Oct 2005 10:40:47 -0700	[thread overview]
Message-ID: <434FED9F.6080909@mvista.com> (raw)
In-Reply-To: <20051014054522.GA22749@elte.hu>

Hi Ingo

As we pass same flags from glibc to kernel we thought it will be good to 
have defines in one place to avoid going out of sync in future.
glibc already has dependency on kernel headers. So were looking for ways 
to fix futex.h in kernel so that glibc can use it. But if it is not 
feasible then we can define these flags inside glibc headers. Only 
additional burden will be that these headers will be required to be 
synced whenever there is any change pertaining to them(mostly the defines.)

Thanks

Khem

Ingo Molnar wrote:

>* David Singleton <dsingleton@mvista.com> wrote:
>
>  
>
>>Index: linux-2.6.13/include/linux/futex.h
>>===================================================================
>>--- linux-2.6.13.orig/include/linux/futex.h
>>+++ linux-2.6.13/include/linux/futex.h
>>@@ -1,8 +1,6 @@
>> #ifndef _LINUX_FUTEX_H
>> #define _LINUX_FUTEX_H
>> 
>>-#include <linux/fs.h>
>>-
>> /* Second argument to futex syscall */
>> 
>>    
>>
>
>this chunk broke the build, so i added the #include back. Really, the 
>robust mutex glibc patches should _NOT_ automatically include the 
>kernel's futex.h file. If they do so and rely on an installed 
>kernel-headers package then they are broken. Just copy the file into the 
>glibc tree and remove the #include line.
>
>	Ingo
>  
>
>------------------------------------------------------------------------
>
>_______________________________________________
>robustmutexes mailing list
>robustmutexes@lists.osdl.org
>https://lists.osdl.org/mailman/listinfo/robustmutexes
>  
>

-- 
Khem Raj <kraj@mvista.com>
MontaVista Software, Inc.
www.mvista.com


      reply	other threads:[~2005-10-14 17:40 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-10-13  0:00 Robust Futex update David Singleton
2005-10-14  5:35 ` Ingo Molnar
2005-10-14  5:45 ` Ingo Molnar
2005-10-14 17:40   ` Khem Raj [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=434FED9F.6080909@mvista.com \
    --to=kraj@mvista.com \
    --cc=dsingleton@mvista.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=robustmutexes@lists.osdl.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox