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

Re: [PATCH 0/5] Small PCI refactoring


  • To: Teddy Astie <teddy.astie@xxxxxxxxxx>
  • From: dmukhin@xxxxxxxx
  • Date: Tue, 19 May 2026 20:34:25 -0700
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 205.220.161.53) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=ford.com; dmarc=pass (p=reject sp=reject pct=100) action=none header.from=ford.com; dkim=pass (signature was verified) header.d=saarlouis.ford.com; dkim=pass (signature was verified) header.d=ford.com; arc=none (0)
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=z+z3JbfLUtKKu8H0tn306Fwsm106GqK2cyILhROeg5I=; b=cnACWeiG9sA0rMzWGu2n6xIe0XbITwefj9j0ur7ze1iTG2YixLIsjaH1GwVPpSsofavuIO1wgD+DwvBGWRhIxWGi4h5dzfGFRDNV57u6uwD+KFaxHgyp8+Yby2Pw0gOlmjFlg1k57WTV7khwQWEyC0bKG+nFYccNYktaiDpPhuApdJPti1B5A43jST3QqFgFEhyPan8dAP0l5gXncN0MU4Nn+hKSwD9jXlmoWrLnIb6WJnXzATO8+6ib7hhTAH8udbynMdEdEHPnO51dwvrbbAlOFEJ6OLsc4rdC+HkqVu1g+9G0oHDGFy1LKdSPDxmwiGlq9rVwi+PpEQnDlsUpHA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=UiLfogI/i26HuvXq93jngElb7GjVbF72Rp7bwKD47b76rb1qVp+EKqPA92cSRkjbm4/8AGXBE21NgUpXTo00s7hWvEFgZar3lqchIuhaqrjiroaWVhduHdsl0TZvGpRedcOdc+T8Zl8n4cbI+sSH6oSZd8pd0EzzDu00MVKQ3t+9fuRbscSUf3hsF/LnrguM9jAgAjrz/1d5+vKt+vZB3DKxstv6smp52MzMVpDP8xLUxLc9OAsrPHoMhgvhxRwRQ3JF4onuIQaMx/zbyJ10QSQZ4oKD1RiPykEcXPHwMoSp39iz1mbkxx68Firm3k6yGBGq7GMv3ecFN4RHZiDjvQ==
  • Authentication-results: eu.smtp.expurgate.cloud; dkim=pass header.s=ppford header.d=ford.com header.i="@ford.com" header.h="Cc:Content-Transfer-Encoding:Content-Type:Date:From:In-Reply-To:Message-ID:MIME-Version:References:Subject:To"; dkim=fail header.s=selector2-azureford-onmicrosoft-com header.d=azureford.onmicrosoft.com header.i="@azureford.onmicrosoft.com"; dkim=pass header.s=ppserprodsaar header.d=saarlouis.ford.com header.i="@saarlouis.ford.com" header.h="Cc:Content-Transfer-Encoding:Content-Type:Date:From:In-Reply-To:Message-ID:MIME-Version:References:Subject:To"; dkim=pass header.s=ppfserpocford header.d=ford.com header.i="@ford.com" header.h="Cc:Content-Transfer-Encoding:Content-Type:Date:From:In-Reply-To:Message-ID:MIME-Version:References:Subject:To"
  • Cc: xen-devel@xxxxxxxxxxxxxxxxxxxx, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Julien Grall <julien@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Jason Andryuk <jason.andryuk@xxxxxxx>
  • Delivery-date: Wed, 20 May 2026 03:34:41 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Pser-m365-app: SER-APP

On Mon, May 18, 2026 at 07:20:15PM +0200, Teddy Astie wrote:
> Le 18/05/2026 à 17:22, Teddy Astie a écrit :
> > The goal of this series is to make some refactoring of some
> > pci primitives to improve codegen and make code less verbose.
> > 
> > A big chunk of it is converting many places where (seg, bus, dev, fn)
> > is split into multiples variables and convert it into being just
> > pci_sbdf_t, in particular in some PCI function parameters to reduce
> > parameter count which usually translate into less registers to pass
> > to the function. Moreover, we also avoid translating back and forth
> > between pci_sbdf_t and individual (seg, bus, dev, fn).
> > 
> > Latest patch attempts to improve codegen of pci_conf_{read,write}N()
> > by making them inline specialized variants of pci_mmcfg_{read,write}()
> > in order to eliminate a particular `switch (len)` at compile time.
> > 
> > No intended functional change, aside some parts of the codebase that will
> > now correctly handle PCI segment when parsed while it was previously
> > ignored (e.g dbgp).
> > 

I would schedule a full CI cycle against the series for smoke testing.

--
Denis



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.