Re: patch: include/asm-mips/system.h __cmpxchg64 bugfix and cleanup

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

 



Wouldn't it be better to check the macro in the preprocessor instead of
runtime? And why are those defined to 0 instead of explicitly undef'ed?
I've found one bug because it was assumed to be undefined instead of 0.
If no one objects I'll post a patch undefing those and fix any bugs I've
found because of them.

On Fri, 13 Oct 2006 15:11:01 +0100
Ralf Baechle <ralf@xxxxxxxxxxxxxx> wrote:

> On Fri, Oct 13, 2006 at 09:53:05AM -0400, Antonio SJ Musumeci wrote:
> 
> > Should I apply my patch on top of this one?
> 
> No, the two patches conflict in what they're doing.  The important
> part of your patch, the fix to the if condition I've already applied.
> 
>   Ralf
> 


[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux