[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 07/10] xen/physinfo: encode Arm SVE vector length in arch_capabilities
- To: Luca Fancellu <luca.fancellu@xxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Wed, 15 Mar 2023 10:41:45 +0100
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=TwzEvm9XOJIlS1DQsIx/e2DBLzmRCg1SiOgzZP7dD7s=; b=BcCt/b39fJDNndNJWNujcTE5Hfu5UyjhiqVgCTJuSGZRgwhvhGbdWIoWwrnM5J0B7ypHxTmQWxcVWEicRNUQ8iyK9GHh98qRFCKc/OhBjz1BkG0JGz5Xvu+x0qn5P8cpj6GXH5gBM8bR8BJvzllLcjRrSv2YykIR4AwqoDlIhJtndXJkP8FmUR1w/pGjZSoR3+VAd7IawWQq6wNGewjfA+3FJBIxjZSaY92IyAUCC01l2k8C/6MSI+IYdu0GRSQO1jAxOdmDWlRpR9fUptsFSY4OJouQfaYi6WVbG1cheHV9TVnLom7sQMsTQQPhukHrk1uOvfE9KTLD9+IdtEGldQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=V68CyonK/zMf5bTOQk7cw5XWgLrFnZpqbBrnM3o7Xs9yPqLTcmlxMpkow0Lxbs+RwQ4GbSY0gkzSJLQoX5QPRzJi+/B1rdsKetCeaBHWbVi3zoJZRkYkIuDzEsABvK2BtT6vsvOgJPa1Y1ZBTN3ZTyTepVkctLvN13bB8oJp/+2bOumydiaTYbu7nmErpEgZJLg7CxCgQaUKmJ5EA4rj5DhlNd41jGGneG7le3od/9NrUJF9MD45qG3BBru/93vc+T86bbS6TgdrfFDq01DBMYgJUXbEIK9ZVTHUbOPBY/CXskOU6JhcSbAcJbcxqYih4iQuHpOeArRyS9PjsSWOFw==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: bertrand.marquis@xxxxxxx, wei.chen@xxxxxxx, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
- Delivery-date: Wed, 15 Mar 2023 09:41:57 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 15.03.2023 10:05, Luca Fancellu wrote:
> --- a/xen/include/public/sysctl.h
> +++ b/xen/include/public/sysctl.h
> @@ -94,6 +94,9 @@ struct xen_sysctl_tbuf_op {
> /* Max XEN_SYSCTL_PHYSCAP_* constant. Used for ABI checking. */
> #define XEN_SYSCTL_PHYSCAP_MAX XEN_SYSCTL_PHYSCAP_gnttab_v2
>
> +#define XEN_SYSCTL_PHYSCAP_ARM_SVE_MASK (0x1FU)
> +#define XEN_SYSCTL_PHYSCAP_ARM_SVE_SHFT (0)
The second of these can be inferred from the first, so I'd like to ask
that redundant definitions be omitted from the public headers. For the
code using the constant we specifically have MASK_INSR().
Just like there already are x86-specific sections in this file, I think
the remaining single #define also wants enclosing in "#ifdef __aarch64__"
here.
Jan
|
