Received: by 2002:a05:6358:701b:b0:131:369:b2a3 with SMTP id 27csp2950186rwo; Mon, 24 Jul 2023 04:09:00 -0700 (PDT) X-Google-Smtp-Source: APBJJlHDDJhu7IM875UapZIV7KuQxaDPHBZq1yrZ2a4Xn+QfTjKZRyZA9SHW6cNvkV4AGSNKCeka X-Received: by 2002:a17:902:ecc3:b0:1bb:141c:3034 with SMTP id a3-20020a170902ecc300b001bb141c3034mr10411565plh.12.1690196940395; Mon, 24 Jul 2023 04:09:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1690196940; cv=none; d=google.com; s=arc-20160816; b=Mf9txSsZoQnhniddNs32Gea7fDRWWIPw6Z4DNoGHiIxYMf4mLsgRPrEaD4QQNzb2r/ EHD44XkfnFTCA9mrCYul/wOwbVxrapYbqjOpzTYZssDL8ZrJ3/b8vaLXRzYa6BfM3Wil oCCIjBJ1Yp4KzjEL7iwfbQTHxJRZ0h7fhB2YLYhpvD3mW6T2IPE/bIw+k8yiTenfh9uN 83vqjE7/4ZATJ2At+qBxQavY3HXBwmczHR6Mw22A3RQJpJtnYhBHbTYuhfDp6zRe10f3 CDc31mnf+Ucwoa3D0tyVnvEG65ILtLFqramVs0r7bqX9lz+dWPeEby0fExbQgH6yIFO6 7YNQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:from:references:cc:to :content-language:subject:mime-version:date:message-id:feedback-id :dkim-signature:dkim-signature; bh=bRwBeznDm7R257YGjVw6nEKDUe4UcUMR/8qE2bCz4vc=; fh=a6YSzKfo3DoQ3p0/z8k5jihwyIkOsElUvbP/LrUGgPE=; b=Va76PIftK3BzoslLMbOnpwML4Jlq0Yux1SUGilXHs34HOBZYvbV8urrUJ4NrMde/fc 3M21z0bZD5b0cjaCVQakPgG072RXjMR3hT3iuXxM5X9dpe/k3/dAiBlb9hNSKDtVmywI MbDqQTf7Ktzu5QEG6JuRDYo3zApEmnFQyWRAvowryPaO7D3SeOLF8eBEWhVAK45mRjH9 7bl3wTt+TPmzpql4dpGg7ohZGVrtnVKt+tpYWFxg+S66fkWOY0+mDWwo+4Hk9FMQcF36 pPzz8bwmWPl3qa3k25DaMKQRyCr8sthohtoaXa9nnRAtjyYCuAeKEAUGrd7602GE6dyc CIUA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@invisiblethingslab.com header.s=fm3 header.b=GX2ZDMvA; dkim=pass header.i=@messagingengine.com header.s=fm3 header.b=0CY6hNqR; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id c2-20020a170902d48200b001bba879700fsi1530455plg.514.2023.07.24.04.08.47; Mon, 24 Jul 2023 04:09:00 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@invisiblethingslab.com header.s=fm3 header.b=GX2ZDMvA; dkim=pass header.i=@messagingengine.com header.s=fm3 header.b=0CY6hNqR; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229583AbjGXKbQ (ORCPT + 99 others); Mon, 24 Jul 2023 06:31:16 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59148 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231270AbjGXKa7 (ORCPT ); Mon, 24 Jul 2023 06:30:59 -0400 Received: from wout4-smtp.messagingengine.com (wout4-smtp.messagingengine.com [64.147.123.20]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 91138199C for ; Mon, 24 Jul 2023 03:29:28 -0700 (PDT) Received: from compute2.internal (compute2.nyi.internal [10.202.2.46]) by mailout.west.internal (Postfix) with ESMTP id E886B320091C; Mon, 24 Jul 2023 06:29:24 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute2.internal (MEProxy); Mon, 24 Jul 2023 06:29:25 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= invisiblethingslab.com; h=cc:cc:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to; s=fm3; t= 1690194564; x=1690280964; bh=bRwBeznDm7R257YGjVw6nEKDUe4UcUMR/8q E2bCz4vc=; b=GX2ZDMvAXgehf2SahEhgC/O4Cd/11DUqRB4BEwVtVLEZHRQCz7w wuARVsCq8jmdpJlCVBWu5P7fUuD96g2XeAkF2GvVafQdE0cv3QK9LCje/B26nQtL pKN9mhe0XnYsOn014yTIxaSLoq4tiCuQ07xB01LaoGh73DrH4EERYU3Ko9gzqqKw vPyZOdhqANRpVSRyRvLVkVac6PR1amCUJq35F/R8JBNJkW/go21J0YI6pnpw9n/Z F1ykGw3cWq56UqrXOWHgubhaBaLXAo4zkkA2H7JmBtVopZVlH/REGPr//C+LEO8M JMZzaY/Zj6yZ1WeMFcWPDRcaD/ToWgvxV2Q== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm3; t=1690194564; x=1690280964; bh=bRwBeznDm7R25 7YGjVw6nEKDUe4UcUMR/8qE2bCz4vc=; b=0CY6hNqRUvzNjyEMbdOXDFJN71QjD zHNksNkVVRhUKnrLsRJKzBMJz0ILMbDRbhoNyOO0xahVyXHkWRPGhkVuS1pNjDRR oC69Pg8F0xw3N9tj0/rirn8oDhlcWusRNmWh83m90FPEeDrUX8zyeyAMKuTbc8BM csz3gosk7BvwGrrwWmDsrLx5/plVTCt7fJyKOEBHWcro5YRr6Xwm6jG0Nqwe5DGy 71KYbSfuPHKVLnQh53Vel+tfBYFHfRDIsHz5+heGJ4AvYQ8LO4IrG96SobEQyd+A KO3p+DK2Q1It9YjgvQEyc0JhAzrPAaVT6TUA787bejxWxL5BAI7Ls4tXw== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedviedrheekgddvjecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefkffgguffvvehfhfgjtgesghdtreertddtjeenucfhrhhomhepufhimhhonhcu ifgrihhsvghruceoshhimhhonhesihhnvhhishhisghlvghthhhinhhgshhlrggsrdgtoh hmqeenucggtffrrghtthgvrhhnpeeftdefudeltdevhfehkeefhfeutdfhffdugeetffdv gfehfeekhfetgedvuefggfenucffohhmrghinhepkhgvrhhnvghlrdhorhhgnecuvehluh hsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepshhimhhonhesihhn vhhishhisghlvghthhhinhhgshhlrggsrdgtohhm X-ME-Proxy: Feedback-ID: idc5945a3:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Mon, 24 Jul 2023 06:29:22 -0400 (EDT) Message-ID: <81ca146e-986c-b934-3567-2e1267a31ad7@invisiblethingslab.com> Date: Mon, 24 Jul 2023 12:29:09 +0200 MIME-Version: 1.0 Subject: Re: [PATCH] xen/events: Add wakeup support to xen-pirq Content-Language: en-US To: Juergen Gross , Stefano Stabellini , Oleksandr Tyshchenko Cc: xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org References: <20230313134102.3157-1-simon@invisiblethingslab.com> From: Simon Gaiser In-Reply-To: <20230313134102.3157-1-simon@invisiblethingslab.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="------------Hf0VWW22SHCK1Z6ZaG9MPeYj" X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H5,RCVD_IN_MSPIKE_WL,SPF_HELO_PASS,SPF_NONE, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --------------Hf0VWW22SHCK1Z6ZaG9MPeYj Content-Type: multipart/mixed; boundary="------------NM7xgrV6rptQJ4hiPTQm8Yjd"; protected-headers="v1" From: Simon Gaiser To: Juergen Gross , Stefano Stabellini , Oleksandr Tyshchenko Cc: xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org Message-ID: <81ca146e-986c-b934-3567-2e1267a31ad7@invisiblethingslab.com> Subject: Re: [PATCH] xen/events: Add wakeup support to xen-pirq References: <20230313134102.3157-1-simon@invisiblethingslab.com> In-Reply-To: <20230313134102.3157-1-simon@invisiblethingslab.com> --------------NM7xgrV6rptQJ4hiPTQm8Yjd Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Simon Gaiser: > This allows entering and exiting s2idle. Actual S0ix residency is > another topic [1]. >=20 > Without this the ACPI code currently ignores the error enable_irq_wake(= ) > returns when being used on a xen-pirq and the system goes to idle for > ever since the wakeup IRQ doesn't gets enabled. With [2] the error is > handled and the system refuses to go to s2idle. >=20 > Link: https://lore.kernel.org/xen-devel/9051e484-b128-715a-9253-48af8e4= 7bb9d@invisiblethingslab.com/ # [1] > Link: https://lore.kernel.org/linux-acpi/20230313125344.2893-1-simon@in= visiblethingslab.com/ # [2] > Signed-off-by: Simon Gaiser > --- >=20 > While I think that the set of flags I set is correct, I'm not familiar > with that code, so please pay special attention during review if they > are actually correct for xen-pirq. >=20 > drivers/xen/events/events_base.c | 4 ++++ > 1 file changed, 4 insertions(+) >=20 > diff --git a/drivers/xen/events/events_base.c b/drivers/xen/events/even= ts_base.c > index c7715f8bd452..991082f04f05 100644 > --- a/drivers/xen/events/events_base.c > +++ b/drivers/xen/events/events_base.c > @@ -2176,6 +2176,10 @@ static struct irq_chip xen_pirq_chip __read_most= ly =3D { > .irq_set_affinity =3D set_affinity_irq, > =20 > .irq_retrigger =3D retrigger_dynirq, > + > + .flags =3D IRQCHIP_SKIP_SET_WAKE | > + IRQCHIP_ENABLE_WAKEUP_ON_SUSPEND | > + IRQCHIP_MASK_ON_SUSPEND, > }; > =20 > static struct irq_chip xen_percpu_chip __read_mostly =3D { ping --------------NM7xgrV6rptQJ4hiPTQm8Yjd-- --------------Hf0VWW22SHCK1Z6ZaG9MPeYj Content-Type: application/pgp-signature; name="OpenPGP_signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="OpenPGP_signature" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE3E8ezGzG3N1CTQ//kO9xfO/xly8FAmS+UncACgkQkO9xfO/x ly9H4w/9H6fMRreUIMniGKL0lM56B1kdKr6WGIfsRP979zHetmBSz0dKKcjJ7e5b M+ljI+bNMocw/Pq4ph6EKLtrkCjsfH51+44EnZJmDgONbF5kZbxOZc1/YB+WHmWm qr1eRhq6p7ORtCCRiHdc2Rpr72HMoxc+YDNLhfxdF10j3CVI2jShuEp/hf46zZnM T3JSVXd1boQo09O2pt6Ka7BjbblfINJ0wT6zNdyhOeBEZMDI5siFP6cddKf+nrAo 2m7CYP7Njgr0IALV1OC4NC39s+qEx7uNzBTVSLJkBpYvyvKC6XfHyJavyaY40MA7 iyeA8hjq6IWCsZ67OcjAokF06ox0xr4ll7H8kx7QAZSwtSTbC+gK4zBgPn/DBAD1 NWj8qHhxFl3oEb/KBZXTg+0JuABbU7ARk/JZhZxR62WZU2hwHMzUViy1hzUvOzty b1rzZTi3SsSmTaxZOr/vEHBkdde0FYQpLRowLs0v0uUvYB227Ey/HNxr0DJCC6CJ WhcArncJhkRV+A5kYy47iLJeDMQTElGCRNm1Anm42BMo475b2GT2CWcw0Spnhnnl pqGSfJISZTQ2HdZmcwFOzDnwBpTcByYsBI5ou5m2/JfRx5UWNGcsxRQaXSAp5IQ4 QelAecD+Fh6LNjnWn3AXXDhBL3o40A+upI4T1prOr74lPjYZudI= =hLGI -----END PGP SIGNATURE----- --------------Hf0VWW22SHCK1Z6ZaG9MPeYj--