From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 49F631F4CB7 for ; Wed, 12 Mar 2025 14:11:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.129.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741788682; cv=none; b=WLBmxRK+EI6rQRPE/9IpcTQzj9GBhOG5NkWONU5nj/skbU1yOlDvI9QlkJESE+C7R/ZCf3TixbzKP0LUkcbKdbfOyvNP1CxNKTML+S5gNddLWlB0uPm0wm0rCN+lH83JhTWE7/moaaAmr65SMjKFIMgoJ5oPJVryoCg6tZ3aD2g= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1741788682; c=relaxed/simple; bh=+jSC05SJwjl/8d9CPD2/CFb6AqWqRmu7znZ++6eA6VU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=F4Sz59WBOQChPBmk3i6Bw1Ajvyp9RBvMQotzVrbVUgT0DrRtON3b+SfFqOsJBefy4Kc36k/BAcXuXTOVibvbC2KXK2AtXQ0Q8/bDIXnfKdm6RmWFZQxx0DSqZ/pxdB3AV1Lwnu0gXSBPbUay7nz7mq+LmhmCsU85/24EUABEb+Y= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=bbjmKPVk; arc=none smtp.client-ip=170.10.129.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="bbjmKPVk" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1741788680; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=aGnUvFsDdnQ+lw0bRojPA/ofxLu41cVYkHj8QUAW/FI=; b=bbjmKPVkBRh6sFT5HAIoLSSMfzTOQCzCSjak1V9rwSB8JO+ldRJXsuOxIp1NQCFfVPJGcj sAuB12F5qHvxtvmLOtwdfkjdpVmXm6K7WDUvrXPU56rEAdX40NZnnVvjQasifMyT4l/b2x e+mo8nrTousZXbhHNUToctfHxpwZmwg= Received: from mail-wr1-f71.google.com (mail-wr1-f71.google.com [209.85.221.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-341-Q3sywHjCOLKpOQn6w_UR0w-1; Wed, 12 Mar 2025 10:11:18 -0400 X-MC-Unique: Q3sywHjCOLKpOQn6w_UR0w-1 X-Mimecast-MFC-AGG-ID: Q3sywHjCOLKpOQn6w_UR0w_1741788678 Received: by mail-wr1-f71.google.com with SMTP id ffacd0b85a97d-391345e3aa3so3271390f8f.0 for ; Wed, 12 Mar 2025 07:11:18 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1741788677; x=1742393477; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=aGnUvFsDdnQ+lw0bRojPA/ofxLu41cVYkHj8QUAW/FI=; b=lfBRjx4bbCJHSiXa+I/DEtgckpDw7d3bWgvRoI7pPbQD5Ie8YRg03XtDgaswAk653N roAmX3Om0KztRTF4WqMoIC5AV7AyD5Gz92SYGWcup8rmwr/FkKqzQqffBD5mbSuMY47S qzsdHz8oAATtYTpMrOUQa7bxpKpdlSG9tEBHSAL1FUPu1XoCuhv8yoihamM/xd6F48HU 3TFgjurt0YNhXsV7ltsc10MO5FumCOm/98f+apYwSnfMDPXn9C1qqoMZPcF4K7MbvrzD XPQPMLuLKt3HSpQRsyaP//Ts4NXNBntIGJuxXBQy44YAvK1yOvKR2EdHavh5ZwekQ5xu L4Dw== X-Forwarded-Encrypted: i=1; AJvYcCWy88d0GwxLY6vcqJj5GFp9gejP9MUK+mrnPTn91ck+od/1g0wEEIhMws40t9d50HAVoShiuf/yYpA32ec=@vger.kernel.org X-Gm-Message-State: AOJu0YzER0M+ymyaqJcJfp/hXwq4asbB+1FeaDP/9FWsANf2VTAYaMks 6fbT0HRhmvSPvDCLH/8vA64ie2htSvHwqazmrz/9mmitoDPue6BCXswqcVy5Jd4/XJcqkWcw6vg 58I0tN+aNXHwVmdSr3v8HgB1MikPy2Cbf+3ChKyVLZxOXTp2sQRBO6JDHWOfLgw== X-Gm-Gg: ASbGncuDziQbfVPe1Xg4cyft98pMf4TUf5a2pZ8M18yIurt9g2mIvzjdtWhAlvWHxyf QbkCVcJCyMAKyquy1dDtuBHe2PebOrS7FUk98UxPf/ptL7fvcCWC9cIZ2qvjJ5ydSLNeNRnB0zn asSaTz0d60ppWYg3UjtHYBT1EMicEbZ32RVmJumFnXe1Oemji/jk3ffRTUy2QaNylh2UAD0c3a0 B30Rw+YVOGdJnOu3b5yIS4Q4dCVGSg8Ac0P6IZ5xjO661Ie2QUmiyyXh5MyTRJOPrzBE4Nw1d4J ODye2/GoippuopYWuHTm/PAnGE2Gzs7nMGgKxJJYOMw= X-Received: by 2002:a5d:6c6f:0:b0:38f:3224:660b with SMTP id ffacd0b85a97d-392641c041cmr8332722f8f.22.1741788677522; Wed, 12 Mar 2025 07:11:17 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGWA0cY2BC6XgXEbxnii+AMUXiizzDFxBVcUGH0TiCHuNDmWjJcAwhup4D7hTAbzVX27jsUTw== X-Received: by 2002:a5d:6c6f:0:b0:38f:3224:660b with SMTP id ffacd0b85a97d-392641c041cmr8332650f8f.22.1741788676892; Wed, 12 Mar 2025 07:11:16 -0700 (PDT) Received: from jlelli-thinkpadt14gen4.remote.csb ([151.29.49.7]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3912c01cb82sm21812071f8f.51.2025.03.12.07.11.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 12 Mar 2025 07:11:16 -0700 (PDT) Date: Wed, 12 Mar 2025 15:11:13 +0100 From: Juri Lelli To: Waiman Long Cc: Dietmar Eggemann , linux-kernel@vger.kernel.org, cgroups@vger.kernel.org, Ingo Molnar , Peter Zijlstra , Vincent Guittot , Steven Rostedt , Ben Segall , Mel Gorman , Valentin Schneider , Tejun Heo , Johannes Weiner , Michal =?iso-8859-1?Q?Koutn=FD?= , Qais Yousef , Sebastian Andrzej Siewior , Swapnil Sapkal , Shrikanth Hegde , Phil Auld , luca.abeni@santannapisa.it, tommaso.cucinotta@santannapisa.it, Jon Hunter Subject: Re: [PATCH v3 4/8] sched/deadline: Rebuild root domain accounting after every update Message-ID: References: <797146a4-97d6-442e-b2d3-f7c4f438d209@arm.com> <398c710f-2e4e-4b35-a8a3-4c8d64f2fe68@redhat.com> <7fb20de6-46a6-4e87-932e-dfc915fff3dc@redhat.com> <724e00ea-eb27-46f1-acc3-465c04ffc84d@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On 12/03/25 09:55, Waiman Long wrote: > On 3/12/25 6:09 AM, Juri Lelli wrote: > > On 12/03/25 10:53, Dietmar Eggemann wrote: > > > On 11/03/2025 15:51, Waiman Long wrote: > > ... > > > > > > You are right. cpuhp_tasks_frozen will be set in the suspend/resume > > > > case. In that case, we do need to add a cpuset helper to acquire the > > > > cpuset_mutex. A test patch as follows (no testing done yet): > > ... > > > > > This seems to work. > > Thanks for testing! > > > > Waiman, how do you like to proceed. Separate patch (in this case can you > > please send me that with changelog etc.) or incorporate your changes > > into my original patch and possibly, if you like, add Co-authored-by? > I think it will be better to merge into a single patch to avoid having a > broken patch. It is up to you if you want me as a co-author. I don't really > mind. > > > > > But what about a !CONFIG_CPUSETS build. In this case we won't have > > > this DL accounting update during suspend/resume since > > > dl_rebuild_rd_accounting() is empty. > > I unfortunately very much suspect !CPUSETS accounting is broken. But if > > that is indeed the case, it has been broken for a while. :( > Without CONFIG_CPUSETS, there will be one and only one global sched domain. > Will this still be a problem? Still need to double check. But I have a feeling we don't restore accounting correctly (at all?!) without CPUSETS. Orthogonal to this issue though, as if we don't, we didn't so far. :/ > > Will need to double check that, but I would probably do it later on > > separated from this set that at least seems to cure the most common > > cases. What do people think? > > I am not aware of any distros without setting CONFIG_CPUSETS. So it is > mostly a theoretical problem if there is one. So I would recommend going > ahead with the current patch series instead of spending more time > investigating this issue. And I would agree (and then find time to look better into !CPUSETS case). If nobody objects, I will refresh the series including Waiman's changes and repost. Thanks! Juri