On ma, 2017-02-20 at 10:00 +0200, Jani Nikula wrote: > On Fri, 17 Feb 2017, Tomasz Lis <[email protected]> wrote: > > > > The recently introduced patch changed behavior of masks when > > the bit number is negative. Instead of no bits set, the new way > > makes all bits set. Problematic patch: > > drm/i915: Avoid BIT(max) - 1 and use GENMASK(max - 1, 0) > > For future reference, please find the commit id of the committed patch, > and reference that with the Fixes: tag. Please Cc the folks from the > commit. > > Whoever commits this must add: > > Fixes: 3c779a49bd7c ("drm/i915: Avoid BIT(max) - 1 and use GENMASK(max - 1, > 0)") > Cc: Joonas Lahtinen <[email protected]> > Cc: Chris Wilson <[email protected]> > Cc: Daniel Vetter <[email protected]> > Cc: Jani Nikula <[email protected]> > > Joonas, Chris, please check the rest of the regressing commit that it > doesn't suffer from the same issue. > > Thanks, > Jani.
This line was actually rest of the commit. So all good, thanks for catching this. Reviewed-by: Joonas Lahtinen <[email protected]> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
