summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorgrehan <grehan@FreeBSD.org>2011-05-19 21:53:25 +0000
committergrehan <grehan@FreeBSD.org>2011-05-19 21:53:25 +0000
commit5a44aef8a38d88c0d5b265a72c5d90ef2d73cf95 (patch)
tree025b0a5578ec9688795c45d9e672d8faf8d42845
parent692383a3a7117fc9a24f086a4638739dacdc8d73 (diff)
downloadFreeBSD-src-5a44aef8a38d88c0d5b265a72c5d90ef2d73cf95.zip
FreeBSD-src-5a44aef8a38d88c0d5b265a72c5d90ef2d73cf95.tar.gz
Changes to allow the GENERIC+bhye kernel built from this branch to
run as a 1/2 CPU guest on an 8.1 bhyve host. bhyve/inout.c inout.h fbsdrun.c - Rather than exiting on accesses to unhandled i/o ports, emulate hardware by returning -1 on reads and ignoring writes to unhandled ports. Support the previous mode by allowing a 'strict' parameter to be set from the command line. The 8.1 guest kernel was vastly cut down from GENERIC and had no ISA devices. Booting GENERIC exposes a massive amount of random touching of i/o ports (hello syscons/vga/atkbdc). bhyve/consport.c dev/bvm/bvm_console.c - implement a simplistic signature for the bvm console by returning 'bv' for an inw on the port. Also, set the priority of the console to CN_REMOTE if the signature was returned. This works better in an environment where multiple consoles are in the kernel (hello syscons) bhyve/rtc.c - return 0 for the access to RTC_EQUIPMENT (yes, you syscons) amd64/vmm/x86.c x86.h - hide a bunch more CPUID leaf 1 bits from the guest to prevent cpufreq drivers from probing. The next step will be to move CPUID handling completely into user-space. This will allow the full spectrum of changes from presenting a lowest-common-denominator CPU type/feature set, to exposing (almost) everything that the host can support. Reviewed by: neel Obtained from: NetApp
-rw-r--r--sys/amd64/vmm/x86.c21
-rw-r--r--sys/amd64/vmm/x86.h1
-rw-r--r--sys/dev/bvm/bvm_console.c16
-rw-r--r--usr.sbin/bhyve/consport.c6
-rw-r--r--usr.sbin/bhyve/fbsdrun.c12
-rw-r--r--usr.sbin/bhyve/inout.c41
-rw-r--r--usr.sbin/bhyve/inout.h2
-rw-r--r--usr.sbin/bhyve/rtc.c6
8 files changed, 91 insertions, 14 deletions
diff --git a/sys/amd64/vmm/x86.c b/sys/amd64/vmm/x86.c
index 45c4c53..f6b38e0 100644
--- a/sys/amd64/vmm/x86.c
+++ b/sys/amd64/vmm/x86.c
@@ -75,13 +75,19 @@ x86_emulate_cpuid(uint32_t *eax, uint32_t *ebx, uint32_t *ecx, uint32_t *edx)
regs[1] |= (0 << CPUID_0000_0001_APICID_SHIFT);
/*
- * Don't expose VMX capability.
+ * Don't expose VMX, SpeedStep or TME capability.
* Advertise x2APIC capability.
*/
- regs[2] &= ~CPUID_0000_0001_FEAT0_VMX;
+ regs[2] &= ~(CPUID_0000_0001_FEAT0_VMX | CPUID2_EST |
+ CPUID2_TM2);
regs[2] |= CPUID2_X2APIC;
/*
+ * Hide thermal monitoring
+ */
+ regs[3] &= ~(CPUID_ACPI | CPUID_TM);
+
+ /*
* Machine check handling is done in the host.
* Hide MTRR capability.
*/
@@ -89,6 +95,17 @@ x86_emulate_cpuid(uint32_t *eax, uint32_t *ebx, uint32_t *ecx, uint32_t *edx)
break;
+ case CPUID_0000_0006:
+ /*
+ * Handle the access, but report 0 for
+ * all options
+ */
+ regs[0] = 0;
+ regs[1] = 0;
+ regs[2] = 0;
+ regs[3] = 0;
+ break;
+
case CPUID_0000_000B:
/*
* XXXSMP fixme
diff --git a/sys/amd64/vmm/x86.h b/sys/amd64/vmm/x86.h
index bc4f8a4..b437d61 100644
--- a/sys/amd64/vmm/x86.h
+++ b/sys/amd64/vmm/x86.h
@@ -34,6 +34,7 @@
#define CPUID_0000_0002 (0x2)
#define CPUID_0000_0003 (0x3)
#define CPUID_0000_0004 (0x4)
+#define CPUID_0000_0006 (0x6)
#define CPUID_0000_000A (0xA)
#define CPUID_0000_000B (0xB)
#define CPUID_8000_0000 (0x80000000)
diff --git a/sys/dev/bvm/bvm_console.c b/sys/dev/bvm/bvm_console.c
index ac12b6f..392346e 100644
--- a/sys/dev/bvm/bvm_console.c
+++ b/sys/dev/bvm/bvm_console.c
@@ -70,6 +70,8 @@ static int alt_break_state;
#define BVM_CONS_PORT 0x220
static int bvm_cons_port = BVM_CONS_PORT;
+#define BVM_CONS_SIG ('b' << 8 | 'v')
+
static void bvm_timeout(void *);
static cn_probe_t bvm_cnprobe;
@@ -171,14 +173,16 @@ bvm_cnprobe(struct consdev *cp)
int disabled, port;
disabled = 0;
+ cp->cn_pri = CN_DEAD;
+
resource_int_value("bvmconsole", 0, "disabled", &disabled);
- if (disabled)
- cp->cn_pri = CN_DEAD;
- else
- cp->cn_pri = CN_NORMAL;
+ if (!disabled) {
+ if (resource_int_value("bvmconsole", 0, "port", &port) == 0)
+ bvm_cons_port = port;
- if (resource_int_value("bvmconsole", 0, "port", &port) == 0)
- bvm_cons_port = port;
+ if (inw(bvm_cons_port) == BVM_CONS_SIG)
+ cp->cn_pri = CN_REMOTE;
+ }
}
static void
diff --git a/usr.sbin/bhyve/consport.c b/usr.sbin/bhyve/consport.c
index 34f94a6..8e2156a 100644
--- a/usr.sbin/bhyve/consport.c
+++ b/usr.sbin/bhyve/consport.c
@@ -41,6 +41,7 @@ __FBSDID("$FreeBSD$");
#include "inout.h"
#define BVM_CONSOLE_PORT 0x220
+#define BVM_CONS_SIG ('b' << 8 | 'v')
static struct termios tio_orig, tio_new;
@@ -103,6 +104,11 @@ console_handler(struct vmctx *ctx, int vcpu, int in, int port, int bytes,
{
static int opened;
+ if (bytes == 2 && in) {
+ *eax = BVM_CONS_SIG;
+ return (0);
+ }
+
if (bytes != 4)
return (-1);
diff --git a/usr.sbin/bhyve/fbsdrun.c b/usr.sbin/bhyve/fbsdrun.c
index 6234958..6f009b5 100644
--- a/usr.sbin/bhyve/fbsdrun.c
+++ b/usr.sbin/bhyve/fbsdrun.c
@@ -85,6 +85,8 @@ static int guest_vmexit_on_hlt, guest_vmexit_on_pause;
static int foundcpus;
+static int strictio;
+
static char *lomem_addr;
static char *himem_addr;
@@ -122,7 +124,7 @@ usage(int code)
{
fprintf(stderr,
- "Usage: %s [-hBHP][-g <gdb port>][-z <hz>][-s <pci>][-p pincpu]"
+ "Usage: %s [-ehBHP][-g <gdb port>][-z <hz>][-s <pci>][-p pincpu]"
"[-n <pci>][-m lowmem][-M highmem] <vm>\n"
" -g: gdb port (default is %d and 0 means don't open)\n"
" -c: # cpus (default 1)\n"
@@ -130,6 +132,7 @@ usage(int code)
" -B: inject breakpoint exception on vm entry\n"
" -H: vmexit from the guest on hlt\n"
" -P: vmexit from the guest on pause\n"
+ " -e: exit on unhandled i/o access\n"
" -h: help\n"
" -z: guest hz (default is %d)\n"
" -s: <slot,driver,configinfo> PCI slot config\n"
@@ -300,7 +303,7 @@ vmexit_inout(struct vmctx *ctx, struct vm_exit *vme, int *pvcpu)
if (out && port == GUEST_NIO_PORT)
return (vmexit_handle_notify(ctx, vme, pvcpu, eax));
- error = emulate_inout(ctx, vcpu, in, port, bytes, &eax);
+ error = emulate_inout(ctx, vcpu, in, port, bytes, &eax, strictio);
if (error == 0 && in)
error = vm_set_register(ctx, vcpu, VM_REG_GUEST_RAX, eax);
@@ -510,7 +513,7 @@ main(int argc, char *argv[])
gdb_port = DEFAULT_GDB_PORT;
guest_ncpus = 1;
- while ((c = getopt(argc, argv, "hBHPxp:g:c:z:s:n:m:M:")) != -1) {
+ while ((c = getopt(argc, argv, "ehBHPxp:g:c:z:s:n:m:M:")) != -1) {
switch (c) {
case 'B':
inject_bkpt = 1;
@@ -551,6 +554,9 @@ main(int argc, char *argv[])
case 'P':
guest_vmexit_on_pause = 1;
break;
+ case 'e':
+ strictio = 1;
+ break;
case 'h':
usage(0);
default:
diff --git a/usr.sbin/bhyve/inout.c b/usr.sbin/bhyve/inout.c
index 84445b1..bf9c5f7 100644
--- a/usr.sbin/bhyve/inout.c
+++ b/usr.sbin/bhyve/inout.c
@@ -48,9 +48,30 @@ static struct {
void *arg;
} inout_handlers[MAX_IOPORTS];
+static int
+default_inout(struct vmctx *ctx, int vcpu, int in, int port, int bytes,
+ uint32_t *eax, void *arg)
+{
+ if (in) {
+ switch (bytes) {
+ case 4:
+ *eax = 0xffffffff;
+ break;
+ case 2:
+ *eax = 0xffff;
+ break;
+ case 1:
+ *eax = 0xff;
+ break;
+ }
+ }
+
+ return (0);
+}
+
int
emulate_inout(struct vmctx *ctx, int vcpu, int in, int port, int bytes,
- uint32_t *eax)
+ uint32_t *eax, int strict)
{
int flags;
inout_func_t handler;
@@ -58,7 +79,9 @@ emulate_inout(struct vmctx *ctx, int vcpu, int in, int port, int bytes,
assert(port < MAX_IOPORTS);
- if ((handler = inout_handlers[port].handler) == NULL)
+ handler = inout_handlers[port].handler;
+
+ if (strict && handler == default_inout)
return (-1);
flags = inout_handlers[port].flags;
@@ -74,7 +97,21 @@ void
init_inout(void)
{
struct inout_port **iopp, *iop;
+ int i;
+
+ /*
+ * Set up the default handler for all ports
+ */
+ for (i = 0; i < MAX_IOPORTS; i++) {
+ inout_handlers[i].name = "default";
+ inout_handlers[i].flags = IOPORT_F_IN | IOPORT_F_OUT;
+ inout_handlers[i].handler = default_inout;
+ inout_handlers[i].arg = NULL;
+ }
+ /*
+ * Overwrite with specified handlers
+ */
SET_FOREACH(iopp, inout_port_set) {
iop = *iopp;
assert(iop->port < MAX_IOPORTS);
diff --git a/usr.sbin/bhyve/inout.h b/usr.sbin/bhyve/inout.h
index b15011d..f6a8db2 100644
--- a/usr.sbin/bhyve/inout.h
+++ b/usr.sbin/bhyve/inout.h
@@ -59,7 +59,7 @@ struct inout_port {
void init_inout(void);
int emulate_inout(struct vmctx *, int vcpu, int in, int port, int bytes,
- uint32_t *eax);
+ uint32_t *eax, int strict);
int register_inout(struct inout_port *iop);
#endif /* _INOUT_H_ */
diff --git a/usr.sbin/bhyve/rtc.c b/usr.sbin/bhyve/rtc.c
index a6f44e0..f8b894e 100644
--- a/usr.sbin/bhyve/rtc.c
+++ b/usr.sbin/bhyve/rtc.c
@@ -68,6 +68,8 @@ __FBSDID("$FreeBSD$");
#define RTC_RSTCODE 0x0f
+#define RTC_EQUIPMENT 0x14
+
static int addr;
/* XXX initialize these to default values as they would be from BIOS */
@@ -136,6 +138,7 @@ rtc_addr_handler(struct vmctx *ctx, int vcpu, int in, int port, int bytes,
case RTC_STATUSD:
case RTC_DIAG:
case RTC_RSTCODE:
+ case RTC_EQUIPMENT:
break;
default:
return (-1);
@@ -228,6 +231,9 @@ rtc_data_handler(struct vmctx *ctx, int vcpu, int in, int port, int bytes,
case RTC_RSTCODE:
*eax = rstcode;
return (0);
+ case RTC_EQUIPMENT:
+ *eax = 0;
+ return (0);
default:
return (-1);
}
OpenPOWER on IntegriCloud