Re: [Qemu-devel] [PATCH v2 RFC 0/2] Implement Hex file loader and add test case
Thanks for your reply! Owning to cultural difference, I used to regard “RFC” as a way of show modest... I will apply your suggestion in v3. Best, Su Hang "Eric Blake"wrote: > 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 >
Re: [Qemu-devel] [PATCH v2 RFC 0/2] Implement Hex file loader and add test case
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
[Qemu-devel] [PATCH v2 RFC 0/2] Implement Hex file loader and add test case
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. 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 -- 2.7.4