Re: [ndctl PATCH v3 1/6] cxl/monitor: Enable default_log and refactor sanity check

[Verma, Vishal L]

On Mon, 2023-07-10 at 10:53 +0000, Zhijian Li (Fujitsu) wrote:
> > > 
> > >          log_init(&monitor.ctx, "cxl/monitor",
> > > "CXL_MONITOR_LOG");
> > > -       monitor.ctx.log_fn = log_standard;
> > > +       if (monitor.log)
> > > +               log = monitor.log;
> > > +       else
> > > +               log = monitor.daemon ? default_log :
> > > "./standard";
> > 
> > I think the original './standard' was used that way because
> > fix_filename() added the './' prefix. 
> 
> At present, './' will still be added by parse_options_prefix() at the
> beginning.
> 
Ah yes I missed this - I must've confused my test attempts - you're
right, 'standard' behaves as expected (stdout) and './standard also
works as expected (creates a file called standard in the cwd).