On Fri, Apr 19, 2024, vsntk18@xxxxxxxxx wrote:
> From: Vasant Karasulli <vkarasulli@xxxxxxx>
>
> This enables sharing common definitions across testcases and lib/.
>
> Signed-off-by: Varad Gautam <varad.gautam@xxxxxxxx>
> Signed-off-by: Vasant Karasulli <vkarasulli@xxxxxxx>
> Reviewed-by: Marc Orr <marcorr@xxxxxxxxxx>
> ---
> {x86 => lib/x86}/svm.h | 0
No, there is far, far more crud in svm.h than belongs in lib/. The architectural
definitions and whatnot belong in lib/, but all of the nSVM support code does not.
> x86/svm.c | 2 +-
> x86/svm_tests.c | 2 +-
> 3 files changed, 2 insertions(+), 2 deletions(-)
> rename {x86 => lib/x86}/svm.h (100%)
>
> diff --git a/x86/svm.h b/lib/x86/svm.h
> similarity index 100%
> rename from x86/svm.h
> rename to lib/x86/svm.h
> diff --git a/x86/svm.c b/x86/svm.c
> index e715e270..252d5301 100644
> --- a/x86/svm.c
> +++ b/x86/svm.c
> @@ -2,7 +2,7 @@
> * Framework for testing nested virtualization
> */
>
> -#include "svm.h"
> +#include "x86/svm.h"
> #include "libcflat.h"
> #include "processor.h"
> #include "desc.h"
> diff --git a/x86/svm_tests.c b/x86/svm_tests.c
> index c81b7465..a180939f 100644
> --- a/x86/svm_tests.c
> +++ b/x86/svm_tests.c
> @@ -1,4 +1,4 @@
> -#include "svm.h"
> +#include "x86/svm.h"
> #include "libcflat.h"
> #include "processor.h"
> #include "desc.h"
> --
> 2.34.1
>
