Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754855AbbEUONH (ORCPT ); Thu, 21 May 2015 10:13:07 -0400 Received: from bombadil.infradead.org ([198.137.202.9]:52067 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751702AbbEUONF (ORCPT ); Thu, 21 May 2015 10:13:05 -0400 Date: Thu, 21 May 2015 16:12:59 +0200 From: Peter Zijlstra To: Stephane Eranian Cc: Ingo Molnar , Vince Weaver , Jiri Olsa , LKML Subject: Re: [PATCH 09/10] perf/x86: Remove intel_excl_states::init_state Message-ID: <20150521141259.GU3644@twins.programming.kicks-ass.net> References: <20150521111710.475482798@infradead.org> <20150521111933.412630324@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2012-12-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1373 Lines: 27 On Thu, May 21, 2015 at 06:39:29AM -0700, Stephane Eranian wrote: > On Thu, May 21, 2015 at 4:17 AM, Peter Zijlstra wrote: > > For some obscure reason intel_{start,stop}_scheduling() copy the HT > > state to an intermediate array. This would make sense if we ever were > > to make changes to it which we'd have to discard. > > > > Except we don't. By the time we call intel_commit_scheduling() we're; > > as the name implies; committed to them. We'll never back out. > > > Well, I remember there was a reason why this state was introduced. I thought > we could do it without initially but had to add it to solve a problem. > We do backtrack > in the scheduling algorithm because of the tincremental transaction between the > generic layer and the x86 layer. I'll have to remember the exact case where this > happens. > > Now, it may be that because you changed the logic of the xl vs. xlo, this saved > state is not needed anymore. I'll have to look at that some more. Another hint its pointless is that stop does an unconditional copy back. So no matter what we did, all changes are always published. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/