Re: [PATCH,HURD] implement syncfs

2012-11-19 Thread Roland McGrath
That's fine.

Re: [PATCH,HURD] implement syncfs

2012-11-19 Thread Pino Toscano
Hi, Alle lunedì 19 novembre 2012, Pino Toscano ha scritto: > simple implementation of the Linux-ish syncfs on Hurd. Updated patch according to Roland's and Christoph's comments (thanks!) -- Pino Toscano Hurd: implement syncfs 2012-11-19 Pino Toscano * sysdeps/mach/hurd/syncfs.c: New file.

Re: [PATCH,HURD] implement syncfs

2012-11-19 Thread Christoph Hellwig
> +/* Make all changes done to all files on the file system associated > + with FD actually appear on disk. */ > +int > +syncfs (int fd) > +{ > + /* This is not actually synchronous; we don't wait. */ > + error_t err = HURD_DPORT_USE (fd, __file_syncfs (port, 0, 0)); Note that syncfs in Linu

Re: [PATCH,HURD] implement syncfs

2012-11-19 Thread Roland McGrath
> > +/* Make all changes done to all files on the file system associated > > + with FD actually appear on disk. */ > > +int > > +syncfs (int fd) > > +{ > > + /* This is not actually synchronous; we don't wait. */ > > + error_t err = HURD_DPORT_USE (fd, __file_syncfs (port, 0, 0)); > > Note t

[PATCH,HURD] implement syncfs

2012-11-19 Thread Pino Toscano
Hi, simple implementation of the Linux-ish syncfs on Hurd. Thanks, -- Pino Toscano Hurd: implement syncfs 2012-11-19 Pino Toscano * sysdeps/mach/hurd/syncfs.c: New file. --- /dev/null +++ b/sysdeps/mach/hurd/syncfs.c @@ -0,0 +1,33 @@ +/* Copyright (C) 2012 Free Software Foundation, Inc. +

Re: [PATCH,HURD] implement syncfs

2012-11-19 Thread Roland McGrath
The top line of every new file is a descriptive comment. You don't need #include in that file.