* [PATCH] cpupower: fix breakage from libpci API change
@ 2015-04-13 20:24 Lucas Stach
2015-04-14 16:28 ` Thomas Renninger
0 siblings, 1 reply; 4+ messages in thread
From: Lucas Stach @ 2015-04-13 20:24 UTC (permalink / raw)
To: Thomas Renninger; +Cc: linux-pm, linux-kernel
libpci 3.3.0 introduced an additional member in the pci_filter struct
which needs to be initialized to -1 to get the same behavior as before
the API change. The libpci internal helpers got updated accordingly,
but as the cpupower pci helpers initialized the struct themselves the
behavior changed.
Use the libpci helper pci_filter_init() to fix this and guard against
similar breakages in the future.
This fixes probing of the AMD fam12h/14h cpuidle monitor on systems
with libpci >= 3.3.0.
Signed-off-by: Lucas Stach <dev@lynxeye.de>
---
tools/power/cpupower/utils/helpers/pci.c | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/tools/power/cpupower/utils/helpers/pci.c b/tools/power/cpupower/utils/helpers/pci.c
index 9690798..8b27898 100644
--- a/tools/power/cpupower/utils/helpers/pci.c
+++ b/tools/power/cpupower/utils/helpers/pci.c
@@ -25,14 +25,21 @@
struct pci_dev *pci_acc_init(struct pci_access **pacc, int domain, int bus,
int slot, int func, int vendor, int dev)
{
- struct pci_filter filter_nb_link = { domain, bus, slot, func,
- vendor, dev };
+ struct pci_filter filter_nb_link;
struct pci_dev *device;
*pacc = pci_alloc();
if (*pacc == NULL)
return NULL;
+ pci_filter_init(*pacc, &filter_nb_link);
+ filter_nb_link.domain = domain;
+ filter_nb_link.bus = bus;
+ filter_nb_link.slot = slot;
+ filter_nb_link.func = func;
+ filter_nb_link.vendor = vendor;
+ filter_nb_link.device = dev;
+
pci_init(*pacc);
pci_scan_bus(*pacc);
--
2.1.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] cpupower: fix breakage from libpci API change
2015-04-13 20:24 [PATCH] cpupower: fix breakage from libpci API change Lucas Stach
@ 2015-04-14 16:28 ` Thomas Renninger
2015-04-28 19:26 ` Lucas Stach
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Renninger @ 2015-04-14 16:28 UTC (permalink / raw)
To: Lucas Stach; +Cc: linux-pm, linux-kernel, rjw
Hi,
On Monday, April 13, 2015 10:24:01 PM Lucas Stach wrote:
> libpci 3.3.0 introduced an additional member in the pci_filter struct
> which needs to be initialized to -1 to get the same behavior as before
> the API change.
Sounds not that clever, but there probably is a reason for this...
I am not that familiar with the pci lib and its recent changes, but
below patch looks reasonable.
Acked-by: Thomas Renninger <trenn@suse.de>
> The libpci internal helpers got updated accordingly,
> but as the cpupower pci helpers initialized the struct themselves the
> behavior changed.
>
> Use the libpci helper pci_filter_init() to fix this and guard against
> similar breakages in the future.
>
> This fixes probing of the AMD fam12h/14h cpuidle monitor on systems
> with libpci >= 3.3.0.
>
> Signed-off-by: Lucas Stach <dev@lynxeye.de>
> ---
> tools/power/cpupower/utils/helpers/pci.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/tools/power/cpupower/utils/helpers/pci.c
> b/tools/power/cpupower/utils/helpers/pci.c index 9690798..8b27898 100644
> --- a/tools/power/cpupower/utils/helpers/pci.c
> +++ b/tools/power/cpupower/utils/helpers/pci.c
> @@ -25,14 +25,21 @@
> struct pci_dev *pci_acc_init(struct pci_access **pacc, int domain, int bus,
> int slot, int func, int vendor, int dev)
> {
> - struct pci_filter filter_nb_link = { domain, bus, slot, func,
> - vendor, dev };
> + struct pci_filter filter_nb_link;
> struct pci_dev *device;
>
> *pacc = pci_alloc();
> if (*pacc == NULL)
> return NULL;
>
> + pci_filter_init(*pacc, &filter_nb_link);
> + filter_nb_link.domain = domain;
> + filter_nb_link.bus = bus;
> + filter_nb_link.slot = slot;
> + filter_nb_link.func = func;
> + filter_nb_link.vendor = vendor;
> + filter_nb_link.device = dev;
> +
> pci_init(*pacc);
> pci_scan_bus(*pacc);
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] cpupower: fix breakage from libpci API change
2015-04-14 16:28 ` Thomas Renninger
@ 2015-04-28 19:26 ` Lucas Stach
2015-04-29 0:05 ` Rafael J. Wysocki
0 siblings, 1 reply; 4+ messages in thread
From: Lucas Stach @ 2015-04-28 19:26 UTC (permalink / raw)
To: Thomas Renninger; +Cc: linux-pm, linux-kernel, rjw
So, who is going to pick up this patch?
Regards,
Lucas
Am Dienstag, den 14.04.2015, 18:28 +0200 schrieb Thomas Renninger:
> Hi,
>
> On Monday, April 13, 2015 10:24:01 PM Lucas Stach wrote:
> > libpci 3.3.0 introduced an additional member in the pci_filter struct
> > which needs to be initialized to -1 to get the same behavior as before
> > the API change.
> Sounds not that clever, but there probably is a reason for this...
>
> I am not that familiar with the pci lib and its recent changes, but
> below patch looks reasonable.
>
> Acked-by: Thomas Renninger <trenn@suse.de>
>
>
> > The libpci internal helpers got updated accordingly,
> > but as the cpupower pci helpers initialized the struct themselves the
> > behavior changed.
> >
> > Use the libpci helper pci_filter_init() to fix this and guard against
> > similar breakages in the future.
> >
> > This fixes probing of the AMD fam12h/14h cpuidle monitor on systems
> > with libpci >= 3.3.0.
> >
> > Signed-off-by: Lucas Stach <dev@lynxeye.de>
> > ---
> > tools/power/cpupower/utils/helpers/pci.c | 11 +++++++++--
> > 1 file changed, 9 insertions(+), 2 deletions(-)
> >
> > diff --git a/tools/power/cpupower/utils/helpers/pci.c
> > b/tools/power/cpupower/utils/helpers/pci.c index 9690798..8b27898 100644
> > --- a/tools/power/cpupower/utils/helpers/pci.c
> > +++ b/tools/power/cpupower/utils/helpers/pci.c
> > @@ -25,14 +25,21 @@
> > struct pci_dev *pci_acc_init(struct pci_access **pacc, int domain, int bus,
> > int slot, int func, int vendor, int dev)
> > {
> > - struct pci_filter filter_nb_link = { domain, bus, slot, func,
> > - vendor, dev };
> > + struct pci_filter filter_nb_link;
> > struct pci_dev *device;
> >
> > *pacc = pci_alloc();
> > if (*pacc == NULL)
> > return NULL;
> >
> > + pci_filter_init(*pacc, &filter_nb_link);
> > + filter_nb_link.domain = domain;
> > + filter_nb_link.bus = bus;
> > + filter_nb_link.slot = slot;
> > + filter_nb_link.func = func;
> > + filter_nb_link.vendor = vendor;
> > + filter_nb_link.device = dev;
> > +
> > pci_init(*pacc);
> > pci_scan_bus(*pacc);
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] cpupower: fix breakage from libpci API change
2015-04-28 19:26 ` Lucas Stach
@ 2015-04-29 0:05 ` Rafael J. Wysocki
0 siblings, 0 replies; 4+ messages in thread
From: Rafael J. Wysocki @ 2015-04-29 0:05 UTC (permalink / raw)
To: Lucas Stach; +Cc: Thomas Renninger, linux-pm, linux-kernel
On Tuesday, April 28, 2015 09:26:49 PM Lucas Stach wrote:
> So, who is going to pick up this patch?
Should be in the Linus' tree already, isn't it?
I pick up PM tools patches if ACKed by their respective maintainers as a rule.
turbostat is an exception.
> Am Dienstag, den 14.04.2015, 18:28 +0200 schrieb Thomas Renninger:
> > Hi,
> >
> > On Monday, April 13, 2015 10:24:01 PM Lucas Stach wrote:
> > > libpci 3.3.0 introduced an additional member in the pci_filter struct
> > > which needs to be initialized to -1 to get the same behavior as before
> > > the API change.
> > Sounds not that clever, but there probably is a reason for this...
> >
> > I am not that familiar with the pci lib and its recent changes, but
> > below patch looks reasonable.
> >
> > Acked-by: Thomas Renninger <trenn@suse.de>
> >
> >
> > > The libpci internal helpers got updated accordingly,
> > > but as the cpupower pci helpers initialized the struct themselves the
> > > behavior changed.
> > >
> > > Use the libpci helper pci_filter_init() to fix this and guard against
> > > similar breakages in the future.
> > >
> > > This fixes probing of the AMD fam12h/14h cpuidle monitor on systems
> > > with libpci >= 3.3.0.
> > >
> > > Signed-off-by: Lucas Stach <dev@lynxeye.de>
> > > ---
> > > tools/power/cpupower/utils/helpers/pci.c | 11 +++++++++--
> > > 1 file changed, 9 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/tools/power/cpupower/utils/helpers/pci.c
> > > b/tools/power/cpupower/utils/helpers/pci.c index 9690798..8b27898 100644
> > > --- a/tools/power/cpupower/utils/helpers/pci.c
> > > +++ b/tools/power/cpupower/utils/helpers/pci.c
> > > @@ -25,14 +25,21 @@
> > > struct pci_dev *pci_acc_init(struct pci_access **pacc, int domain, int bus,
> > > int slot, int func, int vendor, int dev)
> > > {
> > > - struct pci_filter filter_nb_link = { domain, bus, slot, func,
> > > - vendor, dev };
> > > + struct pci_filter filter_nb_link;
> > > struct pci_dev *device;
> > >
> > > *pacc = pci_alloc();
> > > if (*pacc == NULL)
> > > return NULL;
> > >
> > > + pci_filter_init(*pacc, &filter_nb_link);
> > > + filter_nb_link.domain = domain;
> > > + filter_nb_link.bus = bus;
> > > + filter_nb_link.slot = slot;
> > > + filter_nb_link.func = func;
> > > + filter_nb_link.vendor = vendor;
> > > + filter_nb_link.device = dev;
> > > +
> > > pci_init(*pacc);
> > > pci_scan_bus(*pacc);
> >
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2015-04-28 23:41 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-04-13 20:24 [PATCH] cpupower: fix breakage from libpci API change Lucas Stach
2015-04-14 16:28 ` Thomas Renninger
2015-04-28 19:26 ` Lucas Stach
2015-04-29 0:05 ` Rafael J. Wysocki
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.