On 04/04/2018 10:19 AM, Su Hang wrote:
> These series of patchs implement Intel Hexadecimal File loader and
> add QTest testcase to verify the correctness of Loader.
>
> v1: Basic version.
> v2: Replace `do{}while(cond);` block with `for(;;)` block to
> suppress code style complain.Better than rewriting code to work around a false positive is to fix the false positive in the first place. And I see you have a patch proposed for that, but it was still marked RFC; unfortunately, during freeze, patches that are marked RFC tend to be deferred in favor of patches marked 'for-2.12' (RFC means the patch may be incomplete, and there's no rush to get an incomplete patch in before the release, after all). I'll reply on that patch, now that it's got my attention; but for this particular series, I'm more comfortable leaving the review to the maintainer, modulo one comment: > > Su Hang (2): > Implement .hex file loader > Add QTest testcase for the Intel Hexadecimal Object File Loader. > > hw/arm/boot.c | 9 +- > hw/core/loader.c | 280 > +++++++++++++++++++++++++++++++++++++++++++++++++ > include/hw/loader.h | 17 +++ > tests/Makefile.include | 2 + > tests/hexloader-test.c | 56 ++++++++++ > tests/test.hex | 11 ++ > 6 files changed, 374 insertions(+), 1 deletion(-) > create mode 100644 tests/hexloader-test.c > create mode 100644 tests/test.hex This creates new files that are not covered by MAINTAINERS. Please be sure to patch MAINTAINERS as well in your v3. -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
