Re: [PATCH spice-gtk v2] channel-display-gst: Use recorder for frame statistics

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

On Tue, Feb 19, 2019 at 6:02 PM Victor Toso <victortoso@xxxxxxxxxx> wrote:
>
> Hi,
>
> On Tue, Feb 19, 2019 at 05:37:38PM +0100, Marc-André Lureau wrote:
> > > > > -        SPICE_DEBUG("frame mm_time %u size %u creation time %"
> > > > > G_GINT64_FORMAT
> > > > > -                    " decoded time %" G_GINT64_FORMAT " queue %u",
> > > > > -                    frame->mm_time, frame->size, frame->creation_time,
> > > > > -                    duration, decoder->decoding_queue->length);
> > > > > +        record(frames_stats,
> > > > > +               "frame mm_time %u size %u creation time %" PRId64
> > > > > +               " decoded time %" PRId64 " queue %u",
> > > > > +               frame->mm_time, frame->size, frame->creation_time,
> > > > > +               duration, decoder->decoding_queue->length);
> > > >
> > > > Why SPICE_DEBUG log is removed?
> > > >
> > > > Why is the "recorder" stuff necessary here?
> > > >
> > >
> > > Measurement instrumentation
> >
> > Ok, but can we get have the regular log as well?
>
> This really is measurement type of logs. We don't have structured

And "trace"(?) are better than "log" for "measurements"?

> logging so I do prefer this kind of logging being enabled by some
> dynamic tweaking instead of seeing it all the time.

You can filter it out with grep, fairly easily.

I proposed a series using structured logging and categories for
SPICE_DEBUG= in the past iirc. I guess I should revisit it.




-- 
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]