On 06/09/18 10:08, Paul Durrant wrote: >> -----Original Message----- >> From: Andrew Cooper [mailto:[email protected]] >> Sent: 05 September 2018 19:12 >> To: Xen-devel <[email protected]> >> Cc: Andrew Cooper <[email protected]>; Jan Beulich >> <[email protected]>; Wei Liu <[email protected]>; Roger Pau Monne >> <[email protected]>; Paul Durrant <[email protected]>; Stefano >> Stabellini <[email protected]>; Julien Grall <[email protected]> >> Subject: [PATCH 2/5] x86/hvm: Switch hvm_allow_set_param() to use a >> whitelist >> >> There are holes in the HVM_PARAM space, some of which are from >> deprecated >> parameters, but toolstack and device models currently has (almost) blanket > s/has/have > >> write access. >> >> Rearrange hvm_allow_get_param() to have a whitelist of toolstack-writeable > s/get/set
Both fixed. >> d = rcu_lock_domain_by_any_id(a.domid); >> if ( d == NULL ) >> return -ESRCH; >> @@ -4209,15 +4228,7 @@ static int hvmop_set_param( >> case HVM_PARAM_ACPI_IOPORTS_LOCATION: >> rc = pmtimer_change_ioport(d, a.value); >> break; >> - case HVM_PARAM_MEMORY_EVENT_CR0: >> - case HVM_PARAM_MEMORY_EVENT_CR3: >> - case HVM_PARAM_MEMORY_EVENT_CR4: >> - case HVM_PARAM_MEMORY_EVENT_INT3: >> - case HVM_PARAM_MEMORY_EVENT_SINGLE_STEP: >> - case HVM_PARAM_MEMORY_EVENT_MSR: >> - /* Deprecated */ >> - rc = -EOPNOTSUPP; >> - break; > Does anything rely on this EOPNOTSUPP vs the EINVAL that would be returned > after this patch is applied? Nothing I can spot, although in searching, I see that xc_hvm_param_deprecated_check() is a thing. ~Andrew _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
