* Arjan van de Ven <[email protected]> wrote: > > even better than that, why not use the solution that we've implemented > > for the -rt patchset, more than a year ago? > > > > the solution i took was this: > > > > - i did not touch the 'struct semaphore' namespace, but introduced a > > 'struct compat_semaphore'. > > which I think is wrong. THis naming sucks. Sure doing a full sed on > the tree is not pretty but it's also not THAT painful. And the pain of > wrong names is something the kernel needs to carry around for years. well, i'm all for renaming struct semaphore to struct mutex, but dont the same arguments apply as to 'struct timer_list'? just to see the scope, i've attached semaphore-to-mutex.patch, which just dumbly converts all 'struct semaphore' occurances to 'struct mutex', against Linus-git-curr: 405 files changed, 568 insertions(+), 568 deletions(-) it's not _that_ bad, if done overnight. It does not touch any of the down/up APIs. Touching those would create a monster patch and monster impact. Ingo
Attachment:
semaphore-to-mutex.patch.bz2
Description: BZip2 compressed data
- Follow-Ups:
- Re: [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- From: Arjan van de Ven <[email protected]>
- Re: [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- From: Andi Kleen <[email protected]>
- Re: [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- References:
- [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- From: David Howells <[email protected]>
- Re: [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- From: Andrew Morton <[email protected]>
- Re: [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- From: Ingo Molnar <[email protected]>
- Re: [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- From: Arjan van de Ven <[email protected]>
- [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- Prev by Date: Re: [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- Next by Date: Re: [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- Previous by thread: Re: [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- Next by thread: Re: [PATCH 1/19] MUTEX: Introduce simple mutex implementation
- Index(es):
 |