> -----Original Message----- > From: Xen-devel [mailto:xen-devel-boun...@lists.xenproject.org] On Behalf Of > Roger Pau Monne > Sent: 07 June 2019 10:22 > To: xen-devel@lists.xenproject.org > Cc: Stefano Stabellini <sstabell...@kernel.org>; Wei Liu <w...@xen.org>; > Konrad Rzeszutek Wilk > <konrad.w...@oracle.com>; George Dunlap <george.dun...@citrix.com>; Andrew > Cooper > <andrew.coop...@citrix.com>; Ian Jackson <ian.jack...@citrix.com>; Tim > (Xen.org) <t...@xen.org>; Julien > Grall <julien.gr...@arm.com>; Jan Beulich <jbeul...@suse.com>; Roger Pau > Monne <roger....@citrix.com> > Subject: [Xen-devel] [PATCH v3 02/13] pci: introduce a devfn field to > pci_sbdf_t > > This is equivalent to the current extfunc field in term of contents. > > Switch the two current users of extfunc to use devfn instead for > correctness. > > No functional change. > > Requested-by: Jan Beulich <jbeul...@suse.com> > Signed-off-by: Roger Pau Monné <roger....@citrix.com>
Reviewed-by: Paul Durrant <paul.durr...@citrix.com> > --- > Cc: Andrew Cooper <andrew.coop...@citrix.com> > Cc: George Dunlap <george.dun...@eu.citrix.com> > Cc: Ian Jackson <ian.jack...@eu.citrix.com> > Cc: Jan Beulich <jbeul...@suse.com> > Cc: Julien Grall <julien.gr...@arm.com> > Cc: Konrad Rzeszutek Wilk <konrad.w...@oracle.com> > Cc: Stefano Stabellini <sstabell...@kernel.org> > Cc: Tim Deegan <t...@xen.org> > Cc: Wei Liu <w...@xen.org> > --- > Changes since v2: > - Switch declaration order of the devfn and extfunc fields. > > Changes since v1: > - New in this version. > --- > xen/drivers/vpci/vpci.c | 4 ++-- > xen/include/xen/pci.h | 3 ++- > 2 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/xen/drivers/vpci/vpci.c b/xen/drivers/vpci/vpci.c > index b03232fe30..ca598675ea 100644 > --- a/xen/drivers/vpci/vpci.c > +++ b/xen/drivers/vpci/vpci.c > @@ -327,7 +327,7 @@ uint32_t vpci_read(pci_sbdf_t sbdf, unsigned int reg, > unsigned int size) > } > > /* Find the PCI dev matching the address. */ > - pdev = pci_get_pdev_by_domain(d, sbdf.seg, sbdf.bus, sbdf.extfunc); > + pdev = pci_get_pdev_by_domain(d, sbdf.seg, sbdf.bus, sbdf.devfn); > if ( !pdev ) > return vpci_read_hw(sbdf, reg, size); > > @@ -432,7 +432,7 @@ void vpci_write(pci_sbdf_t sbdf, unsigned int reg, > unsigned int size, > * Find the PCI dev matching the address. > * Passthrough everything that's not trapped. > */ > - pdev = pci_get_pdev_by_domain(d, sbdf.seg, sbdf.bus, sbdf.extfunc); > + pdev = pci_get_pdev_by_domain(d, sbdf.seg, sbdf.bus, sbdf.devfn); > if ( !pdev ) > { > vpci_write_hw(sbdf, reg, size, data); > diff --git a/xen/include/xen/pci.h b/xen/include/xen/pci.h > index dab1a86dd4..c61fd7e262 100644 > --- a/xen/include/xen/pci.h > +++ b/xen/include/xen/pci.h > @@ -49,7 +49,8 @@ typedef union { > uint8_t fn : 3, > dev : 5; > }; > - uint8_t extfunc; > + uint8_t devfn, > + extfunc; > }; > uint8_t bus; > }; > -- > 2.20.1 (Apple Git-117) > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xenproject.org > https://lists.xenproject.org/mailman/listinfo/xen-devel _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel