Re: [PATCH v3 1/2] x86, msr: allow rdmsr_safe_on_cpu() to schedule

From: Ingo Molnar
Date: Tue Mar 27 2018 - 05:36:58 EST



* Thomas Gleixner <tglx@xxxxxxxxxxxxx> wrote:

> On Mon, 26 Mar 2018, Eric Dumazet wrote:
> > On Sun, Mar 25, 2018 at 11:40 PM Ingo Molnar <mingo@xxxxxxxxxx> wrote:
> > > If performance matters then the relevant MSR events should be added and
> > > gsysd
> > > should be updated to support MSR events.
> > >
> >
> > Thanks for the hints Ingo. I have forwarded them.
> >
> > I am pretty sure gsysd was designed and written years before perf even
> > existed,
> > so I am not sure all this will ever be implemented.
> >
> > I guess I will apply the kernel patches locally for the time being.
>
> I'm still considering to merge them because there is no real reason not to
> do so. The interfaces exist and are used. Reducing their impact is
> certainly worthwhile.

Ok, that's a good point, and the patches themselves are sane:

Acked-by: Ingo Molnar <mingo@xxxxxxxxxx>

Thanks,

Ingo