Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-03-01 Thread Andrew Cooper
On 28/02/2023 7:11 am, Jan Beulich wrote: > On 28.02.2023 08:09, Xenia Ragiadakou wrote: >> On 2/27/23 14:17, Jan Beulich wrote: >>> On 27.02.2023 13:06, Andrew Cooper wrote: On 27/02/2023 11:33 am, Jan Beulich wrote: > On 27.02.2023 12:15, Andrew Cooper wrote: >> On 27/02/2023 10:46 a

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-27 Thread Jan Beulich
On 28.02.2023 08:09, Xenia Ragiadakou wrote: > > On 2/27/23 14:17, Jan Beulich wrote: >> On 27.02.2023 13:06, Andrew Cooper wrote: >>> On 27/02/2023 11:33 am, Jan Beulich wrote: On 27.02.2023 12:15, Andrew Cooper wrote: > On 27/02/2023 10:46 am, Jan Beulich wrote: >> On 24.02.2023 22:

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-27 Thread Xenia Ragiadakou
On 2/27/23 14:17, Jan Beulich wrote: On 27.02.2023 13:06, Andrew Cooper wrote: On 27/02/2023 11:33 am, Jan Beulich wrote: On 27.02.2023 12:15, Andrew Cooper wrote: On 27/02/2023 10:46 am, Jan Beulich wrote: On 24.02.2023 22:33, Andrew Cooper wrote: But I think we want to change tact slight

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-27 Thread Jan Beulich
On 27.02.2023 13:06, Andrew Cooper wrote: > On 27/02/2023 11:33 am, Jan Beulich wrote: >> On 27.02.2023 12:15, Andrew Cooper wrote: >>> On 27/02/2023 10:46 am, Jan Beulich wrote: On 24.02.2023 22:33, Andrew Cooper wrote: > But I think we want to change tact slightly at this point, so I'm n

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-27 Thread Andrew Cooper
On 27/02/2023 11:33 am, Jan Beulich wrote: > On 27.02.2023 12:15, Andrew Cooper wrote: >> On 27/02/2023 10:46 am, Jan Beulich wrote: >>> On 24.02.2023 22:33, Andrew Cooper wrote: But I think we want to change tact slightly at this point, so I'm not going to do any further tweaking on comm

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-27 Thread Jan Beulich
On 27.02.2023 12:15, Andrew Cooper wrote: > On 27/02/2023 10:46 am, Jan Beulich wrote: >> On 24.02.2023 22:33, Andrew Cooper wrote: >>> But I think we want to change tact slightly at this point, so I'm not >>> going to do any further tweaking on commit. >>> >>> Next, I think we want to rename asm/h

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-27 Thread Andrew Cooper
On 27/02/2023 10:46 am, Jan Beulich wrote: > On 24.02.2023 22:33, Andrew Cooper wrote: >> But I think we want to change tact slightly at this point, so I'm not >> going to do any further tweaking on commit. >> >> Next, I think we want to rename asm/hvm/svm/svm.h to asm/hvm/svm.h, >> updating the no

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-27 Thread Jan Beulich
On 24.02.2023 22:33, Andrew Cooper wrote: > But I think we want to change tact slightly at this point, so I'm not > going to do any further tweaking on commit. > > Next, I think we want to rename asm/hvm/svm/svm.h to asm/hvm/svm.h, > updating the non-SVM include paths as we go.  Probably best to >

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-24 Thread Xenia Ragiadakou
On 2/24/23 23:33, Andrew Cooper wrote: On 24/02/2023 8:08 pm, Xenia Ragiadakou wrote: On 2/24/23 21:29, Andrew Cooper wrote: On 24/02/2023 6:49 pm, Xenia Ragiadakou wrote: There are more detailed per-patch changesets. Xenia Ragiadakou (14):    x86/svm: move declarations used only by svm c

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-24 Thread Andrew Cooper
On 24/02/2023 8:08 pm, Xenia Ragiadakou wrote: > > On 2/24/23 21:29, Andrew Cooper wrote: >> On 24/02/2023 6:49 pm, Xenia Ragiadakou wrote: >>> There are more detailed per-patch changesets. >>> >>> Xenia Ragiadakou (14): >>>    x86/svm: move declarations used only by svm code from svm.h to >>> priv

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-24 Thread Xenia Ragiadakou
On 2/24/23 21:29, Andrew Cooper wrote: On 24/02/2023 6:49 pm, Xenia Ragiadakou wrote: There are more detailed per-patch changesets. Xenia Ragiadakou (14): x86/svm: move declarations used only by svm code from svm.h to private header x86/svm: make asid.h private x86/svm: delete h

Re: [PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-24 Thread Andrew Cooper
On 24/02/2023 6:49 pm, Xenia Ragiadakou wrote: > There are more detailed per-patch changesets. > > Xenia Ragiadakou (14): > x86/svm: move declarations used only by svm code from svm.h to private > header > x86/svm: make asid.h private > x86/svm: delete header asm/hvm/svm/intr.h Thankyou

[PATCH v3 00/14] x86/hvm: {svm,vmx} {c,h} cleanup

2023-02-24 Thread Xenia Ragiadakou
This patch series attempts a cleanup of files {svm,vmx} files and headers by removing redundant headers and sorting the rest, reducing the scope of declarations and definitions, moving functions used only by internal {svm,vmx} code to private headers, fix coding style, replace u* with uint*_t types