| Message ID | 20241007-dcd-type2-upstream-v4-3-c261ee6eeded@intel.com (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Delegated to: | Ira Weiny |
| Headers | show |
| Series | DCD: Add support for Dynamic Capacity Devices (DCD) | expand |
On Mon, 07 Oct 2024 18:16:09 -0500 Ira Weiny <ira.weiny@intel.com> wrote: > Now that there is a printk specifier for struct range use it in > debug output of CDAT data. > > To: Petr Mladek <pmladek@suse.com> > To: Steven Rostedt <rostedt@goodmis.org> > To: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > To: Rasmus Villemoes <linux@rasmusvillemoes.dk> > To: Sergey Senozhatsky <senozhatsky@chromium.org> > To: Jonathan Corbet <corbet@lwn.net> (maintainer:DOCUMENTATION) > Cc: linux-doc@vger.kernel.org > Cc: linux-kernel@vger.kernel.org (open list) > Signed-off-by: Ira Weiny <ira.weiny@intel.com> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
On Mon, Oct 07, 2024 at 06:16:09PM -0500, Ira Weiny wrote: > Now that there is a printk specifier for struct range use it in > debug output of CDAT data. > > To: Petr Mladek <pmladek@suse.com> > To: Steven Rostedt <rostedt@goodmis.org> > To: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > To: Rasmus Villemoes <linux@rasmusvillemoes.dk> > To: Sergey Senozhatsky <senozhatsky@chromium.org> > To: Jonathan Corbet <corbet@lwn.net> (maintainer:DOCUMENTATION) > Cc: linux-doc@vger.kernel.org > Cc: linux-kernel@vger.kernel.org (open list) > Signed-off-by: Ira Weiny <ira.weiny@intel.com> > --- Reviewed-by: Fan Ni <fan.ni@samsung.com> > drivers/cxl/core/cdat.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/cxl/core/cdat.c b/drivers/cxl/core/cdat.c > index ef1621d40f05..438869df241a 100644 > --- a/drivers/cxl/core/cdat.c > +++ b/drivers/cxl/core/cdat.c > @@ -247,8 +247,8 @@ static void update_perf_entry(struct device *dev, struct dsmas_entry *dent, > dpa_perf->dpa_range = dent->dpa_range; > dpa_perf->qos_class = dent->qos_class; > dev_dbg(dev, > - "DSMAS: dpa: %#llx qos: %d read_bw: %d write_bw %d read_lat: %d write_lat: %d\n", > - dent->dpa_range.start, dpa_perf->qos_class, > + "DSMAS: dpa: %pra qos: %d read_bw: %d write_bw %d read_lat: %d write_lat: %d\n", > + &dent->dpa_range, dpa_perf->qos_class, > dent->coord[ACCESS_COORDINATE_CPU].read_bandwidth, > dent->coord[ACCESS_COORDINATE_CPU].write_bandwidth, > dent->coord[ACCESS_COORDINATE_CPU].read_latency, > @@ -279,8 +279,8 @@ static void cxl_memdev_set_qos_class(struct cxl_dev_state *cxlds, > range_contains(&pmem_range, &dent->dpa_range)) > update_perf_entry(dev, dent, &mds->pmem_perf); > else > - dev_dbg(dev, "no partition for dsmas dpa: %#llx\n", > - dent->dpa_range.start); > + dev_dbg(dev, "no partition for dsmas dpa: %pra\n", > + &dent->dpa_range); > } > } > > > -- > 2.46.0 >
diff --git a/drivers/cxl/core/cdat.c b/drivers/cxl/core/cdat.c index ef1621d40f05..438869df241a 100644 --- a/drivers/cxl/core/cdat.c +++ b/drivers/cxl/core/cdat.c @@ -247,8 +247,8 @@ static void update_perf_entry(struct device *dev, struct dsmas_entry *dent, dpa_perf->dpa_range = dent->dpa_range; dpa_perf->qos_class = dent->qos_class; dev_dbg(dev, - "DSMAS: dpa: %#llx qos: %d read_bw: %d write_bw %d read_lat: %d write_lat: %d\n", - dent->dpa_range.start, dpa_perf->qos_class, + "DSMAS: dpa: %pra qos: %d read_bw: %d write_bw %d read_lat: %d write_lat: %d\n", + &dent->dpa_range, dpa_perf->qos_class, dent->coord[ACCESS_COORDINATE_CPU].read_bandwidth, dent->coord[ACCESS_COORDINATE_CPU].write_bandwidth, dent->coord[ACCESS_COORDINATE_CPU].read_latency, @@ -279,8 +279,8 @@ static void cxl_memdev_set_qos_class(struct cxl_dev_state *cxlds, range_contains(&pmem_range, &dent->dpa_range)) update_perf_entry(dev, dent, &mds->pmem_perf); else - dev_dbg(dev, "no partition for dsmas dpa: %#llx\n", - dent->dpa_range.start); + dev_dbg(dev, "no partition for dsmas dpa: %pra\n", + &dent->dpa_range); } }
Now that there is a printk specifier for struct range use it in debug output of CDAT data. To: Petr Mladek <pmladek@suse.com> To: Steven Rostedt <rostedt@goodmis.org> To: Andy Shevchenko <andriy.shevchenko@linux.intel.com> To: Rasmus Villemoes <linux@rasmusvillemoes.dk> To: Sergey Senozhatsky <senozhatsky@chromium.org> To: Jonathan Corbet <corbet@lwn.net> (maintainer:DOCUMENTATION) Cc: linux-doc@vger.kernel.org Cc: linux-kernel@vger.kernel.org (open list) Signed-off-by: Ira Weiny <ira.weiny@intel.com> --- drivers/cxl/core/cdat.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)