[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-4.18 v5] xen/pdx: Make CONFIG_PDX_COMPRESSION a common Kconfig option
- To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
- From: Henry Wang <Henry.Wang@xxxxxxx>
- Date: Thu, 12 Oct 2023 01:05:31 +0000
- Accept-language: zh-CN, en-US
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.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=U9fA+DDpFbZWBcM4hQVXI+SVSbAVkOPh3O/2RdXmwrg=; b=LXMy98DZEp1F7oHCzwuFm3HNn4NtuFw7El++XXMQ41HaXuob5t69nrOfNq/MNnjvGheFuFccf9SEhdeGqKxjM7ZUaRT88yGcdeKzAgBeq/gPwkeWuVvbBTcXuYKEBUd0L5ar6vsnocpB8lipStg+0VtGbbvJuxTqrO50XYPjeJakPOFHR1ieJjqh22NKyiUZEZQpDkSp62cO9jd+Rfv4RW5HflIP7hHkIzfU9QVcnLVmeEtdHZ6mTd071zC7r7VaaO3dzaxQ/X+cZTmUkNNm+LVFJbitFDX5VVpsGrSjwnA9N33ATFGyTrcdTzSVw7Sbn7/RzW6ZJR1KwJPsKYruxg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gEF/YCd0GBQYPBhRjlz0GjwGOH3/17np1bU2YMJD5488JFMBGPRM25GO8O0OGJGyDAiGEZFpPVeydp9PnHw7hevHfbQWeJEQMgM5ZJhdR9HPmas6Cn2LKegVpwaCZvfs2Z3r8d5dcaRfs0lNQn/aa4Xr2Fu/0k00ixl2abIwtciobRkIPjG4wYSiJCSAg6lHRfSUlkcqLm5rGrgpQNS2s+FX5evRf9qm0k6Q0NbESjIfaYe7Ed3sbBnXeoT3wtUTfeUnbk/q9oq/H4VlQXpj4YY7pJAyiiLp4+sElrgM2ZtGfywUfId0EPb9HQzaKA90WV9h5ZEv/OP1AK5TEH6zNg==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: Oleksii <oleksii.kurochko@xxxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Alejandro Vallejo <alejandro.vallejo@xxxxxxxxx>, Julien Grall <jgrall@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Shawn Anastasio <sanastasio@xxxxxxxxxxxxxxxxxxxxx>, George Dunlap <George.Dunlap@xxxxxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Julien Grall <julien@xxxxxxx>
- Delivery-date: Thu, 12 Oct 2023 01:06:07 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Nodisclaimer: true
- Original-authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Thread-index: AQHZ+GOg88iN2g5+OkaTPmz6gnCP0bBCrkmAgAAPDICAAfEmAIAAML2AgACAVwA=
- Thread-topic: [PATCH for-4.18 v5] xen/pdx: Make CONFIG_PDX_COMPRESSION a common Kconfig option
Hi,
> On Oct 12, 2023, at 01:25, Andrew Cooper <andrew.cooper3@xxxxxxxxxx> wrote:
>
> On 11/10/2023 10:31 pm, Oleksii wrote:
>> On Tue, 2023-10-10 at 16:52 +0800, Andrew Cooper wrote:
>>> On 10/10/2023 3:58 pm, Oleksii wrote:
>>>>> Oleksii: I've not touched RISCV yet, because I don't know how the
>>>>> platforms
>>>>> typically look. I'm happy to default it active in RISCV too if
>>>>> that's the
>>>>> right thing to do.
>>>> We are still waiting for a platform with hypervisor extension
>>>> support
>>>> but I am using PDX in my Xen's repo.
>>> Yes, but are you using that because Xen wouldn't build without it, or
>>> because it's necessary for RISC-V platforms?
>> You are right I am using it because before this patch Xen can't be
>> built without PDX.
>> This is not necessary for RISC-V. At least platform I know they don't
>> use sparse RAM banks.
>
> Ok thanks. I'll leave the code as is, but tweak the commit message to
> include RISC-V alongside x86.
>
> And as already noted, we can always revisit the decision in the future
> if things change.
Looks like the discussion has been settled so:
Release-acked-by: Henry Wang <Henry.Wang@xxxxxxx>
Kind regards,
Henry
>
> ~Andrew
|
