* [PATCH] libxl: When checking BDF of existing slots, function should be decimal, not hex
@ 2012-05-25 16:34 George Dunlap
2012-05-25 20:27 ` Konrad Rzeszutek Wilk
0 siblings, 1 reply; 3+ messages in thread
From: George Dunlap @ 2012-05-25 16:34 UTC (permalink / raw)
To: xen-devel; +Cc: george.dunlap
# HG changeset patch
# User George Dunlap <george.dunlap@eu.citrix.com>
# Date 1337961666 0
# Node ID 1c28051020488782f1277dd60a2418324580297e
# Parent 69c3ae25bb1ddcb0ea44b7566d36d34e9d6a70aa
libxl: When checking BDF of existing slots, function should be decimal, not hex
Spotted-by: Konrad Wilk <konrad.wilk@oracle.com>
Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
diff --git a/tools/libxl/libxl_pci.c b/tools/libxl/libxl_pci.c
--- a/tools/libxl/libxl_pci.c
+++ b/tools/libxl/libxl_pci.c
@@ -480,7 +480,7 @@ static int pciback_dev_has_slot(libxl__g
return ERROR_FAIL;
}
- while(fscanf(f, "%x:%x:%x.%x\n", &dom, &bus, &dev, &func)==4) {
+ while(fscanf(f, "%x:%x:%x.%d\n", &dom, &bus, &dev, &func)==4) {
if(dom == pcidev->domain
&& bus == pcidev->bus
&& dev == pcidev->dev
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] libxl: When checking BDF of existing slots, function should be decimal, not hex
2012-05-25 16:34 [PATCH] libxl: When checking BDF of existing slots, function should be decimal, not hex George Dunlap
@ 2012-05-25 20:27 ` Konrad Rzeszutek Wilk
2012-05-29 9:00 ` Ian Campbell
0 siblings, 1 reply; 3+ messages in thread
From: Konrad Rzeszutek Wilk @ 2012-05-25 20:27 UTC (permalink / raw)
To: George Dunlap; +Cc: xen-devel
On Fri, May 25, 2012 at 04:34:17PM +0000, George Dunlap wrote:
> # HG changeset patch
> # User George Dunlap <george.dunlap@eu.citrix.com>
> # Date 1337961666 0
> # Node ID 1c28051020488782f1277dd60a2418324580297e
> # Parent 69c3ae25bb1ddcb0ea44b7566d36d34e9d6a70aa
> libxl: When checking BDF of existing slots, function should be decimal, not hex
>
> Spotted-by: Konrad Wilk <konrad.wilk@oracle.com>
Acked-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
>
> diff --git a/tools/libxl/libxl_pci.c b/tools/libxl/libxl_pci.c
> --- a/tools/libxl/libxl_pci.c
> +++ b/tools/libxl/libxl_pci.c
> @@ -480,7 +480,7 @@ static int pciback_dev_has_slot(libxl__g
> return ERROR_FAIL;
> }
>
> - while(fscanf(f, "%x:%x:%x.%x\n", &dom, &bus, &dev, &func)==4) {
> + while(fscanf(f, "%x:%x:%x.%d\n", &dom, &bus, &dev, &func)==4) {
> if(dom == pcidev->domain
> && bus == pcidev->bus
> && dev == pcidev->dev
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] libxl: When checking BDF of existing slots, function should be decimal, not hex
2012-05-25 20:27 ` Konrad Rzeszutek Wilk
@ 2012-05-29 9:00 ` Ian Campbell
0 siblings, 0 replies; 3+ messages in thread
From: Ian Campbell @ 2012-05-29 9:00 UTC (permalink / raw)
To: Konrad Rzeszutek Wilk; +Cc: George Dunlap, xen-devel@lists.xensource.com
On Fri, 2012-05-25 at 21:27 +0100, Konrad Rzeszutek Wilk wrote:
> On Fri, May 25, 2012 at 04:34:17PM +0000, George Dunlap wrote:
> > # HG changeset patch
> > # User George Dunlap <george.dunlap@eu.citrix.com>
> > # Date 1337961666 0
> > # Node ID 1c28051020488782f1277dd60a2418324580297e
> > # Parent 69c3ae25bb1ddcb0ea44b7566d36d34e9d6a70aa
> > libxl: When checking BDF of existing slots, function should be decimal, not hex
> >
> > Spotted-by: Konrad Wilk <konrad.wilk@oracle.com>
>
> Acked-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> > Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
A PCI func can be at most 7 I think so %x vs %d probably doesn't have
any real impact under normal use.
Applied though.
> >
> > diff --git a/tools/libxl/libxl_pci.c b/tools/libxl/libxl_pci.c
> > --- a/tools/libxl/libxl_pci.c
> > +++ b/tools/libxl/libxl_pci.c
> > @@ -480,7 +480,7 @@ static int pciback_dev_has_slot(libxl__g
> > return ERROR_FAIL;
> > }
> >
> > - while(fscanf(f, "%x:%x:%x.%x\n", &dom, &bus, &dev, &func)==4) {
> > + while(fscanf(f, "%x:%x:%x.%d\n", &dom, &bus, &dev, &func)==4) {
> > if(dom == pcidev->domain
> > && bus == pcidev->bus
> > && dev == pcidev->dev
> >
> > _______________________________________________
> > Xen-devel mailing list
> > Xen-devel@lists.xen.org
> > http://lists.xen.org/xen-devel
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-05-29 9:00 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-05-25 16:34 [PATCH] libxl: When checking BDF of existing slots, function should be decimal, not hex George Dunlap
2012-05-25 20:27 ` Konrad Rzeszutek Wilk
2012-05-29 9:00 ` Ian Campbell
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.