lkml.org 
[lkml]   [2009]   [Feb]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] libata-sff: fix 32-bit PIO regression
Sergei Shtylyov wrote:
> Do you really think that the transfers having lengths non-divisible by
> 4 make any *significant* percentage even on the ATAPI devices? I think
> it's you who is really wrong.

The answer depends on workload. Though rare, workloads do exist that
involve a lot of oddball querying via weird, vendor-specific SCSI[-ish]
commands.

Moreover, the likelihood and cost of a branch mispredict are both low in
this case, IMO.

Or a more human version of the rule: if you have to have a long email
thread about unlikely() placement, it is best just to avoid using
unlikely() in that case at all. Branch prediction units in modern CPUs
are damned good anyways, and there is always the likelihood that a
human-placed unlikely() becomes wrong in the future.

Plus the code is more readable without unlikely(), IMO.

Jeff





\
 
 \ /
  Last update: 2009-02-09 01:23    [W:0.072 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site