Cyrill Gorcunov wrote:
> Hi! While been trying to run criu on linux-next (due to recent Kirill's patch 
> related
> to /proc/pid/clear_refs with THP enabled) I noticed that it fails dumping 
> programs when
> moves data from memory pages into an image file. So I wrote pretty idiotic 
> test and
> run it on current fedora 20 kernel and then on linux-next.
> 

+Al. He reworked splice code in linux-next.

> fedora-20 output
> ----------------
> [cyrill@moon criu] ~/pipe 
> Opened pipe-test.SQBcJa
> vmspliced 8192 bytes
> spliced 4096 bytes
> 
> linux-next
> ----------
> [root@fc criu]# ~/pipe
> Opened pipe-test.9nZSW7
> vmspliced 8192 bytes
> spliced 8192 bytes
> 
> In test I fill pipe with 8K data then splice 4K of it into a file (the test
> is below). Is it intended, or I did some silly mistake?
> ---
> #define _GNU_SOURCE
> 
> #include <stdio.h>
> #include <stdlib.h>
> #include <unistd.h>
> #include <string.h>
> #include <fcntl.h>
> 
> #include <sys/uio.h>
> 
> #define ARRAY_SIZE(x) (sizeof(x)/sizeof(x[0]))
> 
> static char buf[8 << 20];
> 
> int main(int argc, char *argv[])
> {
>       char *tmpname, t[64] = "pipe-test.XXXXXX";
>       int _pipe[2], fd, ret = 0;
> 
>       struct iovec iov[] = {
>               [0] = {
>                       .iov_base       = &buf[0],
>                       .iov_len        = 4096,
>               },
> 
>               [1] = {
>                       .iov_base       = &buf[4096],
>                       .iov_len        = 4096,
>               },
>       };
> 
>       if (pipe(_pipe)) {
>               perror("Can't create pipe");
>               exit(1);
>       }
> 
>       memset(buf, 0x1, sizeof(buf));
>       tmpname = mktemp(t);
> 
>       fd = open(tmpname, O_RDWR | O_CREAT | O_TRUNC);
>       if (fd < 0) {
>               perror("Can't open temp file");
>               close(_pipe[0]);
>               close(_pipe[1]);
>               exit(1);
>       }
>       printf("Opened %s\n", tmpname);
> 
>       ret = vmsplice(_pipe[1], iov, ARRAY_SIZE(iov), SPLICE_F_GIFT);
>       printf("vmspliced %li bytes\n", (long)ret);
> 
>       ret = splice(_pipe[0], NULL, fd, NULL, iov[0].iov_len, SPLICE_F_MOVE);
>       printf("spliced %li bytes\n", (long)ret);
> 
> out:
>       close(_pipe[0]);
>       close(_pipe[1]);
>       close(fd);
>       unlink(tmpname);
>       return ret;
> }

-- 
 Kirill A. Shutemov
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to