Re: ftrace use of pci_resource_to_user()

2016-05-11 Thread Bjorn Helgaas
On Fri, May 6, 2016 at 5:33 AM, karol herbst wrote: > 2016-05-06 12:16 GMT+02:00 Pekka Paalanen : >> On Wed, 4 May 2016 14:17:13 -0500 >> Bjorn Helgaas wrote: >> >>> 138295373ccf ("ftrace: mmiotrace update, #2") added this use of >>> pci_resource_to_user(): >>> >>> +static int mmio_print_pcidev

Re: ftrace use of pci_resource_to_user()

2016-05-06 Thread karol herbst
2016-05-06 12:16 GMT+02:00 Pekka Paalanen : > On Wed, 4 May 2016 14:17:13 -0500 > Bjorn Helgaas wrote: > >> 138295373ccf ("ftrace: mmiotrace update, #2") added this use of >> pci_resource_to_user(): >> >> +static int mmio_print_pcidev(struct trace_seq *s, const struct pci_dev >> *dev) >> +{ >

Re: ftrace use of pci_resource_to_user()

2016-05-06 Thread Pekka Paalanen
On Wed, 4 May 2016 14:17:13 -0500 Bjorn Helgaas wrote: > 138295373ccf ("ftrace: mmiotrace update, #2") added this use of > pci_resource_to_user(): > > +static int mmio_print_pcidev(struct trace_seq *s, const struct pci_dev > *dev) > +{ > ... > + /* > +* XXX: is pci_resou

Re: ftrace use of pci_resource_to_user()

2016-05-04 Thread Steven Rostedt
On Wed, 4 May 2016 14:17:13 -0500 Bjorn Helgaas wrote: > 138295373ccf ("ftrace: mmiotrace update, #2") added this use of > pci_resource_to_user(): > > +static int mmio_print_pcidev(struct trace_seq *s, const struct pci_dev > *dev) > +{ > ... > + /* > +* XXX: is pci_resou

ftrace use of pci_resource_to_user()

2016-05-04 Thread Bjorn Helgaas
138295373ccf ("ftrace: mmiotrace update, #2") added this use of pci_resource_to_user(): +static int mmio_print_pcidev(struct trace_seq *s, const struct pci_dev *dev) +{ ... + /* +* XXX: is pci_resource_to_user() appropriate, since we are +* supposed to interpret t