On 19/02/2025 10:02 am, Jan Beulich wrote:
> Avoid using the same literal number (8) in two distinct places.

You say two places but this is only one hunk.  I presume you mean
SIF_PM_MASK as the other place.

In which case I'd suggest that this would be clearer if phrased as "Use
MASK_INTR() to avoid opencoding the literal 8."

>
> Signed-off-by: Jan Beulich <jbeul...@suse.com>

Preferably with some kind of adjustment to the commit message, Acked-by:
Andrew Cooper <andrew.coop...@citrix.com>

Reply via email to