[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 6/7] xsm: drop generic event channel labeling exclusion
- To: "Daniel P. Smith" <dpsmith@xxxxxxxxxxxxxxxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Wed, 25 Aug 2021 17:44:18 +0200
- 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-SenderADCheck; bh=THhpKqPmKaaatPXjE//iDhSYKTHZjgw+fjAN+cCN9vU=; b=ioOwcV7eY3ruEvk0AEe49cG2ez4XuT35Zg73GhM3SxWZYr22mdBmp0vwNBQEMiDaYSp7W33cu5epLNtZYaW05TxZM7nskpy3IGM1r43KATQJrqmCyyTmkCjRoZ8LCWkV9xhhPGmEV/+q4TSSh3wPAHyJC2gNxTaWFwl57GgrNrJTFgT5zXDyX2YVPVpUGZmRSLeAliS1qP6TzXpUQYhE/tUNrMHWF75899Od9eWhvedbEZNDAbeBqPG0UPBLuf56vNrfbXtM2ov2Wzqpk6vkYHgRPbyWjs4yu0krlHUY0x7x6D9W1zB1jsV37wPYT7WRoHPnaqqiy3VUb4vWoQaGVA==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZbVgaP2gMyes2X4AkbgkQ+ppoCBEJb5P93pT44aCx7bI91XH7Me639BBqAxSd3r9FLrnNAG8aNez3gcmLbs/sXv0hTwhsCGgRw03XVL6bKNJvjxt38RBIISdBiipTCiCWi1A0dPO3h8GVFkdaV4dhVLUmcMQiGM3OVSWCf+x7fRjXMwQRxYTacHwi4SiogwTaix8SCymw78SaXyujO9D+O7r1QvpA3w/smIZ8sa6llPAJa7ZHuFeP9Aa9/pXBR7YHXVvQh9q8FYWRpuc+oKKZ+stvFHd8hPhvCEBn4SvtLSR0/HOcOBVcz1c5WqzSoZOrSrnXtVWGmXy9/UW4Qty3w==
- Authentication-results: lists.xenproject.org; dkim=none (message not signed) header.d=none;lists.xenproject.org; dmarc=none action=none header.from=suse.com;
- Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
- Delivery-date: Wed, 25 Aug 2021 15:44:34 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 05.08.2021 16:06, Daniel P. Smith wrote:
> The internal define flag is not used by any XSM module, removing the #ifdef
> leaving the generic event channel labeling as always present.
With this description ...
> --- a/xen/include/xen/sched.h
> +++ b/xen/include/xen/sched.h
> @@ -120,15 +120,12 @@ struct evtchn
> unsigned short notify_vcpu_id; /* VCPU for local delivery notification */
> uint32_t fifo_lastq; /* Data for identifying last queue. */
>
> -#ifdef CONFIG_XSM
> union {
> -#ifdef XSM_NEED_GENERIC_EVTCHN_SSID
> /*
> * If an XSM module needs more space for its event channel context,
> * this pointer stores the necessary data for the security server.
> */
> void *generic;
> -#endif
> #ifdef CONFIG_XSM_FLASK
> /*
> * Inlining the contents of the structure for FLASK avoids unneeded
> @@ -138,7 +135,6 @@ struct evtchn
> uint32_t flask_sid;
> #endif
> } ssid;
> -#endif
> } __attribute__((aligned(64)));
... I can see the inner #ifdef go away, but not the outer one. While
the (imo bogus) attribute means you don't alter the size of the
struct, I'm afraid that's not obvious at all without counting bits
and bytes, and hence this may also want saying explicitly in the
description.
Jan
|
