Re: [Qemu-devel] [Patch] fix /proc/self/maps output

[Christophe Lyon]

Hi,

Sorry I wasn't aware of such restrictions (I'm used to gcc/binutils
conventions), and my thunderbird does display my patch as if it were
inline :-)
So.... trying a cut-and-paste approach in gmail, for this one character patch:


/proc/self/maps: Add a space at end of line when there is no filename
to print, to conform to linux kernel format.

Signed-off-by: Christophe Lyon <address@hidden>

diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index a148d9f..3b0ca86 100644
--- a/linux-user/syscall.c
+++ b/linux-user/syscall.c
@@ -5016,7 +5016,7 @@ static int open_self_maps(void *cpu_env, int fd)
                     " %c%c%c%c %08" PRIx64 " %02x:%02x %d%s%s\n",
                     h2g(min), h2g(max), flag_r, flag_w,
                     flag_x, flag_p, offset, dev_maj, dev_min, inode,
-                    path[0] ? "          " : "", path);
+                    path[0] ? "          " : " ", path);
         }
     }


On 29 March 2013 15:44, Peter Maydell <address@hidden> wrote:
> On 29 March 2013 14:33, Christophe Lyon <address@hidden> wrote:
>> Hi,
>> May I ping for this small patch?
>
> Hi Christophe; I'm afraid the patch can't be applied because it's
> missing a signed-off by. (Also if it was sent inline in the usual
> format rather than as an attachment it would be easier to review.)
>
> thanks
> -- PMM