From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH] vfio/pci: Virtualize Maximum Payload Size To: Alex Williamson , Auger Eric Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, jcm@redhat.com, "linux-pci@vger.kernel.org" References: <20170919164952.13200.78849.stgit@gimli.home> <20170919142024.05723289@t450s.home> <3dd92928-3cda-50c1-6a67-10721c003c10@redhat.com> <73d55003-6fcd-3741-73a1-365c73933538@redhat.com> <20170920101133.33b68d8f@w520.home> From: Sinan Kaya Message-ID: <2c094ad6-4641-ebee-7faa-21ad9beeca72@codeaurora.org> Date: Wed, 20 Sep 2017 12:29:26 -0400 MIME-Version: 1.0 In-Reply-To: <20170920101133.33b68d8f@w520.home> Content-Type: text/plain; charset=utf-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: On 9/20/2017 12:11 PM, Alex Williamson wrote: > My impression is that the issue would be inefficiency. There should be > nothing functionally wrong with a read request less than MPS, but we're > not "filling" the TLP as much as the topology allows. Is that your > understanding as well, Sinan? That's my understanding as well. It would be a performance hit to use a value less than MPS. -- Sinan Kaya Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.