From: Su Hang <suhang16@mails.ucas.ac.cn>
To: stefanha@gmail.com, jim@groklearning.com, joel@jms.id.au
Cc: qemu-devel@nongnu.org
Subject: [Qemu-devel] [PATCH v6 0/2] Implement Hex file loader and add test case
Date: Thu, 26 Apr 2018 21:51:36 +0800 [thread overview]
Message-ID: <1524750698-16322-1-git-send-email-suhang16@mails.ucas.ac.cn> (raw)
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.
v3: Add two new files information in MAINTAINERS.
v4: Correct the 'test.hex' path in hexloader-test.c.
v5: Split huge parse_hex_blob() function into four small function;
Add check for memory bounds;
Check validation for Record type;
Replace function ctoh() with glib function g_ascii_xdigit_value();
Remove check for '.hex' suffix;
Remove unnecessary type cast;
Remove duplicate zero-initialization;
Correct typos;
v6: Call Intel HEX loader after load_uimage_as();
Remove use of KERNEL_LOAD_ADDR;
Change (hwaddr) type argument to (hwaddr *) type;
Use new struct HexParser to contain all arguments needed by
handle_record_type();
Enable discontiguous data records (again);
Remove unnecessary memory clean: bin_buf and HexLine line;
Correct typo;
Remove unnecessary check for hex file size;
Add entry point handle for START_SEG_ADDR_RECORD and
START_LINEAR_ADDR_RECORD record type;
Use hwaddr * type to store entry point;
Su Hang (2):
Implement .hex file loader
Add QTest testcase for the Intel Hexadecimal Object File Loader.
MAINTAINERS | 6 +
hw/arm/boot.c | 17 ++-
hw/core/loader.c | 241 +++++++++++++++++++++++++++++++++++
include/hw/loader.h | 16 +++
tests/Makefile.include | 2 +
tests/hex-loader-check-data/test.hex | 12 ++
tests/hexloader-test.c | 56 ++++++++
7 files changed, 345 insertions(+), 5 deletions(-)
create mode 100644 tests/hex-loader-check-data/test.hex
create mode 100644 tests/hexloader-test.c
--
2.7.4
next reply other threads:[~2018-04-26 13:51 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-26 13:51 Su Hang [this message]
2018-04-26 13:51 ` [Qemu-devel] [PATCH v6 1/2] Implement .hex file loader Su Hang
2018-04-30 14:23 ` Stefan Hajnoczi
2018-04-30 15:40 ` Su Hang
2018-05-07 9:27 ` Su Hang
2018-05-09 13:24 ` Stefan Hajnoczi
2018-04-26 13:51 ` [Qemu-devel] [PATCH v6 2/2] Add QTest testcase for the Intel Hexadecimal Object File Loader Su Hang
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1524750698-16322-1-git-send-email-suhang16@mails.ucas.ac.cn \
--to=suhang16@mails.ucas.ac.cn \
--cc=jim@groklearning.com \
--cc=joel@jms.id.au \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).