On Tue, 10 Apr 2001, David Howells wrote:
>
> Here's a patch that fixes RW semaphores on the i386 architecture. It is very
> simple in the way it works.
XADD only works on Pentium+.
That's no problem if we make this SMP-specific - I doubt anybody actually
uses SMP on i486's even if the machines exist, as I think they all had
special glue logic that Linux would have trouble with anyway. But the
advantages of being able to use one generic kernel that works on plain UP
i386 machines as well as SMP P6+ machines is big enough that I would want
to be able to say "CONFIG_X86_GENERIC" + "CONFIG_SMP".
Even if it would be noticeably slower (ie a fallback to a spinlock might
be perfectly ok).
If you do this, I woul dsuggest having asm-i386/{rwsem.h|rwsem-xadd.h},
and just having a
#ifndef CONFIG_XADD
#include <asm/rwsem.h>
#else
#include <asm/rwsem-xadd.h>
#endif
(And adding "CONFIG_XADD" to the list of generated optimization
configuration options in arch/i386/config.in, of course).
That way we don't make the semaphore.h file even more unreadable.
Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Sun Apr 15 2001 - 21:00:14 EST