<div dir="ltr">Hi,<div><br></div><div>please check inline,<br><div class="gmail_extra"><br><div class="gmail_quote">2017-02-09 8:31 GMT+00:00 Pavel Emelyanov <span dir="ltr">&lt;<a href="mailto:xemul@virtuozzo.com" target="_blank">xemul@virtuozzo.com</a>&gt;</span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Sorry for long delay (again) in response...<br>
<br>
I&#39;m ready to merge the patches into criu-dev, and later, when we have<br>
all this functionality tested by zdtm.py and code improved, I will merge<br>
the code into master. But in order to facilitate the 1st merging (into<br>
criu-dev branch) happen, can you do some cosmetic changes (mostly text<br>
fixes and splits)?<br>
<br>
Comments are inline.<br>
<br>
Later (after merge) I&#39;ll send more comments about what should be done on<br>
top of this set to make criu-dev -&gt; master merge.<br>
<br>
&gt; @@ -105,6 +105,7 @@ extern bool img_common_magic;<br>
&gt;  #define O_DUMP               (O_WRONLY | O_CREAT | O_TRUNC)<br>
&gt;  #define O_SHOW               (O_RDONLY | O_NOBUF)<br>
&gt;  #define O_RSTR               (O_RDONLY)<br>
&gt; +#define O_FORCE_LOCAL        (O_SYNC)<br>
<br>
Please, send the O_FORCE_LOCAL flag and image descs marked with it (3 hunks)<br>
as separate patch.<br>
<br></blockquote><div> </div><div>Okey! </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
&gt; @@ -501,7 +528,11 @@ static int init_pagemaps(struct page_read *pr)<br>
&gt;       off_t fsize;<br>
&gt;       int nr_pmes, nr_realloc;<br>
&gt;<br>
&gt; -     fsize = img_raw_size(pr-&gt;pmi);<br>
&gt; +     if (!opts.remote)<br>
&gt; +             fsize = img_raw_size(pr-&gt;pmi);<br>
&gt; +     else<br>
&gt; +             fsize = 1024; /*FIXME*/<br>
<br>
Please, send the O_FORCE_LOCAL flag and image descs marked with it (3 hunks)<br>
as separate patch.<br></blockquote><div><br></div><div>Same comment?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
&gt; @@ -224,22 +229,8 @@ int restore_sk_queue(int fd, unsigned int peer_id)<br>
&gt;                * boundaries messages should be saved.<br>
&gt;                */<br>
&gt;<br>
&gt; -             buf = xmalloc(entry-&gt;length);<br>
&gt; -             if (buf ==NULL)<br>
&gt; -                     goto err;<br>
&gt; -<br>
&gt; -             if (lseek(img_raw_fd(img), pkt-&gt;img_off, SEEK_SET) == -1) {<br>
&gt; -                     pr_perror(&quot;lseek() failed&quot;);<br>
&gt; -                     xfree(buf);<br>
&gt; -                     goto err;<br>
&gt; -             }<br>
&gt; -             if (read_img_buf(img, buf, entry-&gt;length) != 1) {<br>
&gt; -                     xfree(buf);<br>
&gt; -                     goto err;<br>
&gt; -             }<br>
&gt; -<br>
&gt; -             ret = write(fd, buf, entry-&gt;length);<br>
&gt; -             xfree(buf);<br>
&gt; +             ret = write(fd, pkt-&gt;data, entry-&gt;length);<br>
&gt; +             xfree(pkt-&gt;data);<br>
<br>
Please, send the restore_sk_queue() rework as separate patch.<br></blockquote><div><br></div><div>Okey! <br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888">
-- Pavel<br>
</font></span></blockquote></div><br></div></div></div>