<div dir="ltr">Hi Isaac,<div><br></div><div>Thanks. I have incorporated your comments.</div><div>Please find the updated files attached herewith.</div><div><br></div><div>Regards,</div><div>Vivek<br><div class="gmail_extra">

<br><br><div class="gmail_quote">On Thu, Feb 6, 2014 at 12:38 AM, Isaac Dunham <span dir="ltr"><<a href="mailto:ibid.ag@gmail.com" target="_blank">ibid.ag@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div>On Wed, Feb 05, 2014 at 06:49:03PM +0530, Vivek Bhagat wrote:<br>
> Hi List,<br>
><br>
> This is my first post on ToyBox community.<br>
><br>
</div>Welcome!<br>
<br>
I'm sorry I can't do inline comments this time, between the mime types and<br>
the editor I'm using; but I'll try to comment.<br>
<br>
And thanks for breaking them out into separate patches.<br>
<br>
Anyhow, first there are some smaller details that apply to all of them:<br>
<br>
1. It would be nice if in the future new toys were submitted by<br>
attaching the source code, rather than a patch (eg, freeramdisk.c,<br>
openvt.c, and deallocvt.c)<br>
<br>
2. Rob puts submissions into toys/pending first and sets them to "default n"<br>
<br>
3. For toybox, the standard indentation is 2 spaces for every level.<br>
<br>
4. I note that the NEWTOY line has more options than TOYFLAG_USR & TOYFLAG_BIN:<br>
All of these should have TOYFLAG_NEEDROOT, and freeramdisk really belongs in<br>
/sbin (TOYFLAG_SBIN)<br>
<div><br>
> Please find the patches attached herewith for adding 3 new commands -<br>
> 1. freeramdisk - If we unmount or detach the RAM disk based file system the<br>
> Linux Kernel<br>
>    will not free the allocated memory associated with the RAM device. This<br>
> can be useful if<br>
>    one wants to mount this device again:  All data will be preserved.<br>
>    If we need to free the memory back to the Kernel, one can use the<br>
> command: "toybox freeramdisk <RAM device>".<br>
<br>
</div>This one looked pretty good.<br>
<div><br>
><br>
> 2. openvt - Successfully opens a new virtual terminal as mentioned with -c<br>
> option<br>
>         otherwise search and open next available VT.<br>
>  with -s option it switches to new VT<br>
> with -s -w option, it switch back successfully to originating VT.<br>
<br>
</div>A couple of smaller details...<br>
This bit:<br>
>+      vt_fd = open(toybuf, O_RDWR);<br>
>+      if (vt_fd < 0) perror_exit("failed to open /dev/tty%d", TT.vt_num);<br>
could be replaced with<br>
vt_fd = xopen(toybuf, O_RDWR);<br>
<br>
Also, full POSIX conformance requires STDIN_FILENO to be 0,<br>
and toybox relies on this quite heavily.<br>
<div><br>
> 3. deallocvt - Deallocate specified virtual teminal.<br>
>     if no virtual terminal is specified, it deallocates all unused VT.<br>
><br>
<br>
</div>Uses find_console_fd() from openvt.c, and does not depend on openvt.<br>
The options are:<br>
-add "depends on openvt" to the kconfig entry<br>
-move find_console_fd() to lib/<br>
-put deallocvt in openvt.c<br>
<div><br>
> Please find the test log attached here for above 3 commands.<br>
> Your inputs are welcome.<br>
><br>
> Thanks.<br>
> Vivek<br>
<br>
<br>
</div>Thank you,<br>
Isaac Dunham<br>
</blockquote></div><br></div></div></div>