On Wed, Nov 15, 2023 at 02:08:03AM -0800, Vishal Verma wrote:
> On Sun, 2023-10-01 at 15:31 -0700, [email protected] wrote:
snip
> > +CXL_EXPORT int cxl_memdev_trigger_poison_list(struct cxl_memdev *memdev)
> > +{
> > + struct cxl_ctx *ctx = cxl_memdev_get_ctx(memdev);
> > + char *path = memdev->dev_buf;
> > + int len = memdev->buf_len, rc;
> > +
> > + if (snprintf(path, len, "%s/trigger_poison_list", memdev->dev_path)
> > >=
> > + len) {
>
> I see this unfortunate line break Jonathan commented on still crept in,
> agreed that breaking up snprintf's args would look better.
Fixed up in v3.
Thanks!
>
>
