[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [kvm-devel] PATCH: dont call exit() from pci_nic_init()
From: |
Marcelo Tosatti |
Subject: |
[Qemu-devel] Re: [kvm-devel] PATCH: dont call exit() from pci_nic_init(), let caller handle |
Date: |
Thu, 20 Mar 2008 09:01:13 -0300 |
User-agent: |
Mutt/1.4.2.1i |
On Wed, Mar 19, 2008 at 07:19:51PM -0500, Ryan Harper wrote:
> While exploring the PCI hotplug code recently posted, I encountered a
> situation where I don't believe the current behavior is ideal. With
> hotplug, we can add additional pci-based nic devices like e1000 and
> rtl8139 from the qemu monitor. If one mistakenly specifies model=ne2000
> (the ISA version), qemu just exits. If a command is run from the
> monitor and specifies bogus values, I don't believe the right behavior
> is to exit out of the guest entirely. The attached patch (which doesn't
> apply directly against qemu-cvs since hotplug hasn't been merged)
> changes pci_nic_init() to return NULL on error instead of exiting
> and then I've replaced all callers to check the return value and exit(),
> preserving the existing behavior, but allowing flexibility so
> hotplug can do the right thing and just report the error rather than
> exiting the guest.
Hi Ryan,
Looks good, thanks.
There might still be some exit()'s lurking around due to device/cpu hot/add
failure.