Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 29 Apr 2022 10:43:20 +0100
From:   Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
To:     Dexuan Cui <decui@microsoft.com>
Cc:     Jake Oshins <jakeo@microsoft.com>,
        "bhelgaas@google.com" <bhelgaas@google.com>,
        Bjorn Helgaas <helgaas@kernel.org>,
        "wei.liu@kernel.org" <wei.liu@kernel.org>,
        Alex Williamson <alex.williamson@redhat.com>,
        KY Srinivasan <kys@microsoft.com>,
        Haiyang Zhang <haiyangz@microsoft.com>,
        Stephen Hemminger <sthemmin@microsoft.com>,
        "linux-hyperv@vger.kernel.org" <linux-hyperv@vger.kernel.org>,
        "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Michael Kelley (LINUX)" <mikelley@microsoft.com>,
        "robh@kernel.org" <robh@kernel.org>, "kw@linux.com" <kw@linux.com>,
        "kvm@vger.kernel.org" <kvm@vger.kernel.org>
Subject: Re: [EXTERNAL] Re: [PATCH] PCI: hv: Do not set PCI_COMMAND_MEMORY to
 reduce VM boot time
Message-ID: <YmuzOJy/6F5wSjY7@lpieralisi>
References: <SN4PR2101MB0878E466880C047D3A0D0C92ABFB9@SN4PR2101MB0878.namprd21.prod.outlook.com>
 <20220428191213.GA36573@bhelgaas>
 <SN4PR2101MB08786192CE6DFC3450BEC571ABFD9@SN4PR2101MB0878.namprd21.prod.outlook.com>
 <BYAPR21MB12709F42ED7C2496D366792EBFFC9@BYAPR21MB1270.namprd21.prod.outlook.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <BYAPR21MB12709F42ED7C2496D366792EBFFC9@BYAPR21MB1270.namprd21.prod.outlook.com>
Precedence: bulk

On Fri, Apr 29, 2022 at 01:11:29AM +0000, Dexuan Cui wrote:
> > From: Jake Oshins <jakeo@microsoft.com>
> > Sent: Thursday, April 28, 2022 12:21 PM
> > ... 
> > Thanks everybody for responding to my questions.
> > 
> > Bjorn, from your response, it sounds like this change is safe until some possible
> > future which new functionality is introduced for rebalancing resources.
> > 
> > Dexuan, I don't have any further objection to the patch.
> > 
> > -- Jake Oshins
> 
> Thank all for the informative discussion!!
> 
> @Bjorn, Lorenzo: can this patch go through the hyperv tree?
> I see some recent pci-hyperv commits in the hyperv tree:
> https://git.kernel.org/pub/scm/linux/kernel/git/hyperv/linux.git

For this patch it should be fine as long as it does not become a rule,
we still want to be in the review loop for PCI hyper-V changes.

I will comment and ACK shortly.

Lorenzo