[Buildroot] [Bug 3475] New: Calling sync on large filesystems when not always necessary

[Buildroot] [Bug 3475] New: Calling sync on large filesystems when not always necessary

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=3475

           Summary: Calling sync on large filesystems when not always
                    necessary
           Product: buildroot
           Version: unspecified
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P5
         Component: Other
        AssignedTo: unassigned at buildroot.uclibc.org
        ReportedBy: orionrobots at gmail.com
                CC: buildroot at uclibc.org
   Estimated Hours: 0.0


When using build root to cross compile for other devices, especially when doing
many builds (multiple targets), on a CI system with 10s of gigs of ram and many
intermediate targets, then sync is not really desirable here.

The makedevs step makes a call to /bin/sync.

I am suggesting a "--nosync" be introduced so buildroot will not sync the
drives then? While of course preserving existing behaviour for the majority
that don't find any problems.

-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 3475] Calling sync on large filesystems when not always necessary

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=3475

--- Comment #1 from Danny Staple <orionrobots@gmail.com>  ---
For info - this sync is called as a shell out to "/bin/sync" in makedevs.c.


if (system("/bin/sync"))
        bb_error_msg("sync failed, continuing anyway");

-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 3475] Calling sync on large filesystems when not always necessary

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=3475

Peter Korsgaard <jacmet@uclibc.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED

--- Comment #2 from Peter Korsgaard <jacmet@uclibc.org>  ---
I think it makes more sense to simply get rid of the sync. There's no real
reason why we would need to sync in the first place.

-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

[Buildroot] [Bug 3475] Calling sync on large filesystems when not always necessary

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=3475

Peter Korsgaard <jacmet@uclibc.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Peter Korsgaard <jacmet@uclibc.org>  ---
Removed in git, thanks

-- 
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.