[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- To: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Michal Orzel <michal.orzel@xxxxxxx>
- From: Henry Wang <Henry.Wang@xxxxxxx>
- Date: Wed, 7 Sep 2022 13:49:29 +0000
- Accept-language: zh-CN, en-US
- Arc-authentication-results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.xenproject.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=armh.onmicrosoft.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com])
- 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=2; 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=dCLOjIdtjuWE/LHmRffdfEN0QuE/zrOgtgmC/4qgZ0U=; b=ANyB5agPgqy/Vd3xMHxnuntymWVtpwsOz3oM1xEo6mbyFiI8qWyH4wPxokfsjvEK2rO0N1ligyBvIkbjnFkHWdfiOReUWORK+MrdjZASPCtdG1VRHPvIhYJlnS4QHgoQttD0vjanbYz0j05iF1F1KcXiBLuYLyR46SOw4yq5jL1mIlpSgi7G/0R+5OeQ9jWSlaT68H82Zn288k6MmUXk/nqB/Us7N+P6yYObppNubdhzhJMRqNOmaxcW8feEqkV5mFxffFHSVroMaFsITFHXFEfNs+f2ZIR2AMED6ANlE7yLSuCQj2NFu2r02FeN/Dh5azyULA1y7dX7wKANUQZ2HQ==
- 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=dCLOjIdtjuWE/LHmRffdfEN0QuE/zrOgtgmC/4qgZ0U=; b=cFK7EmPUfizibVA6PjGKK56dtsBURUYjOQ5XFt82u7ALvRAYctRoi96+Y2vP8p8J65PrCykG9AG5hl7DcyuiWaleVUYG7KDHflPrGZ7a9mSqt/Fz4pr3N6CjJt3+FrJmtJdfuKisNXaNcbHMtFuRNNK9hFxABh6rGRg+R1KDiHMhvcrg090cwH35VSRHAe/nYjY5Vm8MAbbiLsvoA2YAIXrSna8q1JOnbkWfrgAmuRsjv8E2qF+C/+qW7nyEe8xM8EPXRd7Q7W6R6f4pVF15tSNOe2OrXyCwPu1301Hsk0ZB93jwn+t7IeRLOMqnM5jCxa3+TSsEaYjLpp+sJ6hExA==
- Arc-seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=iqgIhA0/TNEao6IBGPWdKBiAEDBz1hspTwB/wD7Dy4xjTfqZcoWUE+GMBcbDZVeWc+7N1shIe61cuZf5VpEMpl4KWiqx88OeD1XpHFW24lyncSu9I/FErpCmCch8bims223zsMh6i3/lzuha66W59PE7ncI5DBblyjI8JheWtwk0e3rv20BX9UXjac6f7lIxvxlbsioGHVCbuu+rVwlFf3uNrQSpIdV0pPfEDPbrKe6k4exet6d0Ti2XFpw4AcSnUcXW7ta+5kN11xUx8mzJI680uICJanbfpjXCWM0L5tWv2VcVBegsY3sbGNjfGdFq3+eIycXDl/tjQogY3CzeZg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=CG8qGhoi4WQGF9cEIaqulXoW+9hgo1lxzCxHB+vPyw7LAAkAjmJi3FF94B8RFuUUg7dXOwgfTyVGYLoISKw0MRTiINohujBGp2c4VZ0EZQz7iOAh2UthehVGbOFJ0waplMgSfBGjj9Av4cgSehs88nMzoFZXo8cU/1HXCSDi9m3wXsHjqS0aVRRHGElLqZXxcwZM4oUD6OCJ4pCkMzp0C8mA8V3X1bCTQoOVE5kCe46I8TZT9YUozyGUozATTsyquqwKOfbXhBOMwHXZM51LwPb16/icDZBOdeoDyXo0nQEEaGIUSKH5a61Jcvaq2uJc6vm7oK+HbgeDzVnoKKeHhA==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: Julien Grall <julien@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Chen <Wei.Chen@xxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, Penny Zheng <Penny.Zheng@xxxxxxx>
- Delivery-date: Wed, 07 Sep 2022 13:49:43 +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: AQHYwpUJ7IYztv5II0KGaSuPgw9aFq3T1xaAgAAKD4CAAAVdgIAAAr4AgAAA5ICAAAbIgIAABXCAgAAA4QCAAAB+gIAAARoAgAACMwCAAABysA==
- Thread-topic: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
Hi Bertrand and Michal,
I don't want to spam the email so I just reply here...
> -----Original Message-----
> From: Bertrand Marquis <Bertrand.Marquis@xxxxxxx>
> >>>> But in any case we should only add one pair here for sure, as you say
> the
> >>>> only implication is to add a couple of 0 in the worst case.
> >>> I agree. The only drawback is the need to modify the already introduced
> properties
> >>> to be coherent.
> >>
> >> Agree, someone will need to do a pass on the whole doc which might be
> easier with all things in.
> >>
> > Well, not only docs. If we decide to use a single pair of #address-cells and
> #size-cells, then
> > we need to modify the code that expects different properties e.g.
> xen,static-mem-{address/size}-cells.
>
> Right I forgot that some parts are already in.
> So we will need an extra patch to handle those.
I think I've addressed all comments from Julien regarding my series,
so I think I've got some bandwidth to do the clean-up patch tomorrow
after the agreement, unless someone would like to do it himself?
Kind regards,
Henry
>
> Bertrand
>
> >
> >> Cheers
> >> Bertrand
> >>
> >>>
> >>>>
> >>>> Cheers
> >>>> Bertrand
> >>>>
> >>>>>
> >>>>>>
> >>>>>> Cheers,
> >>>>>>
> >>>>>> --
> >>>>>> Julien Grall
>
- References:
- [PATCH v3 0/4] Introduce static heap
- [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
- Re: [PATCH v3 2/4] docs, xen/arm: Introduce static heap memory
|