[erlang-patches] [PATCH] erts: change SENDFILE_CHUNK_SIZE from signed to unsigned

Henrik Nord <>
Tue Dec 6 10:02:49 CET 2011


Thank you, I have included this patch in 'pu'.

It is however unlikely that it will be included in the R15 release
Thank you for the contribution!


On 12/06/2011 07:50 AM, Jovi Zhang wrote:
> > From 918d76da1a8becd231bea92cbc26358dd5363ddf Mon Sep 17 00:00:00 2001
> From: Jovi Zhang<>
> Date: Tue, 29 Nov 2011 02:04:04 +0800
> Subject: [PATCH] erts: change SENDFILE_CHUNK_SIZE from signed to unsigned
>
> It's reasonable to use UL in SENDFILE_CHUNK_SIZE
> ---
> erts/emulator/drivers/unix/unix_efile.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/erts/emulator/drivers/unix/unix_efile.c
> b/erts/emulator/drivers/unix/unix_efile.c
> index 7291164..eb2c5f5 100644
> --- a/erts/emulator/drivers/unix/unix_efile.c
> +++ b/erts/emulator/drivers/unix/unix_efile.c
> @@ -1469,7 +1469,7 @@ efile_fadvise(Efile_error* errInfo, int fd, Sint64 offset,
> }
>
> #ifdef HAVE_SENDFILE
> -#define SENDFILE_CHUNK_SIZE ((1<<  30) -1)
> +#define SENDFILE_CHUNK_SIZE ((1UL<<  30) -1)
>
> /*
>   * sendfile: The implementation of the sendfile system call varies
> --
> 1.7.2.3
>
>
> _______________________________________________
> erlang-patches mailing list
> 
> http://erlang.org/mailman/listinfo/erlang-patches

-- 
/Henrik Nord Erlang/OTP

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://erlang.org/pipermail/erlang-patches/attachments/20111206/5e923769/attachment.html>


More information about the erlang-patches mailing list