Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Mon, 26 Mar 2018 16:23:02 +0200 (CEST)
From:   Thomas Gleixner <tglx@linutronix.de>
To:     Eric Dumazet <edumazet@google.com>
cc:     Ingo Molnar <mingo@kernel.org>, Borislav Petkov <bp@alien8.de>,
        Eric Dumazet <eric.dumazet@gmail.com>,
        the arch/x86 maintainers <x86@kernel.org>,
        LKML <linux-kernel@vger.kernel.org>,
        "H. Peter Anvin" <hpa@zytor.com>, Ingo Molnar <mingo@redhat.com>,
        Hugh Dickins <hughd@google.com>, a.p.zijlstra@chello.nl
Subject: Re: [PATCH v3 1/2] x86, msr: allow rdmsr_safe_on_cpu() to schedule
In-Reply-To: <CANn89iKy_jVpBvAebJFm1UKVKfG=p+R4B1tXmC4waeK7YzZh2g@mail.gmail.com>
Message-ID: <alpine.DEB.2.21.1803261621430.1585@nanos.tec.linutronix.de>
References: <20180323215818.127774-1-edumazet@google.com> <20180324080946.3db4xdkl5i6jx2rc@gmail.com> <336355a3-c11d-44fc-0642-671670980ac0@gmail.com> <20180325141242.GC21878@pd.tnic> <20180326064042.c6xod5n24kqttdiw@gmail.com>
 <CANn89iKy_jVpBvAebJFm1UKVKfG=p+R4B1tXmC4waeK7YzZh2g@mail.gmail.com>
User-Agent: Alpine 2.21 (DEB 202 2017-01-01)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Mon, 26 Mar 2018, Eric Dumazet wrote:
> On Sun, Mar 25, 2018 at 11:40 PM Ingo Molnar <mingo@kernel.org> 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.

Thanks,

	tglx