On Mon, 5 Dec 2022 at 09:20, gaos...@loongson.cn <gaos...@loongson.cn> wrote: > > > 在 2022/12/5 15:24, Philippe Mathieu-Daudé 写道: > > On 2/12/22 11:25, Song Gao wrote: > >> The following changes since commit > >> c4ffd91aba1c3d878e99a3e7ba8aad4826728ece: > >> > >> Update VERSION for v7.2.0-rc3 (2022-11-29 18:15:26 -0500) > >> > >> are available in the Git repository at: > >> > >> https://gitlab.com/gaosong/qemu.git tags/pull-loongarch-20221202 > >> > >> for you to fetch changes up to 14dccc8ea6ece7ee63273144fb55e4770a05e0fd: > >> > >> hw/loongarch/virt: Add cfi01 pflash device (2022-12-02 18:03:05 > >> +0800) > >> > >> ---------------------------------------------------------------- > >> pull for 7.2-rc4 > >> > >> We need this patch. > > > > FTR this is not a security/regression fix, but a mere feature. > > > > Certainly not justified for a rc4 IMO. > > > We hope LoongArch 7.2 version support pflash, > otherwise the subsequent BIOS support pflash may qemu 7.2 does not support.
Regardless of how much it might be nice to have a new feature supported, new features cannot go in after softfreeze, only fixes for bugs. At rc4, changes should really be release critical bugs and regression fixes only. Further, any pull request that should be going in for rc4 should have a clear statement of what the changes do and why they are release critical. "We need this patch" is much too vague. QEMU's release schedule is regular, so there will always be another release in 4 months time. There is generally no need to be in a huge hurry to get a feature in. I would favour reverting this change. thanks -- PMM
