From: Peter Korsgaard <jacmet@sunsite.dk>
Date: Sat, 12 Mar 2011 21:24:34 +0000 (+0100)
Subject: makedevs: don't call /bin/sync after creating nodes
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=707dc469aa4f51a85a62b2587ac57ef2a4db7873;p=buildroot.git

makedevs: don't call /bin/sync after creating nodes

Closes #3475

There's no real reason to sync after creating the device nodes,
and it slows down the file system image creating quite a lot on
busy systems, so get rid of it.

Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---

diff --git a/CHANGES b/CHANGES
index adc92987af..eafa2016b4 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,8 +2,12 @@
 
 	Updated/fixed packages: avahi, busybox, dropbear, ffmpeg,
 	file, gnuconfig, gst-ffmpeg, imagemagick, libdnet, libpng,
-	libxml2, libxslt, mpg123, pkg-config, qt, samba, squashfs,
-	xerces, xorg-server, xz
+	libxml2, libxslt, makedevs, mpg123, pkg-config, qt, samba,
+	squashfs, xerces, xorg-server, xz
+
+	Issues resolved (http://bugs.uclibc.org):
+
+	#3475: Calling sync on large filesystems when not always necessary
 
 2011.02, Released February 28th, 2011:
 
diff --git a/package/makedevs/makedevs.c b/package/makedevs/makedevs.c
index 79cfc609be..9fc71af54d 100644
--- a/package/makedevs/makedevs.c
+++ b/package/makedevs/makedevs.c
@@ -536,8 +536,5 @@ loop:
 	}
 	fclose(table);
 
-	if (system("/bin/sync"))
-		bb_error_msg("sync failed, continuing anyway");
-
 	return 0;
 }