qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH v2 08/15] hw/nvme: Implement the Function Level Reset


From: Łukasz Gieryk
Subject: Re: [PATCH v2 08/15] hw/nvme: Implement the Function Level Reset
Date: Wed, 17 Nov 2021 12:22:31 +0100
User-agent: Mutt/1.9.4 (2018-02-28)

On Tue, Nov 16, 2021 at 01:28:19PM -0800, Keith Busch wrote:
> On Tue, Nov 16, 2021 at 04:34:39PM +0100, Łukasz Gieryk wrote:
> >      if (!pci_is_vf(&n->parent_obj) && n->params.sriov_max_vfs) {
> > -        pcie_sriov_pf_disable_vfs(&n->parent_obj);
> > +        if (rst != NVME_RESET_CONTROLLER) {
> > +            pcie_sriov_pf_disable_vfs(&n->parent_obj);
> 
> Shouldn't this be 'if (rst == NVME_RESET_FUNCTION)'?

The NVMe Spec lists five possible reset types (triggers). According
to my understanding, only the Controller Reset doesn’t affect the VFs'
state, hence the '!='.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]