>> > Good to know, thanks.
>
> I have made a big mistake. I was comparing the resulting SSDT of this patch 
> to an SSDT
> Generated by a much older version of kvm/qemu. The latest kvm/qemu upstream 
> builds
> the SSDT with the operation region contained inside of methods. This should 
> resolve
> MEMA of the operation region when the method is invoked. The older SSDT that 
> I was
> looking at declared the memory region outside of methods and had a forward 
> reference
> to MEMA which we no longer support.
>
> So the latest qemu/kvm works just fine and this patch is not needed, Dan has 
> confirmed that
> the latest qemu/kvm works for his nvdimm setup.
>
> Thanks for all of the help and I'm sorry for any confusion that I might have 
> caused!

Yes, my bad as well. I saw the regression on v2.8.0, but updating to
latest qemu-mainline also fixes the problem for me.

Thanks for the debug.

Reply via email to