From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f180.google.com (mail-pl1-f180.google.com [209.85.214.180]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6B46B2BDC0B for ; Mon, 24 Nov 2025 23:43:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.180 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764027806; cv=none; b=URn/EirLccodaqjsPGfRlUOZOJ/1yJ+IbAVow8X7MzgSHsDFijEV1mtXSLj1HTdEs/+PjhrdPchihn3AXahuKKdD6XzRohb7rfoCrDjTSxmMVS9KqFnS+YmOX94R44Z8OgrbIob+6hmaZrdqMBDzm2dwzmqYj6LJziVXQfjuc10= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1764027806; c=relaxed/simple; bh=0GnmzL1h+o+i6liQM8CrD5wefDSntZzSj6EmuSiJ1W4=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=BljoVNOctMN496LUpvzqgu2jzHk5yHsvN4ItFywbTgiLKyaRYpoGIQ2UTtH60SVGRsPRCaRJvVtIUVi+U6bAyj3RGAJZU4Eo5QB3AqcPUG2eI+Rlk4ZdWI95LG23MQTyp3/wXIG72jaUkfFWHp0o3m62E5Xp4k3EO712PYC5pwo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=nU8y+sSB; arc=none smtp.client-ip=209.85.214.180 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="nU8y+sSB" Received: by mail-pl1-f180.google.com with SMTP id d9443c01a7336-2958db8ae4fso49740415ad.2 for ; Mon, 24 Nov 2025 15:43:24 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1764027804; x=1764632604; darn=vger.kernel.org; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:from:to:cc:subject:date:message-id:reply-to; bh=gZmcA4UnelU4Uyw0x3ObtizJbVjq478wDgpSBsRd5ww=; b=nU8y+sSBRa+lbatcQ3hd1WggsGdcnjdm8W+1man3lBhQj0uSg4QhxCLijq2ssMZCAu H128FoUOZ6LJdWWmHJ7I87vbqqY+FsMcKCnw/AO+d/rZ6kJLcdOGte5gSmfCI5DWhBqZ qU4qpq8cMr43c9qCiIkR7Mkrv+QiUIbjkdxiGCHyT1GiYZuPDz+KE7xdD9vc2Co9mHoX dSsTEO+qRnUj02oDTUpTv0KoDxBOIfaduCAWRYUzZuWHkVaf6AYX2TXzaNNeGEkWRCfm ZHeOBT9DfZtVmCs+XFk1nrXr9/Imvp3nb7rDDSW8Ve5Nje88ROluPQq7xiUXTGJ6jtqw YkNQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1764027804; x=1764632604; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=gZmcA4UnelU4Uyw0x3ObtizJbVjq478wDgpSBsRd5ww=; b=eH0rZ8qULEZpD1IbP3jxNrGf1GILK6ABAzFXjOgu3ItHVjXjSm1ZVVU+mr1F5vBC4+ a68BT+x0TVcvxA0b/gc6LaYF6yA0sr2yyt0W5x7XzlJdzTheUgVH0rU3aPbfUc6JjoLG bHSOwUbWxV82WRAH3gbXlD3Adrja4wV22mRla+uRSCSq7w1WlW0lJZqabprRbJjHTi7e zjAcuBP4MtCkH6B9BtmFkezmWGGCK5YgnqVm2dA1vigjZ3av2xmRLKLWP6WfbAEzk1A+ zotUQi3rrebTvshYMcTzjNLQRZDZig6RMAumuVStw3igLa8cWA/u/xBkdf1e1CUd+QYN 1VYg== X-Forwarded-Encrypted: i=1; AJvYcCUVnownoyibJJIcJCPF8yVq8Ffu44vzwYH0YEpfoEbt677Xb+Smo+BpVAgYAxOqjw4NaN7yMAlIv4f8YMw=@vger.kernel.org X-Gm-Message-State: AOJu0YzrGu8iWoIbh7tB9yLg8k7kIr5DC1dRxFlPtfPeRmkQRvjvI3tV GBgRzx83bQ0n0jvkfKU8UoNU2qwowYLjRDYZvF1s0uEDBxx/SErXT+RDIkBvBsHJGGQ= X-Gm-Gg: ASbGncuhAOwCSlN26ImbuMNl54pHddJcgFfkDBNViu7rQTmd3M1dw/7FknWzhkgb8PI 7fsigw+cQyvv3Pa+srtfcuwDFlYeLm7GiclZA+Afs8QxW+3V2Hyfxxle6tBPJbPWa/+7TWtKy+I v6n/6pXbC+gMe1W9CfhIXyBZ3bhGVN8jMCbWteYcHpiUu8lZitU5NbH4lvXkTLaA8JpLyAD7caX Pcw/laohjh6VWMiWm8LsiEJ3zmY/MoXjl+jr/m50BT5ntMTy+jfUyTrLPIwlyAEJsZEd5uJexle sn1SYgQvSji5V7zlGohzZIeTHqzWA3gzK938R61TfyTyX65v8XvwSvNZ7IYp8CyEZnQISNHdDiP Osbf4Vhsblqd/xwAHdnMtBsIYbxB9z5Mx81ZAuheZPKz6GwCsX6ZpUdyaF/Po0MCW/f9iIeOF6Y 9fde3ymQgc X-Google-Smtp-Source: AGHT+IE54CeWlou3/0N68zXQXCzYzwWHDBiUc3WLIubNGQV4eRxSRlBCwcc7GxKyinH/+t/QnN92Og== X-Received: by 2002:a17:902:d4ca:b0:294:f6e2:cea1 with SMTP id d9443c01a7336-29b6c571f11mr147367015ad.38.1764027803678; Mon, 24 Nov 2025 15:43:23 -0800 (PST) Received: from localhost ([71.212.208.158]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-29b5b29b1c2sm145545895ad.81.2025.11.24.15.43.22 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 24 Nov 2025 15:43:23 -0800 (PST) From: Kevin Hilman To: Ulf Hansson , "Rafael J . Wysocki" , linux-pm@vger.kernel.org Cc: Vincent Guittot , Peter Zijlstra , Pavel Machek , Len Brown , Daniel Lezcano , Maulik Shah , Prasad Sodagudi , Dhruva Gole , Deepti Jaggi , Ulf Hansson , linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 0/6] PM: QoS: Introduce a CPU system wakeup QoS limit for s2idle In-Reply-To: <20251121100315.316300-1-ulf.hansson@linaro.org> References: <20251121100315.316300-1-ulf.hansson@linaro.org> Date: Mon, 24 Nov 2025 15:43:22 -0800 Message-ID: <7hcy57dmjp.fsf@baylibre.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 Ulf Hansson writes: > Changes in v3: > - Take new the new QoS limit into account for cpuidle too (Rafael). > - Add a new Kconfig for the new QoS interface (Rafael) > - Improved the documentation (Dhruva) > - Clarified commit messages and added acks. > > Changes in v2: > - Limit the new QoS to CPUs and make some corresponding renaming of the > functions along with name of the device node for user space. > - Make sure we deal with the failure/error path correctly when there are > no state available for s2idle. > - Add documentation. > > Some platforms supports multiple low power states for CPUs that can be used > when entering system-wide suspend and s2idle in particular. Currently we are > always selecting the deepest possible state for the CPUs, which can break the > system wakeup latency constraint that may be required for a use case. > > Therefore, this series suggests to introduce a new interface for user space, > allowing us to specify the CPU system wakeup QoS limit. The QoS limit is then > taken into account when selecting a suitable low power state for s2idle/cpuidle. Reviewed-by: Kevin Hilman (TI) Tested-by: Kevin Hilman (TI)