| Message ID | 1428956641-7765-1-git-send-email-dev@lynxeye.de (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Delegated to: | Rafael Wysocki |
| Headers | show |
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
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); > -- 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
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
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);
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(-)