diff options
| author | Petr Uzel | 2010-05-21 14:18:31 +0200 |
|---|---|---|
| committer | Karel Zak | 2010-05-28 14:29:26 +0200 |
| commit | ff14168beedfc05598b77fe490bbbe347c4bf3cb (patch) | |
| tree | 8d6853062e879d20878ed0ed31206aaefbc52aae /.gitignore | |
| parent | mount: fix typo in comment (diff) | |
| download | kernel-qcow2-util-linux-ff14168beedfc05598b77fe490bbbe347c4bf3cb.tar.gz kernel-qcow2-util-linux-ff14168beedfc05598b77fe490bbbe347c4bf3cb.tar.xz kernel-qcow2-util-linux-ff14168beedfc05598b77fe490bbbe347c4bf3cb.zip | |
umount: do not sync() as the kernel does it by itself
On Fri, May 21, 2010 at 08:00:09AM -0400, Christoph Hellwig wrote:
> On Fri, May 21, 2010 at 01:55:17PM +0200, Petr Uzel wrote:
> > SUSE-based distributions have the following patch for some time. More
> > info here: https://bugzilla.novell.com/show_bug.cgi?id=447036
> >
> > From the Novell bugzilla:
> > <quote>
> > > I cannot see any justification for that sync call at all so I'd
> > > probably just remove it. Your
> > > patch is possibly safer so maybe it should be used as is.
> > </quote>
> >
> > So, does anybody know why/if the sync() is actually needed?
>
> It's not needed. The kernel performs a sync by itself.
Thanks, Christoph.
Signed-off-by: Petr Uzel <petr.uzel@suse.cz>
Diffstat (limited to '.gitignore')
0 files changed, 0 insertions, 0 deletions