Re: [PATCH] alpha: simplify and optimize sched_find_first_bit

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

 



On 04/08/2010 11:34 AM, mattst88@xxxxxxxxx wrote:
> -	return __ffs(b0) + ofs;
> +	unsigned long output;
> +	asm(
> +		"cmoveq %0,64,%1        # ofs = (b[0] ? ofs : 64);\n"
> +		"cmoveq %0,%2,%0        # temp = (b[0] ? b[0] : b[1]);\n"
> +		"cttz   %0,%0           # output = cttz(temp);\n "
> +		: "=r" (output), "=r" (ofs)
> +		: "r" (b[1]), "0" (b[0]), "1" (0)
> +	);
> +	return output + ofs;

NACK.

You need to move that cttz out of the asm as well, to continue
to support pre-ev67.  Ack if you adjust to use __ffs.


r~
--
To unsubscribe from this list: send the line "unsubscribe linux-alpha" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Netdev]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux