Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751778AbdHaW6Y (ORCPT ); Thu, 31 Aug 2017 18:58:24 -0400 Received: from mail-wm0-f42.google.com ([74.125.82.42]:35138 "EHLO mail-wm0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751205AbdHaW6W (ORCPT ); Thu, 31 Aug 2017 18:58:22 -0400 X-Google-Smtp-Source: ADKCNb7+KhTZuTawSWQc5lq4zRQkPnXuKNtW42+4bT25dMq7wmOnkUWgHIw+2dzGaA5FrsduV688cA== Date: Fri, 1 Sep 2017 00:58:18 +0200 From: Frederic Weisbecker To: Rik van Riel Cc: LKML , Peter Zijlstra , Chris Metcalf , Thomas Gleixner , Luiz Capitulino , Christoph Lameter , "Paul E . McKenney" , Ingo Molnar , Mike Galbraith , Wanpeng Li Subject: Re: [RFC PATCH 01/12] housekeeping: Move housekeeping related code to its own file Message-ID: <20170831225816.GA18249@lerouge> References: <1503453071-952-1-git-send-email-fweisbec@gmail.com> <1503453071-952-2-git-send-email-fweisbec@gmail.com> <1504210576.12821.6.camel@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1504210576.12821.6.camel@redhat.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 867 Lines: 22 On Thu, Aug 31, 2017 at 04:16:16PM -0400, Rik van Riel wrote: > On Wed, 2017-08-23 at 03:51 +0200, Frederic Weisbecker wrote: > > The housekeeping code is currently tied to the nohz code. As we are > > planning to make housekeeping independant from it, start with moving > > the relevant code to its own file. > > > Why are nohz full and housekeeping being > decoupled from each other? > > Won't people want to use them together? > > What use case am I missing? So nohz is really just a feature and it shouldn't decide about other isolation features. It should be the opposite: isolation picks up nohz, alongside other isolation things. I think we did a layering misdesign. So it's mostly just a code reorganisation. While at it, isolcpus= is also part of the isolation toolset. So I thought we should centralize all this isolation code in a common subsystem.