Skip to content

Commit

Permalink
kgdb: have ebin2mem call probe_kernel_write once
Browse files Browse the repository at this point in the history
Rather than call probe_kernel_write() one byte at a time, process the
whole buffer locally and pass the entire result in one go.  This way,
architectures that need to do special handling based on the length can
do so, or we only end up calling memcpy() once.

[sonic.zhang@analog.com: Reported original problem and preliminary patch]

Signed-off-by: Jason Wessel <jason.wessel@windriver.com>
Signed-off-by: Sonic Zhang <sonic.zhang@analog.com>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
  • Loading branch information
jwessel committed Apr 2, 2010
1 parent 42be79e commit a0279bd
Showing 1 changed file with 9 additions and 14 deletions.
23 changes: 9 additions & 14 deletions kernel/kgdb.c
Expand Up @@ -391,27 +391,22 @@ int kgdb_mem2hex(char *mem, char *buf, int count)

/*
* Copy the binary array pointed to by buf into mem. Fix $, #, and
* 0x7d escaped with 0x7d. Return a pointer to the character after
* the last byte written.
* 0x7d escaped with 0x7d. Return -EFAULT on failure or 0 on success.
* The input buf is overwitten with the result to write to mem.
*/
static int kgdb_ebin2mem(char *buf, char *mem, int count)
{
int err = 0;
char c;
int size = 0;
char *c = buf;

while (count-- > 0) {
c = *buf++;
if (c == 0x7d)
c = *buf++ ^ 0x20;

err = probe_kernel_write(mem, &c, 1);
if (err)
break;

mem++;
c[size] = *buf++;
if (c[size] == 0x7d)
c[size] = *buf++ ^ 0x20;
size++;
}

return err;
return probe_kernel_write(mem, c, size);
}

/*
Expand Down

0 comments on commit a0279bd

Please sign in to comment.