[PATCH 08/35] monitor: add command line options to control logging
Martin Wilck
mwilck at suse.com
Thu Feb 4 04:18:33 EST 2021
On Thu, 2021-02-04 at 08:04 +0100, Hannes Reinecke wrote:
> On 1/26/21 9:32 PM, mwilck at suse.com wrote:
> > From: Martin Wilck <mwilck at suse.com>
> >
> > Standard syslog loglevels are used.
> > Options:
> >
> > -S/--silent to suppress LOG_NOTICE
> > -v/--verbose to enable LOG_INFO (overrides -S)
> > -D/--debug to enable LOG_DEBUG (overrides -S, -v)
> > -C/--clockstamps to enable time stamps on log messages.
> >
> > I tried to use short options that don't conflict with options
> > from nvme connect-all, because many of those options might be
> > useful for the monitor, too.
> >
> > Signed-off-by: Martin Wilck <mwilck at suse.com>
> > ---
> > monitor.c | 47 +++++++++++++++++++++++++++++++++++++++++------
> > 1 file changed, 41 insertions(+), 6 deletions(-)
> > ... and if you had moved the logging functionality prior to the
> > monitor
> code, this patch could be merged with the main monitor functionality,
> too.
I expected that reviewers preferred smaller chunks. If that's not the
case, fine with me, I'll change i
Thanks,
Martin
More information about the Linux-nvme
mailing list