svn commit: r304182 - head/sys/compat/cloudabi

[Konstantin Belousov]

On Mon, Aug 15, 2016 at 07:42:21PM +0000, Ed Schouten wrote:
> Author: ed
> Date: Mon Aug 15 19:42:21 2016
> New Revision: 304182
> URL: https://svnweb.freebsd.org/changeset/base/304182
> 
> Log:
>   Let CloudABI use fdatasync() as well.
>   
>   Now that FreeBSD supports fdatasync() natively, we can tidy up
>   CloudABI's equivalent system call to use that instead.
> 
> Modified:
>   head/sys/compat/cloudabi/cloudabi_fd.c
> 
> Modified: head/sys/compat/cloudabi/cloudabi_fd.c
> ==============================================================================
> --- head/sys/compat/cloudabi/cloudabi_fd.c	Mon Aug 15 19:37:18 2016	(r304181)
> +++ head/sys/compat/cloudabi/cloudabi_fd.c	Mon Aug 15 19:42:21 2016	(r304182)
> @@ -172,12 +172,11 @@ int
>  cloudabi_sys_fd_datasync(struct thread *td,
>      struct cloudabi_sys_fd_datasync_args *uap)
>  {
> -	struct fsync_args fsync_args = {
> +	struct fdatasync_args fdatasync_args = {
>  		.fd = uap->fd
>  	};
>  
> -	/* Call into fsync(), as FreeBSD lacks fdatasync(). */
> -	return (sys_fsync(td, &fsync_args));
> +	return (sys_fdatasync(td, &fdatasync_args));
>  }
>  
Please make kern_fsync() non-static and use it instead of providing fake
fdatasync_args structure.