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

Re: [PATCH v3 4/4] pdx: Add CONFIG_PDX_COMPRESSION as a common Kconfig option


  • To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Mon, 25 Sep 2023 12:15:06 +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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=AzHRahnFty70n84vwIqaw+xyhYXP2aHICyT9GKPyIM8=; b=aNxYMqsPlrVrtL/5Cx/A8GtA8P6DNUAJIGCIY/BDkW+yufINiThAxQJ7IjR0bP8E65c641ni2bqRPGXLJFNUEq1j9ZeohwSkfBqPmFqO4YIaCliyeRn1CoGLNQvanWV8Aqwg9RFvP2RBaDH+U+gK369T8XsE+TeO41/1dE/+FsYdV+uo+LWTXXY6UJh9C2G8NDhOyYzO2FOpkkH4CapPrITnF3rPqXCEfbM1ykpc3d+Kfe6a3+jkkpqKDLYwoYuzXp9o0TO3W3eeH1lH8PuDXNv87Bq/NRr2skBr5Fqm2+tibNmHpsWaWTOI43UP+yWGWuuLzJzhpXBSE2x7FRso1g==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ekTvcAP0wyxJLtjzLZD5zI7t+63r/9eQZkFhi3vNyrk8Wgdfw3tjawPlpMefp7meLXUhAhvSzqKRlgPNKf999zgxJoswlBq8Dp13ACpk8GKlBCizYbfsz+j6l9Q3pkoZ6AWwTxD9pUmD0gOnuW9Jr03w/ygHADj6jRxHg6LFi9NqYvO5O1pQrg/KkzmjwJ4WyjVcY7GoLRzUyeptqNDiCHLen8gEN3QbnDozSb63cSD+c2iS8XNAV4r+l1kzvMKMc5G1i/N+2lOF0uR0N/XD7j+gRPxvsFrdlyfaPn4uLasE2CS67X0oebKmLhOZMrklZJv5NWBvR5kaHQ6n9YAr1A==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Bertrand Marquis <bertrand.marquis@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Julien Grall <jgrall@xxxxxxxxxx>, Henry Wang <Henry.Wang@xxxxxxx>, Alejandro Vallejo <alejandro.vallejo@xxxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Shawn Anastasio <sanastasio@xxxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 25 Sep 2023 10:15:28 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 25.09.2023 12:01, Andrew Cooper wrote:
> On 25/09/2023 10:46 am, Roger Pau Monné wrote:
>> On Mon, Sep 25, 2023 at 08:36:03AM +0200, Jan Beulich wrote:
>>> On 22.09.2023 22:03, Andrew Cooper wrote:
>>>> On 08/08/2023 2:02 pm, Alejandro Vallejo wrote:
>>>>> --- a/xen/common/Kconfig
>>>>> +++ b/xen/common/Kconfig
>>>>> @@ -23,6 +23,16 @@ config GRANT_TABLE
>>>>>  
>>>>>     If unsure, say Y.
>>>>>  
>>>>> +config PDX_COMPRESSION
>>>>> + bool "PDX (Page inDeX) compression support" if EXPERT
>>>> This still needs hiding on x86.  The minimal hack for 4.18 is "if EXPERT
>>>> && !X86".
>>> If you insist on complete hiding and I insist on allowing it to be used by
>>> people who want to play with exotic hardware, then this won't go anywhere.
>>> I think I've come far enough with accepting a compromise, and so I think
>>> it's your turn now to also take a step from your original position.
>> Just because I'm not familiar with this, is there any x86 hardware
>> that has such sparse memory map that would benefit from PDX?
> 
> There is one known system which never shipped.  Xen's implementation was
> from the anticipation of that system shipping.  Nothing else known.
> 
> None of the other major kernels have facilities such as this, which is
> very likely a contributory factor to the system not shipping.

Possibly, yet there's one important difference (mentioned before): VA space
that Xen can use (for directmap and frame_table[]) is far more constrained
than for baremetal systems (Linux, Windows). Hence the guys who got in touch
back at the time were not in need of Linux side changes at all (as far as I
was told back then).

Jan



 


Rackspace

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