On 20/05/2025 03:28, alison.schofi...@intel.com wrote: > diff --git a/ndctl/monitor.c b/ndctl/monitor.c > index bd8a74863476..925b37f4184b 100644 > --- a/ndctl/monitor.c > +++ b/ndctl/monitor.c > @@ -658,7 +658,7 @@ int cmd_monitor(int argc, const char **argv, struct > ndctl_ctx *ctx) > rc = -ENXIO; > goto out; > } > - > + info(&monitor, "monitor ready\n");
This brings to mind my initial contribution to ndctl, where it commented that monitor expects to output content in json format[1]? So this update could break it, does it matter now? [1] https://lore.kernel.org/linux-cxl/4c2341c8a4e579e9643b7daa3eb412b0ac0da98a.ca...@intel.com/ Thanks Zhijian > rc = monitor_event(ctx, &mfa); > out:
