[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 2/3] virtiofsd: add DBus server to handle log leve
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [RFC 2/3] virtiofsd: add DBus server to handle log level changes |
Date: |
Fri, 6 Sep 2019 11:23:28 +0100 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
On Thu, Sep 05, 2019 at 06:27:32PM +0100, Dr. David Alan Gilbert wrote:
> * Stefan Hajnoczi (address@hidden) wrote:
> > Introduce a DBus server thread that runs alongside the other virtiofsd
> > threads. It processes changes to the /org/qemu/virtiofsd object which
> > can be accessed at the org.qemu.virtiofsd location on the bus.
> >
> > This code does not use locking because we are the only writer to the
> > int current_log_level variable. More advanced management commands would
> > require locking to prevent race conditions with the other threads.
> >
> > Signed-off-by: Stefan Hajnoczi <address@hidden>
>
> OK, that is less complex than I'd feared.
> I guess there's something probably nice to do with name/integer mapping
> for warning levels that we could use from one of the libraries.
I used a free-form string because it's what systemd's LogLevel property
also does. But I can investigate the cleanest approach for limiting it
to a set of string constants.
Stefan
signature.asc
Description: PGP signature
[Qemu-devel] [RFC 3/3] virtiofsd: add virtiofsctl command-line management tool, Stefan Hajnoczi, 2019/09/05
Re: [Qemu-devel] [RFC 0/3] virtiofsd: get/set log level via DBus, Dr. David Alan Gilbert, 2019/09/05