<div dir="ltr"><div class="gmail_extra">Hi Dmitry,</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thank you for your review!<br><br><div class="gmail_quote">On Sat, Aug 9, 2014 at 5:05 AM, Dmitry V. Levin <span dir="ltr"><<a href="mailto:ldv@altlinux.org" target="_blank">ldv@altlinux.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Tue, Jul 01, 2014 at 03:13:21PM +0530, <a href="mailto:zubin.mithra@gmail.com">zubin.mithra@gmail.com</a> wrote:<br>

> * defs.h (show_fd_path): Change type to unsigned int.<br>
> * strace.c (show_fd_path): Update usage to count y flag.<br>
> (usage): Update to add -yy option.<br>
> * util.c (print_tracee_cwd): New function.<br>
> (printpathn): Update to use print_tracee_cwd and print<br>
>  absolute path.<br>
> * strace.1: Add description of -yy option.<br>
<br>
</div>I had an opportunity to think about this feature a bit longer,<br>
and now I see two issues I missed before.<br>
<br>
The first issue is the choice of option name.  While -y option enables<br>
*additional* decoding of descriptors, and -yy was planned for extended<br>
decoding of descriptors like socket decoding, this feature *changes*<br>
decoding of all paths.  I'd rather not mix these things.<br></blockquote><div><br></div><div>Sounds good, just to make sure I understand clearly -- </div><div><br></div><div>1. -y should do absolute decoding of file descriptors always.</div>
<div>2. -yy should do extended decoding like socket decoding etc.</div><div><br></div><div>Is that what you mean?</div><div> <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
The second issue is *at syscalls.  This implementation assumes that all<br>
non-absolute paths are relative to cwd, but that is not necessarily the<br>
case for *at syscalls.<br></blockquote><div><br></div><div>I see -- could we try and display canonical paths for fds passed into *at syscalls?</div><div> </div><div>Thanks,</div><div>-- zm</div></div></div></div>