[Intel-gfx] [PATCH] Another flavour of for_each_engine_masked()

Dave Gordon david.s.gordon at intel.com
Thu Sep 1 16:59:44 UTC 2016


On 01/09/16 15:48, Chris Wilson wrote:
> On Thu, Sep 01, 2016 at 03:17:44PM +0100, Dave Gordon wrote:
>> This macro was recently updated to skip testing for non-existent or
>> uninteresting engines by using ffs() to directly find the next engine of
>> interest. However, it required the introduction of a caller-provided
>> temporary variable, which some people regard as inelegant. So, this
>> patch provides another variant, which still uses the fast-skip mechanism
>> but without requiring the temporary, for the cost of a slight increase
>> in code size (~20 bytes per callsite).
>
> Slight? Next.
> -Chris

106 bytes out of 1166488; that's 0.009% or 91ppm.
The adjective "slight" definitely applies :)

.Dave.


More information about the Intel-gfx mailing list