Right, the original patch didn't break anything with PV domains. The case it doesn't handle is an HVM initial domain with an already-running Xenstore domain; I think this applies both to ARM and hybrid/PVH on x86. In that case, usage would be set to LOCAL instead of HVM.
Right, however if I am not mistaken there is no such thing as an HVM dom0 right now on x86 and hybrid/PVH is probably going to return true on xen_pv_domain() and false on xen_hvm_domain().
The other way around. HVM = true, PV = false.
Mukesh, correct me if I am wrong pls.
In the ARM case, given that we don't have a start_info page, we would need another way to figure out whether a xenstore stub domain is already running, so I think we can just postpone the solution of that problem for now.