lkml.org 
[lkml]   [2005]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/19] MUTEX: Introduce simple mutex implementation

* Arjan van de Ven <arjan@infradead.org> 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
[unhandled content-type:application/x-bzip2]
\
 
 \ /
  Last update: 2005-12-13 10:08    [W:0.199 / U:1.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site