On Tue, 2018-09-18 at 08:03 +0200, Juergen Gross wrote: > Add a new domctl for setting domain specific parameters similar to > XEN_SYSCTL_set_parameter for global hypervisor parameters. > > Enhance XEN_SYSCTL_set_parameter to be usable for setting cpupool > specific parameters, too. For now do only extended parameter > checking. > The cpupool parameter setting will be added later. > > Signed-off-by: Juergen Gross <[email protected]> > Reviewed-by: Dario Faggioli <[email protected]>
I mean...
> --- a/xen/include/public/sysctl.h
> +++ b/xen/include/public/sysctl.h
> @@ -1055,12 +1055,18 @@ struct xen_sysctl_livepatch_op {
> * Parameters are a single string terminated by a NUL byte of max.
> size
> * characters. Multiple settings can be specified by separating them
> * with blanks.
> + * Scope can be either global (like boot parameters) or cpupool.
> */
>
> struct xen_sysctl_set_parameter {
> XEN_GUEST_HANDLE_64(char) params; /* IN: pointer to
> parameters. */
> uint16_t size; /* IN: size of
> parameters. */
> - uint16_t pad[3]; /* IN: MUST be zero. */
> + uint8_t scope; /* IN: scope of
> parameters. */
> +#define XEN_SYSCTL_SETPAR_SCOPE_GLOBAL 0
> +#define XEN_SYSCTL_SETPAR_SCOPE_CPUPOOL 1
> + uint8_t pad; /* IN: MUST be zero. */
> + uint32_t instance; /* IN: scope global:
> must be zero */
> + /* scope cpupool:
> cpupool id */
>
... I don't particularly like the name 'instance', but I've not been
able to come up with anything else which sounds better... :-/
Regards,
Dario
--
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Software Engineer @ SUSE https://www.suse.com/
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
