[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Xen 4.18 release: Reminder about code freeze
- To: George Dunlap <george.dunlap@xxxxxxxxx>, "sstabellini@xxxxxxxxxx" <sstabellini@xxxxxxxxxx>
- From: Henry Wang <Henry.Wang@xxxxxxx>
- Date: Mon, 25 Sep 2023 11:08:10 +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=hFATpn4ONnHFcAU/2FHf72ZJR8ubD4ofyg044rqiJoU=; b=oFfcPVJMN/TQ/LH7Gk0SUGswF/LJw64Cw7VJ7s/hivMIsLDTs0bmk4sO9UaKgd3WGDpvoodVWvZxXsi8j2uuHt77Wvy4TS5JyFMaoeqskIfVAEmX4sCmxU6op88ofhEfvIwZa2ZcSR7q6MmYBkCuXvcRLjCd9RE/tXY3JPIuEEHiB/2GQyih7rxdbkR1j9fcFEChD+7VvVlLn/wN0lklWSXCzKMAbY9mRqzN6WgeUTGZn8KeHd961Yp/W6ZlwzQL6zSSH8Rp53shv2yMAdiNiD75A7PEi6MkkiwjFEe2BJk3Yk6B2w2Pp/qOWB+CniAnCYDRpBH6fWdjNwZRyqkMjg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=YcTS46JF5Eky44UcAv/YYMzLg6AncWCdx4+7RUstTImH9QskPeOWC7jWBUS0IVa4OYo2Ltdw779TFiSiQw7VnJLdGt3s+F3ch2hMOtSLxowdPFAMtW8KEbkyVUeo8T/FxmXXjBW83C/68ffsUN1HuVJaAdYd6ET545fSyu0udePdT274DmugGkwixew4l9rslWvtJKse67uFSHMW0EON4/MXGmrxnipg5r8Ns2R/h42xKgnQrQKWiCvlOVnZyOVwtoex5mgkeux5DDIB+lPzgvMbrq0/EYozydoi5mpYQYaHsLDDSJ+Cand+00odPP9ej/EduSK4mTt+O76J6LmOFA==
- Authentication-results-original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com;
- Cc: Julien Grall <julien@xxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, "committers@xxxxxxxxxxxxxx" <committers@xxxxxxxxxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, "jgross@xxxxxxxx" <jgross@xxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, "community.manager@xxxxxxxxxxxxxx" <community.manager@xxxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Mon, 25 Sep 2023 11:08:36 +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: AQHZ705Dwyrpd4I60kahQfZlaJ2JtLArFXGAgAACIQCAADDfgIAACQ8AgAAQ+YA=
- Thread-topic: Xen 4.18 release: Reminder about code freeze
Hi George,
> On Sep 25, 2023, at 18:07, George Dunlap <george.dunlap@xxxxxxxxx> wrote:
>
> On Mon, Sep 25, 2023 at 10:35 AM Julien Grall <julien@xxxxxxx> wrote:
>>
>> Hi Henry,
>>
>> On 25/09/2023 07:40, Henry Wang wrote:
>>>> On Sep 25, 2023, at 14:32, Jan Beulich <jbeulich@xxxxxxxx> wrote:
>>>> This, for example, would then likely mean
>>>> that all Misra work now needs queuing for after the tree re-opens ...
>>>
>>> …I also thought about this, to be honest I am tempted to loose the control
>>> or at least offer some flexibility on this specific part, as normally MISRA
>>> related changes are harmless and actually harden the code. I am wondering
>>> if there are any objections from others…
>>>
>>> Committers, would you mind sharing your opinion on this one? Thanks!
>>
>> I am split. On one hand, I agree they low risk and would be good to have
>> them. But on the other hand, they tend to be invasive and may interfere
>> with any bug we need to fix during the hardening period.
>
> *Theoretically* MISRA patches should have no behavioral side effects;
> but it's quite possible that they will. I'd be in favor of a more
> strict view, that they should all go on a separate branch (or simply
> be reviewed in-principle and re-submitted after we branch) now that
> the feature freeze is done.
Thanks for sharing your opinion. I definitely understand your concern. I think
in
Xen Summit we agreed that the release process should not affect the normal
code review, so MISRA patches can still be posted to the list and be reviewed.
When the staging reopens, these staged MISRA patches can be committed right
away.
> That's my recommendation, but ultimately I'd leave the decision to Henry.
Since this is about MISRA, I would like to wait one more day to see if there is
any input from Stefano, otherwise I think Julien’s suggestion is very good so
we can just follow that proposed timeline.
Kind regards,
Henry
>
> -George
|
