On Thu, Jun 15, 2017 at 11:33:41AM -0500, Tom Lendacky wrote: > Changing the signature back reverts to the original way, so this can be > looked at separate from this patchset then. Right, the patch which added the volatile thing was this one: 4bf5beef578e ("iommu/amd: Don't put completion-wait semaphore on stack") and the commit message doesn't say why the thing needs to be volatile at all. Joerg? -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.