Changeset View
Changeset View
Standalone View
Standalone View
sys/x86/include/xen/xen-os.h
Show All 24 Lines | |||||
* DEALINGS IN THE SOFTWARE. | * DEALINGS IN THE SOFTWARE. | ||||
* | * | ||||
* $FreeBSD$ | * $FreeBSD$ | ||||
*/ | */ | ||||
#ifndef _MACHINE_X86_XEN_XEN_OS_H_ | #ifndef _MACHINE_X86_XEN_XEN_OS_H_ | ||||
#define _MACHINE_X86_XEN_XEN_OS_H_ | #define _MACHINE_X86_XEN_XEN_OS_H_ | ||||
/* x86 is lucky and can simply use the default */ | |||||
#define VM_MEMATTR_XEN_SHARED VM_MEMATTR_DEFAULT | |||||
royger: Better use VM_MEMATTR_WRITE_BACK explicitly here. Not that I think the default will ever change… | |||||
Done Inline ActionsNotice where this is, this is sys/x86/include/xen/xen-os.h. This is preserving existing behavior for x86. Though you could want to change the behavior for x86. The commit for sys/arm64/include/xen/xen-os.h was separate. ehem_freebsd_m5p.com: Notice where this is, this is sys/x86/include/xen/xen-os.h. This is preserving existing… | |||||
Not Done Inline ActionsVM_MEMATTR_DEFAULT is defined as VM_MEMATTR_WRITE_BACK so there is no behaviour change by using one or the other. mhorne: `VM_MEMATTR_DEFAULT` is defined as `VM_MEMATTR_WRITE_BACK` so there is no behaviour change by… | |||||
/* Everything below this point is not included by assembler (.S) files. */ | /* Everything below this point is not included by assembler (.S) files. */ | ||||
#ifndef __ASSEMBLY__ | #ifndef __ASSEMBLY__ | ||||
/* If non-zero, the hypervisor has been configured to use a direct vector */ | /* If non-zero, the hypervisor has been configured to use a direct vector */ | ||||
extern int xen_vector_callback_enabled; | extern int xen_vector_callback_enabled; | ||||
#endif /* !__ASSEMBLY__ */ | #endif /* !__ASSEMBLY__ */ | ||||
#endif /* _MACHINE_X86_XEN_XEN_OS_H_ */ | #endif /* _MACHINE_X86_XEN_XEN_OS_H_ */ |
Better use VM_MEMATTR_WRITE_BACK explicitly here. Not that I think the default will ever change, but better be safe than sorry.