Re: [PATCH 2/7] KVM: Intel virtual mode extensions definitions

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

 



On Thu, 2006-10-19 at 15:48 +0200, Avi Kivity wrote:
> Add some constants for the various bits defined by Intel's VT extensions.
> 
> Most of this file was lifted from the Xen hypervisor.
> 
> Signed-off-by: Yaniv Kamay <[email protected]>
> Signed-off-by: Avi Kivity <[email protected]>
> 
> Index: linux-2.6/drivers/kvm/vmx.h
> ===================================================================
> --- /dev/null
> +++ linux-2.6/drivers/kvm/vmx.h
> @@ -0,0 +1,287 @@

This entire file is also very specific to an architecture. Couldn't it
be put somewhere in arch/x86_64 and not in drivers?

I know that this is all currently focused on Intel and AMD
virtualization platforms, but could you split out the x86_64 specific
stuff and make the rest more generic. Perhaps in the future this will
make it easier for other platforms to use this code as well.

It's hard to do a generic approach when developing it new, but if you
don't think about that now, it will be magnitudes larger in difficulty
to make generic when this is all done.

-- Steve



> purpose register */
> +#define LMSW_SOURCE_DATA_SHIFT 16
> +#define LMSW_SOURCE_DATA  (0xFFFF << LMSW_SOURCE_DATA_SHIFT) /* 16:31
> lmsw source */
> +#define REG_EAX                         (0 << 8)
> +#define REG_ECX                         (1 << 8)
> +#define REG_EDX                         (2 << 8)
> +#define REG_EBX                         (3 << 8)
> +#define REG_ESP                         (4 << 8)
> +#define REG_EBP                         (5 << 8)
> +#define REG_ESI                         (6 << 8)
> +#define REG_EDI                         (7 << 8)
> +#define REG_R8                         (8 << 8)
> +#define REG_R9                         (9 << 8)
> +#define REG_R10                        (10 << 8)
> +#define REG_R11                        (11 << 8)
> +#define REG_R12                        (12 << 8)
> +#define REG_R13                        (13 << 8)
> +#define REG_R14                        (14 << 8)
> +#define REG_R15                        (15 << 8)
> +


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux