On Wed, Jun 14, 2017 at 02:49:02PM -0500, Tom Lendacky wrote: > I guess I don't need the sme_active() check since the second part of the > if statement can only ever be true if SME is active (since mask is > unsigned). ... and you can define sme_me_mask as an u64 directly (it is that already, practically, as we don't do SME on 32-bit) and then get rid of the cast. -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.