[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Call SetVirtualAddressMap() by default


  • To: Demi Marie Obenour <demi@xxxxxxxxxxxxxxxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Thu, 23 Feb 2023 11:16:28 +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=lVxcu6jhcC7IgDaKNcCefb6aRdLAfRJ9BQZg18sTCBc=; b=cklpdSWQqpRc8FdFSSe7KcC5WpYAI6ZPAwHzO5dgSDqRNV+K9S3olYVkXZUqtPKkdL4KmxGN8P40cae1thdf6lwcFwGMxx7JOwaiuI99KwKVE+bf4FQQHh4qDiN7jB3hLGbx+cExYJOrwBae6dQ/sqpwYTPci29aZ5+HEeBF7DV8xnEGHa086cgFyhPTWlXNilVOCcFwAMc9y+1kAX+MmtY2soBa7MHw3a+Ucyh4pn0Ft9B1RjaPgPK9ApN16HDIqs06jIyFLA+Sfm6OHw60RMv+dquLidai67xDROdBBHlIKVpyFTzTCdJFdjju1DB7avoqsSDcCQW8y8WV+lddZQ==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Ti7X2zlswAhXWjwTo8/aq7ZoqIXE7p6XrwdI2tDK1WJQzIXQHgj3cdr8BPBAA/Se/t5+4msLSJhEl5iAddhoOQIKBnzDDSgqokoPuPHn2D7OpCXe2hqNN3l9GUdmCzru1y/DZ2hdvL70ZNlC102pZgdBOfMV90BURlSElcVGzjFGVCbtoVflObOJBj1Sca47/1fPor9tl68cBdZwrzCuB68iU+NagwYkwBesEcoL8DnEQ7v4dHEZ7/E+a1qCkf1nc9fvWUB6rJPdz3QJKlsHQxEORdeJ9jJMPbldk29LRETGj5Gpf5IeXfNd1L6hL07R9qSS4rxc+icMbKhYZRyPQg==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • Delivery-date: Thu, 23 Feb 2023 10:16:38 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 22.02.2023 20:14, Demi Marie Obenour wrote:
> To quote Andrew Cooper:
> 
>> I know we've had this argument before, but not calling
>> SetVirtualAddressMap() isn't a viable option.  It's a prerequisite to
>> function on literally millions of devices
> 
> Qubes OS has been shipping EFI_SET_VIRTUAL_ADDRESS_MAP for years, and I
> believe OpenXT and EVE ship it as well. Mark EFI_SET_VIRTUAL_ADDRESS_MAP
> as SUPPORTED and enable it by default.

This is insufficient justification. The main reason why we don't call
it is because it can be called only once. Any entity taking over later
(e.g. via kexec) can't do anything anymore about the virtual address
associations once set. Hence what's needed to justify a change like
this is an explanation of why this restriction is not really an issue
to anyone in any case.

Jan



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.