On Thu, Jan 09, 2020 at 06:49:07PM -0500, Boris Ostrovsky wrote: > > > On 1/9/20 6:46 PM, Boris Ostrovsky wrote: > > > > > >On 1/7/20 6:37 PM, Anchal Agarwal wrote: > >>+ > >>+static int xen_setup_pm_notifier(void) > >>+{ > >>+ if (!xen_hvm_domain()) > >>+ return -ENODEV; > > > >ARM guests are also HVM domains. Is it OK for them to register the > >notifier? The diffstat suggests that you are supporting ARM. > > I obviously meant *not* supporting ARM, sorry. > > -boris > > > > >-boris > > TBH, I have not yet experimented with these patches on ARM guest yet but that will be the next step. The same code with changes as needed should be made to work for ARM. Currently I am focussed on getting a sane set of patches into mainline for x86 guests. Thanks, Anchal > >>+ > >>+ return register_pm_notifier(&xen_pm_notifier_block); > >>+} > >> > > >