Manuel Bouyer wrote:
Looks good, but did you try building XEN2 kernels ? I can't see the xen_*mb() macros defined for xen2. I think defining them in include/xen-public/xen.h should be enough.
Correct, hypervisor.h is not included for Xen < 3. Guess that it breaks the XEN2 builds ( :( ). I can wait for XEN2 retirement, there is no hurry in such a patch.
BTW, isn't xen-public headers those used by XenSource? Should I really modify this one instead of xen/include/xen.h?
New proposal, XEN2/3_DOM0 builds fine with it: http://www.netbsd.org/~jym/MI-Xen-drivers.diff -- Jean-Yves Migeon jeanyves.migeon%free.fr@localhost