On 24 March 2018 at 18:13, Michael Clark <m...@sifive.com> wrote: > This is a series of bug fixes and code cleanups that we would > like to get in before the QEMU 2.12 release. We are respinning > v6 of this series to include two new bug fixes. These changes > are present in the downstream riscv.org riscv-all branch: > > - https://github.com/riscv/riscv-qemu/commits/riscv-all > > This series also addresses post-merge feedback such as updating > the cpu initialization model to conform with other architectures > as requested by Igor Mammedov.
Hi. It looks to me like a fair number of these patches are already reviewed, so we don't need to wait on the rest being reviewed to get those into master. My suggestion is that you send a pullrequest now for the reviewed patches, and send a patchset for review for the new ones or the ones that still need review. (If there are patches that are reviewed but depend on earlier ones that need to go in set 2 then they go in set 2 as well.) 26 patches is a lot to still be carrying around much beyond rc1, so I would like to see the size of this set reducing rather than increasing. As the release process moves forward the bar for "can this still go in" gradually goes up -- by about rc3 it is at about "is this a really critical bug or regression from the previous release". (Also something seems to have unhelpfully decided to eat or delay about half of your emails in this patchset :-( Patchew only sees 14 of the 26. Our mailing list server does seem to do that occasionally so that would be my first guess at the culprit, but it's possible it's something at your end.) thanks -- PMM
