|
From: | Cao jin |
Subject: | Re: [Qemu-devel] [PATCH v3 8/8] vmxnet3: remove unnecessary internal msix state flag |
Date: | Fri, 30 Sep 2016 14:58:03 +0800 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 |
On 09/29/2016 10:42 PM, Markus Armbruster wrote:
Cao jin <address@hidden> writes:
static int vmxnet3_post_load(void *opaque, int version_id) { VMXNET3State *s = opaque; - PCIDevice *d = PCI_DEVICE(s); net_tx_pkt_init(&s->tx_pkt, PCI_DEVICE(s), s->max_tx_frags, s->peer_has_vhdr); net_rx_pkt_init(&s->rx_pkt, s->peer_has_vhdr); - if (s->msix_used) { - if (!vmxnet3_use_msix_vectors(s, VMXNET3_MAX_INTRS)) { - VMW_WRPRN("Failed to re-use MSI-X vectors"); - msix_uninit(d, &s->msix_bar, &s->msix_bar); - s->msix_used = false; - return -1; - } - } - vmxnet3_validate_queues(s); vmxnet3_validate_interrupts(s);This hunk isn't obvious. Can you explain the change?
flag msix_used is used in VMStateDescription.post_Load().1st, I think msix's code here is not necessary, because in destination, device has been realized before incoming migration, So I don't know why re-use MSI-X vectors here. Dmitry, could help to explain?
2nd, this patch is going to remove this flag, so I removed the hunk. -- Yours Sincerely, Cao jin
[Prev in Thread] | Current Thread | [Next in Thread] |