curl / Mailing Lists / curl-library / Single Mail
Buy commercial curl support from WolfSSL. We help you work out your issues, debug your libcurl applications, use the API, port to new platforms, add new features and more. With a team lead by the curl founder himself.

Re: Feature-Request: sendfile(2)

From: Emil Engler via curl-library <curl-library_at_cool.haxx.se>
Date: Fri, 26 Feb 2021 14:20:51 +0100

Okay, first thanks for your interest and time to reply to my
suggestion =)

I think the arguments listed in this e-mail are probably enough
to dismiss the idea.

Have a nice weekend!
Cheers,
Emil

On Fri, Feb 26, 2021 at 10:10:01AM +0100, Michael Felt via curl-library wrote:
> a) Long before sendfile() there was/is send_file() in the UNIX world.
> b) just read in the linux man pages that sendfile() is limited to 2GB file
> transfers
> c) imho - too much complexity for read/write activity when one medium
> (network) is likely much slower (or faster) than another medium - or even
> over two network interfaces.
>
> As an example - Python has a sendfile function for most, but not all
> platforms. And yes, it can be faster - but some platforms have rather
> complex interfaces - and so end up not getting implemented. re: cURL - more
> complexity means more complex, time-consuming maintenance. If I had a vote -
> I would say no as a "nice to have" but would consider it with a definite
> advantage over not having it. Would have to why the cost of maintenance and
> possible (security) bugs - as people look for ways to abuse it.
>
> On 26/02/2021 08:47, Gisle Vanem via curl-library wrote:
> > Emil Engler wrote:
> >
> > > However implementing this syscall also raises some problems, including
> > > limited support. AFAIK only Linux and FreeBSD support this syscall
> > > (maybe even macOS as it inherited a lot from FreeBSD).
> >
> > FYI, Windows has a similar 'TransmitFile()' function:
> > https://docs.microsoft.com/en-us/windows/win32/api/mswsock/nf-mswsock-transmitfile
> >
>




> -------------------------------------------------------------------
> Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library
> Etiquette: https://curl.se/mail/etiquette.html

-------------------------------------------------------------------
Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library
Etiquette: https://curl.se/mail/etiquette.html
Received on 2021-02-26