[LTP] [PATCH 2/3] syscalls/sendfile: Convert sendfile06 to the new API

Petr Vorel
2021-06-17T20:06:29Z

Hi Ziyao,

> 1. Convert sendfile06 to the new API with file descriptors instead
> of socket descriptors.
I wonder if this is ok, suppose yes, but better if other check.

> 2. Remove the support for UCLINUX.

LGTM.
Reviewed-by: Petr Vorel <pvorel@suse.cz>

...
> + TEST(sendfile(out_fd, in_fd, NULL, sb.st_size));
> + after_pos = SAFE_LSEEK(in_fd, 0, SEEK_CUR);
> +
> + if (TST_RET != sb.st_size)
nit: checkpatch.pl complains, it should be if (sb.st_size != TST_RET)
(can be fixed during merge)

Kind regards,
Petr