From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757650Ab1KQOMp (ORCPT ); Thu, 17 Nov 2011 09:12:45 -0500 Received: from mx1.redhat.com ([209.132.183.28]:9264 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757597Ab1KQOMn (ORCPT ); Thu, 17 Nov 2011 09:12:43 -0500 Message-ID: <4EC51636.3030001@redhat.com> Date: Thu, 17 Nov 2011 16:12:06 +0200 From: Avi Kivity User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:7.0.1) Gecko/20110930 Thunderbird/7.0.1 MIME-Version: 1.0 To: Peter Zijlstra CC: Gleb Natapov , linux-kernel@vger.kernel.org, mingo@elte.hu, Jason Baron , rostedt , Thomas Gleixner Subject: Re: [PATCH RFC] remove jump_label optimization for perf sched events References: <20111117123029.GB16853@redhat.com> <1321534159.27735.33.camel@twins> <20111117130032.GC16853@redhat.com> <1321535413.27735.35.camel@twins> <4EC50B02.1060307@redhat.com> <1321537668.27735.38.camel@twins> In-Reply-To: <1321537668.27735.38.camel@twins> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/17/2011 03:47 PM, Peter Zijlstra wrote: > On Thu, 2011-11-17 at 15:24 +0200, Avi Kivity wrote: > > On 11/17/2011 03:10 PM, Peter Zijlstra wrote: > > > On Thu, 2011-11-17 at 15:00 +0200, Gleb Natapov wrote: > > > > > > > > That said, I'd much rather throttle this particular jump label than > > > > > remove it altogether, some people really don't like all this scheduler > > > > > hot path crap. > > > > What about moving perf_event_task_sched() to sched_(in|out)_preempt_notifiers? > > > > preempt notifiers checking is already on the scheduler hot path, so no > > > > additional overhead for perf case. > > > > > > Same problem really, some people complain about the overhead of preempt > > > notifiers, also not all kernels have those in. > > > > We could combine the two, sort-circuit preempt notifiers with jump > > labels if empty && not much activity on them. > > Jump-labels are still more efficient, also I don't much like preempt > notifiers. > > > > Futhermore I loathe notifier lists because they obscure wtf is done. > > > > That's life in a general purpose kernel, if everyone gets their hook in > > to keep their code clean, the scheduler will bloat. > > Uhm, no. The bloat isn't different, the only difference is you can > actually see it. So I very much prefer direct hooks. > > > An advantage of preempt notifiers is that you can make the perf code > > modular. > > Yeah, and you know I loathe modules even more. Is there something you like? -- error compiling committee.c: too many arguments to function