[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v6 02/10] xsm bits for vNUMA hypercalls

On 07/18/2014 09:50 AM, Konrad Rzeszutek Wilk wrote:
On Fri, Jul 18, 2014 at 01:50:01AM -0400, Elena Ufimtseva wrote:
Define xsm_get_vnumainfo hypercall used for domain which
wish to receive vnuma topology. Add xsm hook for
XEN_DOMCTL_setvnumainfo. Also adds basic policies.

In order to add basic policies, you should also modify the example
XSM policy in tools/flask/policy/policy/modules/xen/xen.{if,te} and add
the permission to the create_domain and/or manage_domain macros.
Otherwise, the commit message should only refer to adding XSM checks
(and the example policy will have to be modified later, so it's really
preferable to update them both now).

diff --git a/xen/include/xsm/xsm.h b/xen/include/xsm/xsm.h
index a85045d..c7ec562 100644
--- a/xen/include/xsm/xsm.h
+++ b/xen/include/xsm/xsm.h
@@ -169,6 +169,7 @@ struct xsm_operations {
      int (*unbind_pt_irq) (struct domain *d, struct xen_domctl_bind_pt_irq 
      int (*ioport_permission) (struct domain *d, uint32_t s, uint32_t e, 
uint8_t allow);
      int (*ioport_mapping) (struct domain *d, uint32_t s, uint32_t e, uint8_t 
+    int (*get_vnumainfo) (struct domain *d);

@@ -653,6 +654,12 @@ static inline int xsm_ioport_mapping (xsm_default_t def, 
struct domain *d, uint3
      return xsm_ops->ioport_mapping(d, s, e, allow);
+static inline int xsm_get_vnumainfo (xsm_default_t def, struct domain *d)
+    return xsm_ops->get_vnumainfo(d);
  #endif /* CONFIG_X86 */

Both of these need to be moved outside of #ifdef CONFIG_X86 since the
hook is not x86-specific. The other files' changes look correct.

Daniel De Graaf
National Security Agency

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.