[Bug 82841] sendfile doesn't work with O_APPEND

[Bug 82841] sendfile doesn't work with O_APPEND

[bugzilla-daemon-590EEB7GvNiWaY/ihj7yzEB+6BGkLq7r]

https://bugzilla.kernel.org/show_bug.cgi?id=82841

Alan <alan-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |alan-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org
          Component|Other                       |man-pages
           Assignee|io_other-ztI5WcYan/vQLgFONoPN6+46SIhDJjyZ@public.gmane.org |documentation_man-pages@ker
                   |rg                          |nel-bugs.osdl.org
            Product|IO/Storage                  |Documentation

--- Comment #1 from Alan <alan-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org> ---
O_APPEND is not supported for sendfile().

It's documented in the splice() manual page but not for sendfile so I've moved
this to a documentation bug

-- 
You are receiving this mail because:
You are watching the assignee of the bug.
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Bug 82841] sendfile doesn't work with O_APPEND

[bugzilla-daemon-590EEB7GvNiWaY/ihj7yzEB+6BGkLq7r]

https://bugzilla.kernel.org/show_bug.cgi?id=82841

--- Comment #2 from Jason Newton <nevion-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> ---
(In reply to Alan from comment #1)
> O_APPEND is not supported for sendfile().
> 
> It's documented in the splice() manual page but not for sendfile so I've
> moved this to a documentation bug

OK so this quirk was not out in the open in documentation - before I opened
this bug report I read a few years old discussion:
http://fixunix.com/kernel/543017-splice-vs-o_append.html

Most of what they're talking about is splice, not sendfile.  They might use the
same implementation but sendfile should be O_APPEND safe as there is no
specifiable offset in the output filedescriptor.  Is there a reason to not
support it?

-- 
You are receiving this mail because:
You are watching the assignee of the bug.
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Bug 82841] sendfile doesn't work with O_APPEND

[bugzilla-daemon-590EEB7GvNiWaY/ihj7yzEB+6BGkLq7r]

https://bugzilla.kernel.org/show_bug.cgi?id=82841

--- Comment #3 from Alan <alan-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org> ---
sendfile is just a wrapper around splice()

Whether it could therefore support O_APPEND is a good question - but one that
would need asking and discussing on the lists not here, and someone to
contribute the patches to do so.

In the meantime we ought to get the man page fixed.

-- 
You are receiving this mail because:
You are watching the assignee of the bug.
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Bug 82841] sendfile doesn't work with O_APPEND

[bugzilla-daemon-590EEB7GvNiWaY/ihj7yzEB+6BGkLq7r]

https://bugzilla.kernel.org/show_bug.cgi?id=82841

Michael Kerrisk <mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
         Resolution|---                         |CODE_FIX

--- Comment #4 from Michael Kerrisk <mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> ---
I added the following text in ERRORS for sendfile(2).

       EINVAL out_fd  was has the O_APPEND flag set.  This is not cur‐
              rently supported by sendfile()

So, am closing this bug now.

-- 
You are receiving this mail because:
You are watching the assignee of the bug.--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html