[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] util: Improved qemu_hexmap() to include an asci
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH] util: Improved qemu_hexmap() to include an ascii dump of the buffer |
Date: |
Tue, 22 Mar 2016 20:14:19 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 |
On 03/22/2016 03:41 AM, Isaac Lozano wrote:
> qemu_hexdump() in util/hexdump.c has been changed to give also include a
> ascii dump of the buffer. Also, calls to hex_dump() in net/net.c have
> been replaced with calls to qemu_hexdump(). This takes care of two misc
> BiteSized Tasks.
>
> Signed-off-by: Isaac Lozano <address@hidden>
> ---
> net/net.c | 31 +------------------------------
> util/hexdump.c | 35 ++++++++++++++++++++++++-----------
> 2 files changed, 25 insertions(+), 41 deletions(-)
>
> diff --git a/net/net.c b/net/net.c
> index 1a78edf..8d51ffb 100644
> --- a/net/net.c
> +++ b/net/net.c
> @@ -79,34 +79,6 @@ int default_net = 1;
> /***********************************************************/
> /* network device redirectors */
>
> -#if defined(DEBUG_NET)
> -static void hex_dump(FILE *f, const uint8_t *buf, int size)
> -{
> - int len, i, j, c;
> -
> - for(i=0;i<size;i+=16) {
> - len = size - i;
> - if (len > 16)
> - len = 16;
> - fprintf(f, "%08x ", i);
> - for(j=0;j<16;j++) {
> - if (j < len)
> - fprintf(f, " %02x", buf[i+j]);
> - else
> - fprintf(f, " ");
> - }
> - fprintf(f, " ");
> - for(j=0;j<len;j++) {
> - c = buf[i+j];
> - if (c < ' ' || c > '~')
> - c = '.';
> - fprintf(f, "%c", c);
> - }
> - fprintf(f, "\n");
> - }
> -}
> -#endif
> -
> static int get_str_sep(char *buf, int buf_size, const char **pp, int sep)
> {
> const char *p, *p1;
> @@ -661,8 +633,7 @@ static ssize_t
> qemu_send_packet_async_with_flags(NetClientState *sender,
> int ret;
>
> #ifdef DEBUG_NET
> - printf("qemu_send_packet_async:\n");
> - hex_dump(stdout, buf, size);
> + qemu_hexdump((const char*)buf, stdout, "qemu_send_packet_async", size);
> #endif
>
> if (sender->link_down || !sender->peer) {
> diff --git a/util/hexdump.c b/util/hexdump.c
> index 1d9c129..886e53c 100644
> --- a/util/hexdump.c
> +++ b/util/hexdump.c
> @@ -18,21 +18,34 @@
>
> void qemu_hexdump(const char *buf, FILE *fp, const char *prefix, size_t size)
> {
> - unsigned int b;
> + unsigned int b, len, i, c;
>
> - for (b = 0; b < size; b++) {
> - if ((b % 16) == 0) {
> - fprintf(fp, "%s: %04x:", prefix, b);
> + for (b = 0; b < size; b += 16) {
> + len = size - b;
> + if (len > 16) {
> + len = 16;
> }
> - if ((b % 4) == 0) {
> - fprintf(fp, " ");
> + fprintf(fp, "%s: %04x:", prefix, b);
> + for (i = 0; i < 16; i++) {
> + if ((i % 4) == 0) {
> + fprintf(fp, " ");
> + }
> + if (i < len) {
> + fprintf(fp, " %02x", (unsigned char)buf[b + i]);
> + }
> + else
> + {
> + fprintf(fp, " ");
> + }
> }
> - fprintf(fp, " %02x", (unsigned char)buf[b]);
> - if ((b % 16) == 15) {
> - fprintf(fp, "\n");
> + fprintf(fp, " ");
> + for (i = 0; i < len; i++) {
> + c = buf[b+i];
> + if (c < ' ' || c > '~') {
> + c = '.';
> + }
> + fprintf(fp, "%c", c);
> }
> - }
> - if ((b % 16) != 0) {
> fprintf(fp, "\n");
> }
> }
>
Don't forget to CC a maintainer when you submit a patch to the open
list, otherwise you run the risk of having your patch get lost. See:
http://wiki.qemu.org/Contribute/SubmitAPatch#CC_the_relevant_maintainer
CCing net/net.c maintainer Jason Wang and the contributor who suggested
the BiteSizedTask, Thomas Huth (...I think. I'm not sure who else 'huth'
on the wiki might be, but he'll certainly correct me if I'm wrong...!)
Thanks,
--js