| Message ID | 20200122202343.5703-4-liuwe@microsoft.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | More Hyper-V infrastructure | expand |
On 22/01/2020 20:23, Wei Liu wrote: > These functions will be used later to make hypercalls to Hyper-V. > > Signed-off-by: Wei Liu <liuwe@microsoft.com> After some experimentation, diff --git a/xen/arch/x86/xen.lds.S b/xen/arch/x86/xen.lds.S index cbc5701214..3708a60b5c 100644 --- a/xen/arch/x86/xen.lds.S +++ b/xen/arch/x86/xen.lds.S @@ -329,6 +329,8 @@ SECTIONS efi = .; #endif + hv_hcall_page = ABSOLUTE(0xffff82d0bfffe000); + /* Sections to be discarded */ /DISCARD/ : { *(.exit.text) in the linker script lets direct calls work correctly: ffff82d080637935: b9 01 00 00 40 mov $0x40000001,%ecx ffff82d08063793a: 0f 30 wrmsr ffff82d08063793c: ba 21 03 00 00 mov $0x321,%edx ffff82d080637941: bf 01 00 00 00 mov $0x1,%edi ffff82d080637946: e8 ac 4f c7 ff callq ffff82d0802ac8f7 <__set_fixmap_x> ffff82d08063794b: 41 b8 00 00 00 00 mov $0x0,%r8d ffff82d080637951: b9 ff ff 00 00 mov $0xffff,%ecx ffff82d080637956: ba 00 00 00 00 mov $0x0,%edx ffff82d08063795b: e8 a0 66 9c 3f callq ffff82d0bfffe000 <hv_hcall_page> ffff82d080637960: 66 83 f8 02 cmp $0x2,%ax but it does throw: Difference at .init:00032edf is 0xc0000000 (expected 0x40000000) Difference at .init:00032edf is 0xc0000000 (expected 0x40000000) as a diagnostic presumably from the final link (both with a standard Debian 2.28 binutils, and upstream 2.33 build). I'm not sure what its trying to complain about, as both xen.gz and xen.efi have correctly generated code. Depending on whether they are benign or not, a linker-friendly fix_to_virt() should be all we need to keep these strictly as direct calls. ~Andrew
On 22.01.2020 22:57, Andrew Cooper wrote: > On 22/01/2020 20:23, Wei Liu wrote: >> These functions will be used later to make hypercalls to Hyper-V. >> >> Signed-off-by: Wei Liu <liuwe@microsoft.com> > > After some experimentation, > > diff --git a/xen/arch/x86/xen.lds.S b/xen/arch/x86/xen.lds.S > index cbc5701214..3708a60b5c 100644 > --- a/xen/arch/x86/xen.lds.S > +++ b/xen/arch/x86/xen.lds.S > @@ -329,6 +329,8 @@ SECTIONS > efi = .; > #endif > > + hv_hcall_page = ABSOLUTE(0xffff82d0bfffe000); > + > /* Sections to be discarded */ > /DISCARD/ : { > *(.exit.text) > > in the linker script lets direct calls work correctly: > > ffff82d080637935: b9 01 00 00 40 mov $0x40000001,%ecx > ffff82d08063793a: 0f 30 wrmsr > ffff82d08063793c: ba 21 03 00 00 mov $0x321,%edx > ffff82d080637941: bf 01 00 00 00 mov $0x1,%edi > ffff82d080637946: e8 ac 4f c7 ff callq ffff82d0802ac8f7 > <__set_fixmap_x> > ffff82d08063794b: 41 b8 00 00 00 00 mov $0x0,%r8d > ffff82d080637951: b9 ff ff 00 00 mov $0xffff,%ecx > ffff82d080637956: ba 00 00 00 00 mov $0x0,%edx > ffff82d08063795b: e8 a0 66 9c 3f callq ffff82d0bfffe000 > <hv_hcall_page> > ffff82d080637960: 66 83 f8 02 cmp $0x2,%ax > > but it does throw: > > Difference at .init:00032edf is 0xc0000000 (expected 0x40000000) > Difference at .init:00032edf is 0xc0000000 (expected 0x40000000) > > as a diagnostic presumably from the final link (both with a standard > Debian 2.28 binutils, and upstream 2.33 build). I'm not sure what its > trying to complain about, as both xen.gz and xen.efi have correctly > generated code. > > Depending on whether they are benign or not, a linker-friendly > fix_to_virt() should be all we need to keep these strictly as direct calls. They're benign in the particular case of them actually resulting from relative CALLs, which hence require no relocation to be recorded in xen.efi's .reloc section. But they should nevertheless be silenced. We've been discussing this on irc, iirc. The absolute address used wants to move by 1Gb for the $(ALT_BASE) intermediate linking step. Jan
On 22.01.2020 21:23, Wei Liu wrote: > --- /dev/null > +++ b/xen/include/asm-x86/guest/hyperv-hcall.h > @@ -0,0 +1,98 @@ > +/****************************************************************************** > + * asm-x86/guest/hyperv-hcall.h > + * > + * This program is free software; you can redistribute it and/or > + * modify it under the terms and conditions of the GNU General Public > + * License, version 2, as published by the Free Software Foundation. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > + * General Public License for more details. > + * > + * You should have received a copy of the GNU General Public > + * License along with this program; If not, see <http://www.gnu.org/licenses/>. > + * > + * Copyright (c) 2019 Microsoft. > + */ > + > +#ifndef __X86_HYPERV_HCALL_H__ > +#define __X86_HYPERV_HCALL_H__ > + > +#include <xen/lib.h> > +#include <xen/types.h> > + > +#include <asm/asm_defns.h> > +#include <asm/fixmap.h> > +#include <asm/guest/hyperv-tlfs.h> > +#include <asm/page.h> > + > +static inline uint64_t hv_do_hypercall(uint64_t control, paddr_t input_addr, > + paddr_t output_addr) > +{ > + uint64_t status; > + register unsigned long r8 asm("r8") = output_addr; > + > + asm volatile ("INDIRECT_CALL %P[hcall_page]" > + : "=a" (status), "+c" (control), > + "+d" (input_addr) ASM_CALL_CONSTRAINT > + : "r" (r8), > + [hcall_page] "p" (fix_x_to_virt(FIX_X_HYPERV_HCALL)) > + : "memory"); > + > + return status; > +} > + > +static inline uint64_t hv_do_fast_hypercall(uint16_t code, > + uint64_t input1, uint64_t input2) > +{ > + uint64_t status; > + uint64_t control = code | HV_HYPERCALL_FAST_BIT; > + register unsigned long r8 asm("r8") = input2; > + > + asm volatile ("INDIRECT_CALL %P[hcall_page]" > + : "=a" (status), "+c" (control), > + "+d" (input1) ASM_CALL_CONSTRAINT > + : "r" (r8), > + [hcall_page] "p" (fix_x_to_virt(FIX_X_HYPERV_HCALL)) > + :); This comes through as a smiley in my mail viewer, because of the missing blanks immediately inside the outermost parentheses. > + > + return status; > +} > + > +static inline uint64_t hv_do_rep_hypercall(uint16_t code, uint16_t rep_count, > + uint16_t varhead_size, > + paddr_t input, paddr_t output) > +{ > + uint64_t control = code; > + uint64_t status; > + uint16_t rep_comp; > + > + control |= (uint64_t)varhead_size << HV_HYPERCALL_VARHEAD_OFFSET; > + control |= (uint64_t)rep_count << HV_HYPERCALL_REP_COMP_OFFSET; What about the upper bit(s) spilling into the next field? Perhaps better use MASK_INSR() here too? Also, this leaves the START field zero, which makes me think you mean ... > + do { > + status = hv_do_hypercall(control, input, output); > + if ( (status & HV_HYPERCALL_RESULT_MASK) != HV_STATUS_SUCCESS ) > + break; > + > + rep_comp = MASK_EXTR(status, HV_HYPERCALL_REP_COMP_MASK); > + > + control &= ~HV_HYPERCALL_REP_START_MASK; > + control |= MASK_INSR(rep_comp, HV_HYPERCALL_REP_COMP_MASK); ... REP_START_MASK here. Jan
On Thu, Jan 23, 2020 at 11:13:10AM +0100, Jan Beulich wrote: > On 22.01.2020 22:57, Andrew Cooper wrote: > > On 22/01/2020 20:23, Wei Liu wrote: > >> These functions will be used later to make hypercalls to Hyper-V. > >> > >> Signed-off-by: Wei Liu <liuwe@microsoft.com> > > > > After some experimentation, > > > > diff --git a/xen/arch/x86/xen.lds.S b/xen/arch/x86/xen.lds.S > > index cbc5701214..3708a60b5c 100644 > > --- a/xen/arch/x86/xen.lds.S > > +++ b/xen/arch/x86/xen.lds.S > > @@ -329,6 +329,8 @@ SECTIONS > > efi = .; > > #endif > > > > + hv_hcall_page = ABSOLUTE(0xffff82d0bfffe000); > > + > > /* Sections to be discarded */ > > /DISCARD/ : { > > *(.exit.text) > > > > in the linker script lets direct calls work correctly: > > > > ffff82d080637935: b9 01 00 00 40 mov $0x40000001,%ecx > > ffff82d08063793a: 0f 30 wrmsr > > ffff82d08063793c: ba 21 03 00 00 mov $0x321,%edx > > ffff82d080637941: bf 01 00 00 00 mov $0x1,%edi > > ffff82d080637946: e8 ac 4f c7 ff callq ffff82d0802ac8f7 > > <__set_fixmap_x> > > ffff82d08063794b: 41 b8 00 00 00 00 mov $0x0,%r8d > > ffff82d080637951: b9 ff ff 00 00 mov $0xffff,%ecx > > ffff82d080637956: ba 00 00 00 00 mov $0x0,%edx > > ffff82d08063795b: e8 a0 66 9c 3f callq ffff82d0bfffe000 > > <hv_hcall_page> > > ffff82d080637960: 66 83 f8 02 cmp $0x2,%ax > > > > but it does throw: > > > > Difference at .init:00032edf is 0xc0000000 (expected 0x40000000) > > Difference at .init:00032edf is 0xc0000000 (expected 0x40000000) > > > > as a diagnostic presumably from the final link (both with a standard > > Debian 2.28 binutils, and upstream 2.33 build). I'm not sure what its > > trying to complain about, as both xen.gz and xen.efi have correctly > > generated code. > > > > Depending on whether they are benign or not, a linker-friendly > > fix_to_virt() should be all we need to keep these strictly as direct calls. > > They're benign in the particular case of them actually resulting > from relative CALLs, which hence require no relocation to be > recorded in xen.efi's .reloc section. But they should nevertheless > be silenced. We've been discussing this on irc, iirc. The absolute > address used wants to move by 1Gb for the $(ALT_BASE) intermediate > linking step. FWIW I don't see those messages with my current code. Wei. > > Jan
On Thu, Jan 23, 2020 at 12:28:00PM +0100, Jan Beulich wrote: > On 22.01.2020 21:23, Wei Liu wrote: > > --- /dev/null > > +++ b/xen/include/asm-x86/guest/hyperv-hcall.h > > @@ -0,0 +1,98 @@ > > +/****************************************************************************** > > + * asm-x86/guest/hyperv-hcall.h > > + * > > + * This program is free software; you can redistribute it and/or > > + * modify it under the terms and conditions of the GNU General Public > > + * License, version 2, as published by the Free Software Foundation. > > + * > > + * This program is distributed in the hope that it will be useful, > > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > + * General Public License for more details. > > + * > > + * You should have received a copy of the GNU General Public > > + * License along with this program; If not, see <http://www.gnu.org/licenses/>. > > + * > > + * Copyright (c) 2019 Microsoft. > > + */ > > + > > +#ifndef __X86_HYPERV_HCALL_H__ > > +#define __X86_HYPERV_HCALL_H__ > > + > > +#include <xen/lib.h> > > +#include <xen/types.h> > > + > > +#include <asm/asm_defns.h> > > +#include <asm/fixmap.h> > > +#include <asm/guest/hyperv-tlfs.h> > > +#include <asm/page.h> > > + > > +static inline uint64_t hv_do_hypercall(uint64_t control, paddr_t input_addr, > > + paddr_t output_addr) > > +{ > > + uint64_t status; > > + register unsigned long r8 asm("r8") = output_addr; > > + > > + asm volatile ("INDIRECT_CALL %P[hcall_page]" > > + : "=a" (status), "+c" (control), > > + "+d" (input_addr) ASM_CALL_CONSTRAINT > > + : "r" (r8), > > + [hcall_page] "p" (fix_x_to_virt(FIX_X_HYPERV_HCALL)) > > + : "memory"); > > + > > + return status; > > +} > > + > > +static inline uint64_t hv_do_fast_hypercall(uint16_t code, > > + uint64_t input1, uint64_t input2) > > +{ > > + uint64_t status; > > + uint64_t control = code | HV_HYPERCALL_FAST_BIT; > > + register unsigned long r8 asm("r8") = input2; > > + > > + asm volatile ("INDIRECT_CALL %P[hcall_page]" > > + : "=a" (status), "+c" (control), > > + "+d" (input1) ASM_CALL_CONSTRAINT > > + : "r" (r8), > > + [hcall_page] "p" (fix_x_to_virt(FIX_X_HYPERV_HCALL)) > > + :); > > This comes through as a smiley in my mail viewer, because of the > missing blanks immediately inside the outermost parentheses. Fixed. > > > + > > + return status; > > +} > > + > > +static inline uint64_t hv_do_rep_hypercall(uint16_t code, uint16_t rep_count, > > + uint16_t varhead_size, > > + paddr_t input, paddr_t output) > > +{ > > + uint64_t control = code; > > + uint64_t status; > > + uint16_t rep_comp; > > + > > + control |= (uint64_t)varhead_size << HV_HYPERCALL_VARHEAD_OFFSET; > > + control |= (uint64_t)rep_count << HV_HYPERCALL_REP_COMP_OFFSET; > > What about the upper bit(s) spilling into the next field? Perhaps > better use MASK_INSR() here too? > I didn't do it because we didn't have the two MASKs defined. Plus I trusted Hyper-V to reject what's abnormal so I wouldn't worry too much about it. > Also, this leaves the START field zero, which makes me think you > mean ... > > > + do { > > + status = hv_do_hypercall(control, input, output); > > + if ( (status & HV_HYPERCALL_RESULT_MASK) != HV_STATUS_SUCCESS ) > > + break; > > + > > + rep_comp = MASK_EXTR(status, HV_HYPERCALL_REP_COMP_MASK); > > + > > + control &= ~HV_HYPERCALL_REP_START_MASK; > > + control |= MASK_INSR(rep_comp, HV_HYPERCALL_REP_COMP_MASK); > > ... REP_START_MASK here. Yes, indeed. Fixed. Wei. > > Jan
On 29.01.2020 19:37, Wei Liu wrote: > On Thu, Jan 23, 2020 at 12:28:00PM +0100, Jan Beulich wrote: >> On 22.01.2020 21:23, Wei Liu wrote: >>> --- /dev/null >>> +++ b/xen/include/asm-x86/guest/hyperv-hcall.h >>> @@ -0,0 +1,98 @@ >>> +/****************************************************************************** >>> + * asm-x86/guest/hyperv-hcall.h >>> + * >>> + * This program is free software; you can redistribute it and/or >>> + * modify it under the terms and conditions of the GNU General Public >>> + * License, version 2, as published by the Free Software Foundation. >>> + * >>> + * This program is distributed in the hope that it will be useful, >>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of >>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU >>> + * General Public License for more details. >>> + * >>> + * You should have received a copy of the GNU General Public >>> + * License along with this program; If not, see <http://www.gnu.org/licenses/>. >>> + * >>> + * Copyright (c) 2019 Microsoft. >>> + */ >>> + >>> +#ifndef __X86_HYPERV_HCALL_H__ >>> +#define __X86_HYPERV_HCALL_H__ >>> + >>> +#include <xen/lib.h> >>> +#include <xen/types.h> >>> + >>> +#include <asm/asm_defns.h> >>> +#include <asm/fixmap.h> >>> +#include <asm/guest/hyperv-tlfs.h> >>> +#include <asm/page.h> >>> + >>> +static inline uint64_t hv_do_hypercall(uint64_t control, paddr_t input_addr, >>> + paddr_t output_addr) >>> +{ >>> + uint64_t status; >>> + register unsigned long r8 asm("r8") = output_addr; >>> + >>> + asm volatile ("INDIRECT_CALL %P[hcall_page]" >>> + : "=a" (status), "+c" (control), >>> + "+d" (input_addr) ASM_CALL_CONSTRAINT >>> + : "r" (r8), >>> + [hcall_page] "p" (fix_x_to_virt(FIX_X_HYPERV_HCALL)) >>> + : "memory"); >>> + >>> + return status; >>> +} >>> + >>> +static inline uint64_t hv_do_fast_hypercall(uint16_t code, >>> + uint64_t input1, uint64_t input2) >>> +{ >>> + uint64_t status; >>> + uint64_t control = code | HV_HYPERCALL_FAST_BIT; >>> + register unsigned long r8 asm("r8") = input2; >>> + >>> + asm volatile ("INDIRECT_CALL %P[hcall_page]" >>> + : "=a" (status), "+c" (control), >>> + "+d" (input1) ASM_CALL_CONSTRAINT >>> + : "r" (r8), >>> + [hcall_page] "p" (fix_x_to_virt(FIX_X_HYPERV_HCALL)) >>> + :); >> >> This comes through as a smiley in my mail viewer, because of the >> missing blanks immediately inside the outermost parentheses. > > Fixed. By dropping the :, I assume? My suggestion of just the missing blank may have been misleading here (albeit the blanks still need adding). Jan
On Thu, Jan 30, 2020 at 09:12:14AM +0100, Jan Beulich wrote: > On 29.01.2020 19:37, Wei Liu wrote: > > On Thu, Jan 23, 2020 at 12:28:00PM +0100, Jan Beulich wrote: > >> On 22.01.2020 21:23, Wei Liu wrote: > >>> --- /dev/null > >>> +++ b/xen/include/asm-x86/guest/hyperv-hcall.h > >>> @@ -0,0 +1,98 @@ > >>> +/****************************************************************************** > >>> + * asm-x86/guest/hyperv-hcall.h > >>> + * > >>> + * This program is free software; you can redistribute it and/or > >>> + * modify it under the terms and conditions of the GNU General Public > >>> + * License, version 2, as published by the Free Software Foundation. > >>> + * > >>> + * This program is distributed in the hope that it will be useful, > >>> + * but WITHOUT ANY WARRANTY; without even the implied warranty of > >>> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > >>> + * General Public License for more details. > >>> + * > >>> + * You should have received a copy of the GNU General Public > >>> + * License along with this program; If not, see <http://www.gnu.org/licenses/>. > >>> + * > >>> + * Copyright (c) 2019 Microsoft. > >>> + */ > >>> + > >>> +#ifndef __X86_HYPERV_HCALL_H__ > >>> +#define __X86_HYPERV_HCALL_H__ > >>> + > >>> +#include <xen/lib.h> > >>> +#include <xen/types.h> > >>> + > >>> +#include <asm/asm_defns.h> > >>> +#include <asm/fixmap.h> > >>> +#include <asm/guest/hyperv-tlfs.h> > >>> +#include <asm/page.h> > >>> + > >>> +static inline uint64_t hv_do_hypercall(uint64_t control, paddr_t input_addr, > >>> + paddr_t output_addr) > >>> +{ > >>> + uint64_t status; > >>> + register unsigned long r8 asm("r8") = output_addr; > >>> + > >>> + asm volatile ("INDIRECT_CALL %P[hcall_page]" > >>> + : "=a" (status), "+c" (control), > >>> + "+d" (input_addr) ASM_CALL_CONSTRAINT > >>> + : "r" (r8), > >>> + [hcall_page] "p" (fix_x_to_virt(FIX_X_HYPERV_HCALL)) > >>> + : "memory"); > >>> + > >>> + return status; > >>> +} > >>> + > >>> +static inline uint64_t hv_do_fast_hypercall(uint16_t code, > >>> + uint64_t input1, uint64_t input2) > >>> +{ > >>> + uint64_t status; > >>> + uint64_t control = code | HV_HYPERCALL_FAST_BIT; > >>> + register unsigned long r8 asm("r8") = input2; > >>> + > >>> + asm volatile ("INDIRECT_CALL %P[hcall_page]" > >>> + : "=a" (status), "+c" (control), > >>> + "+d" (input1) ASM_CALL_CONSTRAINT > >>> + : "r" (r8), > >>> + [hcall_page] "p" (fix_x_to_virt(FIX_X_HYPERV_HCALL)) > >>> + :); > >> > >> This comes through as a smiley in my mail viewer, because of the > >> missing blanks immediately inside the outermost parentheses. > > > > Fixed. > > By dropping the :, I assume? My suggestion of just the missing blank > may have been misleading here (albeit the blanks still need adding). No, by adding a blank. I will drop that ":" locally in my tree for next round of posting. Wei. > > Jan
diff --git a/xen/include/asm-x86/guest/hyperv-hcall.h b/xen/include/asm-x86/guest/hyperv-hcall.h new file mode 100644 index 0000000000..509e57f481 --- /dev/null +++ b/xen/include/asm-x86/guest/hyperv-hcall.h @@ -0,0 +1,98 @@ +/****************************************************************************** + * asm-x86/guest/hyperv-hcall.h + * + * This program is free software; you can redistribute it and/or + * modify it under the terms and conditions of the GNU General Public + * License, version 2, as published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public + * License along with this program; If not, see <http://www.gnu.org/licenses/>. + * + * Copyright (c) 2019 Microsoft. + */ + +#ifndef __X86_HYPERV_HCALL_H__ +#define __X86_HYPERV_HCALL_H__ + +#include <xen/lib.h> +#include <xen/types.h> + +#include <asm/asm_defns.h> +#include <asm/fixmap.h> +#include <asm/guest/hyperv-tlfs.h> +#include <asm/page.h> + +static inline uint64_t hv_do_hypercall(uint64_t control, paddr_t input_addr, + paddr_t output_addr) +{ + uint64_t status; + register unsigned long r8 asm("r8") = output_addr; + + asm volatile ("INDIRECT_CALL %P[hcall_page]" + : "=a" (status), "+c" (control), + "+d" (input_addr) ASM_CALL_CONSTRAINT + : "r" (r8), + [hcall_page] "p" (fix_x_to_virt(FIX_X_HYPERV_HCALL)) + : "memory"); + + return status; +} + +static inline uint64_t hv_do_fast_hypercall(uint16_t code, + uint64_t input1, uint64_t input2) +{ + uint64_t status; + uint64_t control = code | HV_HYPERCALL_FAST_BIT; + register unsigned long r8 asm("r8") = input2; + + asm volatile ("INDIRECT_CALL %P[hcall_page]" + : "=a" (status), "+c" (control), + "+d" (input1) ASM_CALL_CONSTRAINT + : "r" (r8), + [hcall_page] "p" (fix_x_to_virt(FIX_X_HYPERV_HCALL)) + :); + + return status; +} + +static inline uint64_t hv_do_rep_hypercall(uint16_t code, uint16_t rep_count, + uint16_t varhead_size, + paddr_t input, paddr_t output) +{ + uint64_t control = code; + uint64_t status; + uint16_t rep_comp; + + control |= (uint64_t)varhead_size << HV_HYPERCALL_VARHEAD_OFFSET; + control |= (uint64_t)rep_count << HV_HYPERCALL_REP_COMP_OFFSET; + + do { + status = hv_do_hypercall(control, input, output); + if ( (status & HV_HYPERCALL_RESULT_MASK) != HV_STATUS_SUCCESS ) + break; + + rep_comp = MASK_EXTR(status, HV_HYPERCALL_REP_COMP_MASK); + + control &= ~HV_HYPERCALL_REP_START_MASK; + control |= MASK_INSR(rep_comp, HV_HYPERCALL_REP_COMP_MASK); + } while ( rep_comp < rep_count ); + + return status; +} + +#endif /* __X86_HYPERV_HCALL_H__ */ + +/* + * Local variables: + * mode: C + * c-file-style: "BSD" + * c-basic-offset: 4 + * tab-width: 4 + * indent-tabs-mode: nil + * End: + */
These functions will be used later to make hypercalls to Hyper-V. Signed-off-by: Wei Liu <liuwe@microsoft.com> --- v4: 1. Adjust code due to previous patch has changed 2. Address comments --- xen/include/asm-x86/guest/hyperv-hcall.h | 98 ++++++++++++++++++++++++ 1 file changed, 98 insertions(+) create mode 100644 xen/include/asm-x86/guest/hyperv-hcall.h