Re: [PATCH v9 12/19] cxl/extent: Process dynamic partition events and realize region extents
From: Alison Schofield
Date: Mon Apr 14 2025 - 18:10:32 EST
On Sun, Apr 13, 2025 at 05:52:20PM -0500, Ira Weiny wrote:
snip
> +
> +void memdev_release_extent(struct cxl_memdev_state *mds, struct range *range)
> +{
> + struct device *dev = mds->cxlds.dev;
> + struct xarray extent_list;
> +
> + struct cxl_extent extent = {
> + .start_dpa = cpu_to_le64(range->start),
> + .length = cpu_to_le64(range_len(range)),
> + };
> +
> + dev_dbg(dev, "Release response dpa %pra\n", &range);
> +
> + xa_init(&extent_list);
> + if (xa_insert(&extent_list, 0, &extent, GFP_KERNEL)) {
> + dev_dbg(dev, "Failed to release %pra\n", &range);
> + goto destroy;
> + }
> +
> + if (cxl_send_dc_response(mds, CXL_MBOX_OP_RELEASE_DC, &extent_list, 1))
> + dev_dbg(dev, "Failed to release %pra\n", &range);
> +
smatch complains about the above 3 dev_dbg() messages:
memdev_release_extent() error: '%pr' expects argument of type struct range *, but argument 4 has type 'struct range**'
snip