Re: [ndctl PATCH v3 0/6] cxl/monitor and ndctl/monitor fixes

["Zhijian Li (Fujitsu)" <lizhijian@fujitsu.com>]

kindly ping

It has been almost a month, and it's doing some bugfix. The progress is a little bit slow anyway :)



On 31/05/2023 10:19, Li Zhijian wrote:
> V3:
> - update comit log of patch3 and patch6 per Dave's comments.
> 
> V2:
> - exchange order of previous patch1 and patch2
> - add reviewed tag in patch5
> - commit log improvements
> 
> It mainly fix monitor not working when log file is specified. For
> example
> $ cxl monitor -l ./cxl-monitor.log
> It seems that someone missed something at the begining.
> 
> Future, it compares the filename with reserved word more accurately
> 
> patch1-2: It re-enables logfile(including default_log) functionality
> and simplify the sanity check in the combination relative path file
> and daemon mode.
> 
> patch3 and patch6 change strncmp to strcmp to compare the acurrate
> reserved words.
> 
> Li Zhijian (6):
>    cxl/monitor: Enable default_log and refactor sanity check
>    cxl/monitor: replace monitor.log_file with monitor.ctx.log_file
>    cxl/monitor: use strcmp to compare the reserved word
>    cxl/monitor: always log started message
>    Documentation/cxl/cxl-monitor.txt: Fix inaccurate description
>    ndctl/monitor: use strcmp to compare the reserved word
> 
>   Documentation/cxl/cxl-monitor.txt |  3 +--
>   cxl/monitor.c                     | 45 ++++++++++++++++---------------
>   ndctl/monitor.c                   |  4 +--
>   3 files changed, 26 insertions(+), 26 deletions(-)
>