lkml.org 
[lkml]   [2015]   [Jul]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 1/1] suspend: delete sys_sync()
    On Mon, Jul 06, 2015 at 03:52:25PM +0200, Rafael J. Wysocki wrote:
    > On Monday, July 06, 2015 10:06:14 AM Dave Chinner wrote:
    > > On Sat, Jul 04, 2015 at 03:03:46AM +0200, Rafael J. Wysocki wrote:
    > > > > No, your observation was that "sync is slow". Your *solution* is "we
    > > > > need to remove sync".
    > > >
    > > > Not only slow, but pointless too. The argument goes: "It is slow and
    > > > pointless and so it may be dropped."
    > > >
    > > > Now, I can agree that it wasn't clearly demonstrated that the unconditional
    > > > sys_sync() in the suspend code path was pointless, but it also has never
    > > > been clearly shown why it is not pointless on systems that suspend and resume
    > > > reliably.
    > >
    > > I just gave you an example of why sync is needed: Suspend, pull out
    > > USB drive when putting laptop in bag.
    >
    > Exactly the same thing can happen while not suspended. What does make suspend
    > special with respect to that?

    Stop being obtuse. If you remember that you need to pull the USB
    stick before you suspend, then you damn well remember to "safely
    eject" the USB stick and that syncs, unmounts and flushes the drive
    caches before telling you it is safe to pull the stick.

    > > > Moreover, question is if we really need to carry out the sync on *every*
    > > > suspend even if it is not pointless overall. That shouldn't really be
    > > > necessary if we suspend and resume often enough or if we resume only for
    > > > a while and then suspend again. Maybe it should be rate limited somehow
    > > > at least?
    > >
    > > If you suspend and resume frequently, then the cost of the sync
    > > shoul dbe negliable because the amount of data dirtied between
    > > resume/suspend shoul dbe negliable. hence my questions about where
    > > sync is spending too much time, and whether we've actually fixed
    > > those problems or not. If sync speed on clean filesystems is a
    > > problem then we need to fix sync, not work around it.
    >
    > Well, say you never suspend, but you also only shut down the system when you
    > need to replace the kernel on it. How often would you invoke global sync on
    > that system?

    Never, because:

    - the kernel does background writeback of dirty data so you
    don't need to run sync while the system is running; and
    - unmount during shutdown runs sync_filesystem() internally
    (just like sys_sync does) to ensure the filesystem is
    clean and no data is lost.

    Seriously, stop being making ignorant arguments to justify removing
    sys_sync(). *If* there's a problem sys_sync() we need to *fix it*,
    not ignore it.

    Cheers,

    Dave.
    --
    Dave Chinner
    david@fromorbit.com


    \
     
     \ /
      Last update: 2015-07-07 03:41    [W:4.962 / U:0.212 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site