[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: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH] util: Improved qemu_hexmap() to include an ascii dump of the buffer |
Date: |
Wed, 23 Mar 2016 09:04:50 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 |
On 23.03.2016 01:14, John Snow wrote:
>
> 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, that was indeed me - and I would have indeed missed this patch
if you'd not put me on CC:. And I think you could also put Peter
Crosthwaite and Gerd Hoffmann on CC since they are mentioned as author
in the header comment of the util/hexdump.c file.
Thomas