Re: [Qemu-devel] [PATCH v7] kvm: notify host when the guest is panicked

From: Anthony Liguori
Date: Sun Jul 22 2012 - 15:22:41 EST


Sasha Levin <levinsasha928@xxxxxxxxx> writes:

> On 07/21/2012 09:12 AM, Wen Congyang wrote:
>> +#define KVM_PV_PORT (0x505UL)
>> +
>> #ifdef __KERNEL__
>> #include <asm/processor.h>
>>
>> @@ -221,6 +223,11 @@ static inline void kvm_disable_steal_time(void)
>> }
>> #endif
>>
>> +static inline unsigned int kvm_arch_pv_features(void)
>> +{
>> + return inl(KVM_PV_PORT);
>> +}
>> +
>
> Why is this safe?
>
> I'm not sure you can just pick any ioport you'd like and use it.

There are three ways I/O ports get used on a PC:

1) Platform devices
- This is well defined since the vast majority of platform devices are
implemented within a single chip. If you're emulating an i440fx
chipset, the PIIX4 spec has an exhaustive list.

2) PCI devices
- Typically, PCI only allocates ports starting at 0x0d00 to avoid
conflicts with ISA devices.

3) ISA devices
- ISA uses subtractive decoding so any ISA device can access. In
theory, an ISA device could attempt to use port 0x0505 but it's
unlikely. In a modern guest, there aren't really any ISA devices being
added either.

So yes, picking port 0x0505 is safe for something like this (as long as
you check to make sure that you really are under KVM).

Regards,

Anthony Liguori

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