Re: [patch 04/11] mutex subsystem, add include/asm-x86_64/mutex.h

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

 



* Eric Dumazet <[email protected]> wrote:

> >+		:"=D" (dummy)						\
> >+		: "D" (v)						\
> >+		: "rax", "rsi", "rdx", "rcx",				\
> >+		  "r8", "r9", "r10", "r11", "memory");			\
> >+} while (0)
> 
> Hi Ingo
> 
> I do think this assembly is not very fair. It has an *insane* register 
> pressure for the compiler : The fast path is thus not so fast.

if you look at the compiler output you'll notice that it's not a problem 
actually: this fastpath is only inlined into the generic code, where it 
has no clobbering side-effects.

you are right in that if this were to be inlined left and right, this 
would be quite bad.

	Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux