From 6a770f893c6fc6a05da911f8369758aee3e169c2 Mon Sep 17 00:00:00 2001 From: Andrew Price Date: Mon, 8 Sep 2014 11:48:03 +0100 Subject: [PATCH] * Mon Sep 08 2014 Andrew Price - 3.1.7-1 - New upstream release - Drop all patches - gfs2-utils tests: Build unit tests with consistent cpp flags --- .gitignore | 1 + ...ow_Don_t_try_to_open_an_empty_string.patch | 183 ------------------ ...libgfs2_Add_lgfs2_open_mnt_functions.patch | 141 -------------- ...me_mounted_callers_to_lgfs2_open_mnt.patch | 168 ---------------- ...3-libgfs2_Remove_is_pathname_mounted.patch | 94 --------- gfs2-utils.spec | 23 ++- ...unit_tests_with_consistent_cpp_flags.patch | 56 ++++++ sources | 2 +- 8 files changed, 69 insertions(+), 599 deletions(-) delete mode 100644 bz1079286-0-gfs2_grow_Don_t_try_to_open_an_empty_string.patch delete mode 100644 bz1079286-1-libgfs2_Add_lgfs2_open_mnt_functions.patch delete mode 100644 bz1079286-2-Switch_is_pathname_mounted_callers_to_lgfs2_open_mnt.patch delete mode 100644 bz1079286-3-libgfs2_Remove_is_pathname_mounted.patch create mode 100644 gfs2_utils_tests_Build_unit_tests_with_consistent_cpp_flags.patch diff --git a/.gitignore b/.gitignore index 02bd2bd..8e5aacd 100644 --- a/.gitignore +++ b/.gitignore @@ -5,3 +5,4 @@ /gfs2-utils-3.1.4.tar.gz /gfs2-utils-3.1.5.tar.gz /gfs2-utils-3.1.6.tar.gz +/gfs2-utils-3.1.7.tar.gz diff --git a/bz1079286-0-gfs2_grow_Don_t_try_to_open_an_empty_string.patch b/bz1079286-0-gfs2_grow_Don_t_try_to_open_an_empty_string.patch deleted file mode 100644 index c70e38e..0000000 --- a/bz1079286-0-gfs2_grow_Don_t_try_to_open_an_empty_string.patch +++ /dev/null @@ -1,183 +0,0 @@ -commit 23bcbc3512a6609216b6884a3bf088684d1e7c72 -Author: Andrew Price -Date: Thu Aug 8 17:50:47 2013 +0100 - - gfs2_grow: Don't try to open an empty string - - sdp->device_name wasn't getting set in gfs2_grow so is_gfs2() (called - via check_for_gfs2()) fails to open "" and so we get an ENOENT. - - This fixes the open("") by setting sdp->device_name before passing it to - is_pathname_mounted(), which has been updated to make it more clear - which args will be modified by it. - - is_gfs2() and check_for_gfs2() have also been removed from libgfs2 as - these were the only places they were being called and they aren't - needed: superblock checking is done further along via other functions - calling check_sb(). - - Signed-off-by: Andrew Price - -diff --git a/gfs2/fsck/initialize.c b/gfs2/fsck/initialize.c -index b45ae08..6612fe7 100644 ---- a/gfs2/fsck/initialize.c -+++ b/gfs2/fsck/initialize.c -@@ -1491,7 +1491,7 @@ int initialize(struct gfs2_sbd *sdp, int force_check, int preen, - strncpy(sdp->device_name, opts.device, - sizeof(sdp->device_name)); - sdp->path_name = sdp->device_name; /* This gets overwritten */ -- is_mounted = is_pathname_mounted(sdp, &ro); -+ is_mounted = is_pathname_mounted(sdp->path_name, sdp->device_name, &ro); - /* If the device is busy, but not because it's mounted, fail. - This protects against cases where the file system is LVM - and perhaps mounted on a different node. */ -diff --git a/gfs2/libgfs2/libgfs2.h b/gfs2/libgfs2/libgfs2.h -index fe7129a..e994d3b 100644 ---- a/gfs2/libgfs2/libgfs2.h -+++ b/gfs2/libgfs2/libgfs2.h -@@ -716,11 +716,9 @@ extern void decrease_verbosity(void); - extern int compute_heightsize(struct gfs2_sbd *sdp, uint64_t *heightsize, - uint32_t *maxheight, uint32_t bsize1, int diptrs, int inptrs); - extern int compute_constants(struct gfs2_sbd *sdp); --extern int is_pathname_mounted(struct gfs2_sbd *sdp, int *ro_mount); --extern int is_gfs2(struct gfs2_sbd *sdp); -+extern int is_pathname_mounted(char *path_name, char *device_name, int *ro_mount); - extern int find_gfs2_meta(struct gfs2_sbd *sdp); - extern int dir_exists(const char *dir); --extern int check_for_gfs2(struct gfs2_sbd *sdp); - extern int mount_gfs2_meta(struct gfs2_sbd *sdp); - extern void cleanup_metafs(struct gfs2_sbd *sdp); - extern int set_sysfs(const char *fsname, const char *filename, const char *val); -diff --git a/gfs2/libgfs2/misc.c b/gfs2/libgfs2/misc.c -index c05a770..6f2dbdb 100644 ---- a/gfs2/libgfs2/misc.c -+++ b/gfs2/libgfs2/misc.c -@@ -99,7 +99,7 @@ int compute_constants(struct gfs2_sbd *sdp) - return 0; - } - --int is_pathname_mounted(struct gfs2_sbd *sdp, int *ro_mount) -+int is_pathname_mounted(char *path_name, char *device_name, int *ro_mount) - { - FILE *fp; - struct mntent *mnt; -@@ -112,7 +112,7 @@ int is_pathname_mounted(struct gfs2_sbd *sdp, int *ro_mount) - perror("open: /proc/mounts"); - return 0; - } -- if (stat(sdp->path_name, &st_buf) == 0) { -+ if (stat(path_name, &st_buf) == 0) { - if (S_ISBLK(st_buf.st_mode)) { - #ifndef __GNU__ /* The GNU hurd is broken with respect to stat devices */ - file_rdev = st_buf.st_rdev; -@@ -124,12 +124,12 @@ int is_pathname_mounted(struct gfs2_sbd *sdp, int *ro_mount) - } - while ((mnt = getmntent (fp)) != NULL) { - /* Check if they specified the device instead of mnt point */ -- if (strcmp(sdp->device_name, mnt->mnt_fsname) == 0) { -- strcpy(sdp->path_name, mnt->mnt_dir); /* fix it */ -+ if (strcmp(device_name, mnt->mnt_fsname) == 0) { -+ strcpy(path_name, mnt->mnt_dir); /* fix it */ - break; - } -- if (strcmp(sdp->path_name, mnt->mnt_dir) == 0) { -- strcpy(sdp->device_name, mnt->mnt_fsname); /* fix it */ -+ if (strcmp(path_name, mnt->mnt_dir) == 0) { -+ strcpy(device_name, mnt->mnt_fsname); /* fix it */ - break; - } - if (stat(mnt->mnt_fsname, &st_buf) == 0) { -@@ -160,36 +160,6 @@ int is_pathname_mounted(struct gfs2_sbd *sdp, int *ro_mount) - return 1; /* mounted */ - } - --int is_gfs2(struct gfs2_sbd *sdp) --{ -- int fd, rc; -- struct gfs2_sb sb; -- -- fd = open(sdp->device_name, O_RDWR); -- if (fd < 0) -- return 0; -- -- rc = 0; -- if (lseek(fd, GFS2_SB_ADDR * GFS2_BASIC_BLOCK, SEEK_SET) >= 0 && -- read(fd, &sb, sizeof(sb)) == sizeof(sb) && -- be32_to_cpu(sb.sb_header.mh_magic) == GFS2_MAGIC && -- be32_to_cpu(sb.sb_header.mh_type) == GFS2_METATYPE_SB) -- rc = 1; -- close(fd); -- return rc; --} -- --int check_for_gfs2(struct gfs2_sbd *sdp) --{ -- int ro; -- -- if (!is_pathname_mounted(sdp, &ro)) -- return -1; -- if (!is_gfs2(sdp)) -- return -1; -- return 0; --} -- - static int lock_for_admin(struct gfs2_sbd *sdp) - { - int error; -diff --git a/gfs2/mkfs/main_grow.c b/gfs2/mkfs/main_grow.c -index 73c7f84..d324af9 100644 ---- a/gfs2/mkfs/main_grow.c -+++ b/gfs2/mkfs/main_grow.c -@@ -323,6 +323,7 @@ main_grow(int argc, char *argv[]) - int rgcount, rindex_fd; - char rindex_name[PATH_MAX]; - int error = EXIT_SUCCESS; -+ int ro_mnt = 0; - - memset(sdp, 0, sizeof(struct gfs2_sbd)); - sdp->bsize = GFS2_DEFAULT_BSIZE; -@@ -336,17 +337,20 @@ main_grow(int argc, char *argv[]) - int sane; - struct rgrp_tree *last_rgrp; - -+ strncpy(sdp->device_name, argv[optind], PATH_MAX - 1); - sdp->path_name = argv[optind++]; -- sdp->path_fd = open(sdp->path_name, O_RDONLY | O_CLOEXEC); -- if (sdp->path_fd < 0){ -+ -+ if ((!is_pathname_mounted(sdp->path_name, sdp->device_name, &ro_mnt))) { - perror(sdp->path_name); - exit(EXIT_FAILURE); - } - -- if (check_for_gfs2(sdp)) { -+ sdp->path_fd = open(sdp->path_name, O_RDONLY | O_CLOEXEC); -+ if (sdp->path_fd < 0){ - perror(sdp->path_name); - exit(EXIT_FAILURE); - } -+ - sdp->device_fd = open(sdp->device_name, - (test ? O_RDONLY : O_RDWR) | O_CLOEXEC); - if (sdp->device_fd < 0){ -diff --git a/gfs2/mkfs/main_jadd.c b/gfs2/mkfs/main_jadd.c -index 58fb046..2646829 100644 ---- a/gfs2/mkfs/main_jadd.c -+++ b/gfs2/mkfs/main_jadd.c -@@ -491,6 +491,7 @@ void main_jadd(int argc, char *argv[]) - { - struct gfs2_sbd sbd, *sdp = &sbd; - unsigned int total; -+ int ro_mnt = 0; - - memset(sdp, 0, sizeof(struct gfs2_sbd)); - sdp->jsize = GFS2_DEFAULT_JSIZE; -@@ -506,7 +507,7 @@ void main_jadd(int argc, char *argv[]) - exit(EXIT_FAILURE); - } - -- if (check_for_gfs2(sdp)) { -+ if (!is_pathname_mounted(sdp->path_name, sdp->device_name, &ro_mnt)) { - perror(sdp->path_name); - exit(EXIT_FAILURE); - } diff --git a/bz1079286-1-libgfs2_Add_lgfs2_open_mnt_functions.patch b/bz1079286-1-libgfs2_Add_lgfs2_open_mnt_functions.patch deleted file mode 100644 index c55f8f9..0000000 --- a/bz1079286-1-libgfs2_Add_lgfs2_open_mnt_functions.patch +++ /dev/null @@ -1,141 +0,0 @@ -commit 25c6e51bb4d083eb341bbc8541bb2ad278988f10 -Author: Andrew Price -Date: Sat Nov 16 02:10:52 2013 -0600 - - libgfs2: Add lgfs2_open_mnt* functions - - lgfs2_open_mnt is a replacement for is_pathname_mounted which tries to reduce - races by opening paths speculatively and passing back the open fds once they're - known to be correct. lgfs2_open_mnt_{dev,dir} build on this to provide a - convenient way to open just the device or mount directory relating to a path - which could be either. - - Signed-off-by: Andrew Price - -diff --git a/gfs2/libgfs2/libgfs2.h b/gfs2/libgfs2/libgfs2.h -index f864a08..3e5d09c 100644 ---- a/gfs2/libgfs2/libgfs2.h -+++ b/gfs2/libgfs2/libgfs2.h -@@ -12,6 +12,7 @@ - #include - #include - #include -+#include - - #include - #include "osi_list.h" -@@ -715,6 +716,9 @@ extern int compute_heightsize(struct gfs2_sbd *sdp, uint64_t *heightsize, - uint32_t *maxheight, uint32_t bsize1, int diptrs, int inptrs); - extern int compute_constants(struct gfs2_sbd *sdp); - extern int is_pathname_mounted(char *path_name, char *device_name, int *ro_mount); -+extern int lgfs2_open_mnt(const char *path, int dirflags, int *dirfd, int devflags, int *devfd, struct mntent **mnt); -+extern int lgfs2_open_mnt_dev(const char *path, int flags, struct mntent **mnt); -+extern int lgfs2_open_mnt_dir(const char *path, int flags, struct mntent **mnt); - extern int find_gfs2_meta(struct gfs2_sbd *sdp); - extern int dir_exists(const char *dir); - extern int mount_gfs2_meta(struct gfs2_sbd *sdp); -diff --git a/gfs2/libgfs2/misc.c b/gfs2/libgfs2/misc.c -index 7f500e6..195b983 100644 ---- a/gfs2/libgfs2/misc.c -+++ b/gfs2/libgfs2/misc.c -@@ -163,6 +163,100 @@ int is_pathname_mounted(char *path_name, char *device_name, int *ro_mount) - return 1; /* mounted */ - } - -+/* Returns 0 if fd1 and fd2 refer to the same device/file, 1 otherwise, or -1 on error */ -+static int fdcmp(int fd1, int fd2) -+{ -+ struct stat st1, st2; -+ if ((fstat(fd1, &st1) != 0) || (fstat(fd2, &st2) != 0)) -+ return -1; -+ if (S_ISBLK(st1.st_mode) && S_ISBLK(st2.st_mode)) { -+ if (st1.st_rdev == st2.st_rdev) { -+ return 0; -+ } -+ } else if ((st1.st_dev == st2.st_dev) && (st1.st_ino == st2.st_ino)) { -+ return 0; -+ } -+ return 1; -+} -+ -+int lgfs2_open_mnt(const char *path, int dirflags, int *dirfd, int devflags, int *devfd, struct mntent **mnt) -+{ -+ FILE *fp = setmntent("/proc/mounts", "r"); -+ if (fp == NULL) { -+ perror("open: /proc/mounts"); -+ return 1; -+ } -+ /* Assume path is mount point until we know better. */ -+ *dirfd = open(path, dirflags); -+ if (*dirfd < 0) -+ return 1; -+ -+ while ((*mnt = getmntent(fp)) != NULL) { -+ int fd; -+ if (strcmp((*mnt)->mnt_type, "gfs2") != 0) -+ continue; -+ *devfd = open((*mnt)->mnt_fsname, devflags); -+ /* Defer checking *devfd until later: whether it's ok to ignore -+ * the error depends on whether we find the mount point. */ -+ -+ if (strcmp(path, (*mnt)->mnt_dir) == 0) -+ break; -+ if (strcmp(path, (*mnt)->mnt_fsname) == 0 || fdcmp(*dirfd, *devfd) == 0) { -+ /* We have a match but our above assumption was -+ incorrect and *dirfd is actually the device. */ -+ close(*dirfd); -+ *dirfd = open((*mnt)->mnt_dir, dirflags); -+ break; -+ } -+ -+ fd = open((*mnt)->mnt_dir, dirflags); -+ if (fd >= 0) { -+ int diff = fdcmp(*dirfd, fd); -+ close(fd); -+ if (diff == 0) -+ break; -+ } -+ if (*devfd >= 0) -+ close(*devfd); -+ } -+ endmntent(fp); -+ if (*mnt == NULL) { -+ close(*dirfd); -+ return 0; /* Success. Answer is no. Both fds closed. */ -+ } -+ if (*dirfd < 0) { -+ close(*devfd); -+ return 1; -+ } -+ if (*devfd < 0) { -+ close(*dirfd); -+ return 1; -+ } -+ return 0; /* Success. Answer is yes. Both fds open. */ -+} -+ -+int lgfs2_open_mnt_dev(const char *path, int flags, struct mntent **mnt) -+{ -+ int dirfd = -1; -+ int devfd = -1; -+ if (lgfs2_open_mnt(path, O_RDONLY, &dirfd, flags, &devfd, mnt) != 0) -+ return -1; -+ if (*mnt != NULL) -+ close(dirfd); -+ return devfd; -+} -+ -+int lgfs2_open_mnt_dir(const char *path, int flags, struct mntent **mnt) -+{ -+ int dirfd = -1; -+ int devfd = -1; -+ if (lgfs2_open_mnt(path, flags, &dirfd, O_RDONLY, &devfd, mnt) != 0) -+ return -1; -+ if (*mnt != NULL) -+ close(devfd); -+ return dirfd; -+} -+ - static int lock_for_admin(struct gfs2_sbd *sdp) - { - int error; diff --git a/bz1079286-2-Switch_is_pathname_mounted_callers_to_lgfs2_open_mnt.patch b/bz1079286-2-Switch_is_pathname_mounted_callers_to_lgfs2_open_mnt.patch deleted file mode 100644 index ce05d00..0000000 --- a/bz1079286-2-Switch_is_pathname_mounted_callers_to_lgfs2_open_mnt.patch +++ /dev/null @@ -1,168 +0,0 @@ -commit ad11e277ebd8d01a1b0a29ee9f697ba90c8577a4 -Author: Andrew Price -Date: Sat Nov 16 02:19:07 2013 -0600 - - Switch is_pathname_mounted callers to lgfs2_open_mnt* - - Use the new lgfs2_open_mnt* functions in fsck.gfs2, gfs2_grow, and gfs2_jadd. - - Signed-off-by: Andrew Price - -diff --git a/gfs2/fsck/initialize.c b/gfs2/fsck/initialize.c -index 6612fe7..5758607 100644 ---- a/gfs2/fsck/initialize.c -+++ b/gfs2/fsck/initialize.c -@@ -1475,8 +1475,7 @@ int initialize(struct gfs2_sbd *sdp, int force_check, int preen, - - sdp->device_fd = open(opts.device, open_flag); - if (sdp->device_fd < 0) { -- int is_mounted, ro; -- -+ struct mntent *mnt; - if (open_flag == O_RDONLY || errno != EBUSY) { - log_crit( _("Unable to open device: %s\n"), - opts.device); -@@ -1485,30 +1484,23 @@ int initialize(struct gfs2_sbd *sdp, int force_check, int preen, - /* We can't open it EXCL. It may be already open rw (in which - case we want to deny them access) or it may be mounted as - the root file system at boot time (in which case we need to -- allow it.) We use is_pathname_mounted here even though -- we're specifying a device name, not a path name. The -- function checks for device as well. */ -- strncpy(sdp->device_name, opts.device, -- sizeof(sdp->device_name)); -- sdp->path_name = sdp->device_name; /* This gets overwritten */ -- is_mounted = is_pathname_mounted(sdp->path_name, sdp->device_name, &ro); -- /* If the device is busy, but not because it's mounted, fail. -+ allow it.) -+ If the device is busy, but not because it's mounted, fail. - This protects against cases where the file system is LVM -- and perhaps mounted on a different node. */ -- if (!is_mounted) -+ and perhaps mounted on a different node. -+ Try opening without O_EXCL. */ -+ sdp->device_fd = lgfs2_open_mnt_dev(opts.device, O_RDWR, &mnt); -+ if (sdp->device_fd < 0) - goto mount_fail; - /* If the device is mounted, but not mounted RO, fail. This - protects them against cases where the file system is - mounted RW, but still allows us to check our own root - file system. */ -- if (!ro) -- goto mount_fail; -+ if (!hasmntopt(mnt, MNTOPT_RO)) -+ goto close_fail; - /* The device is mounted RO, so it's likely our own root - file system. We can only do so much to protect the users -- from themselves. Try opening without O_EXCL. */ -- if ((sdp->device_fd = open(opts.device, O_RDWR)) < 0) -- goto mount_fail; -- -+ from themselves. */ - was_mounted_ro = 1; - } - -@@ -1591,6 +1583,8 @@ int initialize(struct gfs2_sbd *sdp, int force_check, int preen, - - return FSCK_OK; - -+close_fail: -+ close(sdp->device_fd); - mount_fail: - log_crit( _("Device %s is busy.\n"), opts.device); - return FSCK_USAGE; -diff --git a/gfs2/mkfs/main_grow.c b/gfs2/mkfs/main_grow.c -index 5db91d9..541b0f2 100644 ---- a/gfs2/mkfs/main_grow.c -+++ b/gfs2/mkfs/main_grow.c -@@ -323,7 +323,7 @@ main_grow(int argc, char *argv[]) - int rgcount, rindex_fd; - char rindex_name[PATH_MAX]; - int error = EXIT_SUCCESS; -- int ro_mnt = 0; -+ int devflags = (test ? O_RDONLY : O_RDWR) | O_CLOEXEC; - - memset(sdp, 0, sizeof(struct gfs2_sbd)); - sdp->bsize = GFS2_DEFAULT_BSIZE; -@@ -333,30 +333,23 @@ main_grow(int argc, char *argv[]) - sdp->md.journals = 1; - decode_arguments(argc, argv, sdp); - -- while ((argc - optind) > 0) { -+ for(; (argc - optind) > 0; optind++) { - int sane; -+ struct mntent *mnt; - struct rgrp_tree *last_rgrp; - -- strncpy(sdp->device_name, argv[optind], PATH_MAX - 1); -- sdp->path_name = argv[optind++]; -- -- if ((!is_pathname_mounted(sdp->path_name, sdp->device_name, &ro_mnt))) { -- perror(sdp->path_name); -- exit(EXIT_FAILURE); -- } -- -- sdp->path_fd = open(sdp->path_name, O_RDONLY | O_CLOEXEC); -- if (sdp->path_fd < 0){ -- perror(sdp->path_name); -+ error = lgfs2_open_mnt(argv[optind], O_RDONLY|O_CLOEXEC, &sdp->path_fd, -+ devflags, &sdp->device_fd, &mnt); -+ if (error != 0) { -+ fprintf(stderr, _("Error looking up mount '%s': %s\n"), argv[optind], strerror(errno)); - exit(EXIT_FAILURE); - } -- -- sdp->device_fd = open(sdp->device_name, -- (test ? O_RDONLY : O_RDWR) | O_CLOEXEC); -- if (sdp->device_fd < 0){ -- perror(sdp->device_name); -- exit(EXIT_FAILURE); -+ if (mnt == NULL) { -+ fprintf(stderr, _("%s: not a mounted gfs2 file system\n"), argv[optind]); -+ continue; - } -+ sdp->path_name = mnt->mnt_dir; -+ strncpy(sdp->device_name, mnt->mnt_fsname, PATH_MAX - 1); - - if (lgfs2_get_dev_info(sdp->device_fd, &sdp->dinfo) < 0) { - perror(sdp->device_name); -diff --git a/gfs2/mkfs/main_jadd.c b/gfs2/mkfs/main_jadd.c -index b6cd8e4..815dd52 100644 ---- a/gfs2/mkfs/main_jadd.c -+++ b/gfs2/mkfs/main_jadd.c -@@ -490,8 +490,8 @@ add_j(struct gfs2_sbd *sdp) - void main_jadd(int argc, char *argv[]) - { - struct gfs2_sbd sbd, *sdp = &sbd; -+ struct mntent *mnt; - unsigned int total; -- int ro_mnt = 0; - - memset(sdp, 0, sizeof(struct gfs2_sbd)); - sdp->jsize = GFS2_DEFAULT_JSIZE; -@@ -500,17 +500,18 @@ void main_jadd(int argc, char *argv[]) - - decode_arguments(argc, argv, sdp); - verify_arguments(sdp); -- -- sdp->path_fd = open(sdp->path_name, O_RDONLY | O_CLOEXEC); -- if (sdp->path_fd < 0){ -- perror(sdp->path_name); -+ -+ sbd.path_fd = lgfs2_open_mnt_dir(sbd.path_name, O_RDONLY|O_CLOEXEC, &mnt); -+ if (sbd.path_fd < 0) { -+ fprintf(stderr, _("Error looking up mount '%s': %s\n"), sbd.path_name, strerror(errno)); - exit(EXIT_FAILURE); - } -- -- if (!is_pathname_mounted(sdp->path_name, sdp->device_name, &ro_mnt)) { -- perror(sdp->path_name); -+ if (mnt == NULL) { -+ fprintf(stderr, _("%s: not a mounted gfs2 file system\n"), sbd.path_name); - exit(EXIT_FAILURE); - } -+ sbd.path_name = mnt->mnt_dir; -+ strncpy(sbd.device_name, mnt->mnt_fsname, PATH_MAX - 1); - - gather_info(sdp); - diff --git a/bz1079286-3-libgfs2_Remove_is_pathname_mounted.patch b/bz1079286-3-libgfs2_Remove_is_pathname_mounted.patch deleted file mode 100644 index 28cbb65..0000000 --- a/bz1079286-3-libgfs2_Remove_is_pathname_mounted.patch +++ /dev/null @@ -1,94 +0,0 @@ -commit 63db4e964739673d36c430cc5c78b02d93419f8a -Author: Andrew Price -Date: Sat Nov 16 02:21:36 2013 -0600 - - libgfs2: Remove is_pathname_mounted - - All callers are now using lgfs2_open_mnt* instead. - - Signed-off-by: Andrew Price - -diff --git a/gfs2/libgfs2/libgfs2.h b/gfs2/libgfs2/libgfs2.h -index 3e5d09c..e785017 100644 ---- a/gfs2/libgfs2/libgfs2.h -+++ b/gfs2/libgfs2/libgfs2.h -@@ -715,7 +715,6 @@ extern int metafs_interrupted; - extern int compute_heightsize(struct gfs2_sbd *sdp, uint64_t *heightsize, - uint32_t *maxheight, uint32_t bsize1, int diptrs, int inptrs); - extern int compute_constants(struct gfs2_sbd *sdp); --extern int is_pathname_mounted(char *path_name, char *device_name, int *ro_mount); - extern int lgfs2_open_mnt(const char *path, int dirflags, int *dirfd, int devflags, int *devfd, struct mntent **mnt); - extern int lgfs2_open_mnt_dev(const char *path, int flags, struct mntent **mnt); - extern int lgfs2_open_mnt_dir(const char *path, int flags, struct mntent **mnt); -diff --git a/gfs2/libgfs2/misc.c b/gfs2/libgfs2/misc.c -index 195b983..c4ed722 100644 ---- a/gfs2/libgfs2/misc.c -+++ b/gfs2/libgfs2/misc.c -@@ -102,67 +102,6 @@ int compute_constants(struct gfs2_sbd *sdp) - return 0; - } - --int is_pathname_mounted(char *path_name, char *device_name, int *ro_mount) --{ -- FILE *fp; -- struct mntent *mnt; -- dev_t file_dev=0, file_rdev=0; -- ino_t file_ino=0; -- struct stat st_buf; -- -- *ro_mount = 0; -- if ((fp = setmntent("/proc/mounts", "r")) == NULL) { -- perror("open: /proc/mounts"); -- return 0; -- } -- if (stat(path_name, &st_buf) == 0) { -- if (S_ISBLK(st_buf.st_mode)) { --#ifndef __GNU__ /* The GNU hurd is broken with respect to stat devices */ -- file_rdev = st_buf.st_rdev; --#endif /* __GNU__ */ -- } else { -- file_dev = st_buf.st_dev; -- file_ino = st_buf.st_ino; -- } -- } -- while ((mnt = getmntent (fp)) != NULL) { -- /* Check if they specified the device instead of mnt point */ -- if (strcmp(device_name, mnt->mnt_fsname) == 0) { -- strcpy(path_name, mnt->mnt_dir); /* fix it */ -- break; -- } -- if (strcmp(path_name, mnt->mnt_dir) == 0) { -- strcpy(device_name, mnt->mnt_fsname); /* fix it */ -- break; -- } -- if (stat(mnt->mnt_fsname, &st_buf) == 0) { -- if (S_ISBLK(st_buf.st_mode)) { --#ifndef __GNU__ -- if (file_rdev && (file_rdev == st_buf.st_rdev)) -- break; --#endif /* __GNU__ */ -- } else { -- if (file_dev && ((file_dev == st_buf.st_dev) && -- (file_ino == st_buf.st_ino))) -- break; -- } -- } -- } -- endmntent (fp); -- if (mnt == NULL) -- return 0; -- if (stat(mnt->mnt_dir, &st_buf) < 0) { -- if (errno == ENOENT) -- return 0; -- } -- /* Can't trust fstype because / has "rootfs". */ -- if (file_rdev && (st_buf.st_dev != file_rdev)) -- return 0; -- if (hasmntopt(mnt, MNTOPT_RO)) -- *ro_mount = 1; -- return 1; /* mounted */ --} -- - /* Returns 0 if fd1 and fd2 refer to the same device/file, 1 otherwise, or -1 on error */ - static int fdcmp(int fd1, int fd2) - { diff --git a/gfs2-utils.spec b/gfs2-utils.spec index 9810021..92b012c 100644 --- a/gfs2-utils.spec +++ b/gfs2-utils.spec @@ -1,7 +1,7 @@ ############################################################################### ############################################################################### ## -## Copyright (C) 2004-2013 Red Hat, Inc. All rights reserved. +## Copyright (C) 2004-2014 Red Hat, Inc. All rights reserved. ## ## This copyrighted material is made available to anyone wishing to use, ## modify, copy, or redistribute it subject to the terms and conditions @@ -11,8 +11,8 @@ ############################################################################### Name: gfs2-utils -Version: 3.1.6 -Release: 9%{?dist} +Version: 3.1.7 +Release: 1%{?dist} License: GPLv2+ and LGPLv2+ Group: System Environment/Kernel Summary: Utilities for managing the global file system (GFS2) @@ -40,19 +40,13 @@ URL: https://fedorahosted.org/cluster/wiki/HomePage # ./make-tarball.sh # Source0: https://fedorahosted.org/released/gfs2-utils/gfs2-utils-%{version}.tar.gz -Patch0: bz1079286-0-gfs2_grow_Don_t_try_to_open_an_empty_string.patch -Patch1: bz1079286-1-libgfs2_Add_lgfs2_open_mnt_functions.patch -Patch2: bz1079286-2-Switch_is_pathname_mounted_callers_to_lgfs2_open_mnt.patch -Patch3: bz1079286-3-libgfs2_Remove_is_pathname_mounted.patch +Patch0: gfs2_utils_tests_Build_unit_tests_with_consistent_cpp_flags.patch BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) %prep -%setup -q -n gfs2-utils -%patch0 -p1 -b .bz1079286-0-gfs2_grow_Don_t_try_to_open_an_empty_string -%patch1 -p1 -b .bz1079286-1-libgfs2_Add_lgfs2_open_mnt_functions -%patch2 -p1 -b .bz1079286-2-Switch_is_pathname_mounted_callers_to_lgfs2_open_mnt -%patch3 -p1 -b .bz1079286-3-libgfs2_Remove_is_pathname_mounted +%setup -q -n gfs2-utils-%{version} +%patch0 -p1 -b .gfs2_utils_tests_Build_unit_tests_with_consistent_cpp_flags %build ./autogen.sh @@ -99,6 +93,11 @@ file systems. %{_mandir}/man5/* %changelog +* Mon Sep 08 2014 Andrew Price - 3.1.7-1 +- New upstream release +- Drop all patches +- gfs2-utils tests: Build unit tests with consistent cpp flags + * Sat Aug 16 2014 Fedora Release Engineering - 3.1.6-9 - Rebuilt for https://fedoraproject.org/wiki/Fedora_21_22_Mass_Rebuild diff --git a/gfs2_utils_tests_Build_unit_tests_with_consistent_cpp_flags.patch b/gfs2_utils_tests_Build_unit_tests_with_consistent_cpp_flags.patch new file mode 100644 index 0000000..e61d3bf --- /dev/null +++ b/gfs2_utils_tests_Build_unit_tests_with_consistent_cpp_flags.patch @@ -0,0 +1,56 @@ +commit 54c5ee8ee1b5bf008400d1c86af3a44f15eb4970 +Author: Andrew Price +Date: Mon Sep 8 12:03:40 2014 +0200 + + gfs2-utils tests: Build unit tests with consistent cpp flags + + The check_rgrp unit test was failing on ARM and s390 due to struct + gfs2_sbd having a different alignment from libgfs2. This was due to the + unit tests being built with different options to libgfs2. Fix up the + unit test CPPFLAGS to make sure the alignment is consistent. + + Also bump the test timeout for check_rgrp to make sure it completes on + slower build machines. + + Signed-off-by: Andrew Price + +diff --git a/tests/Makefile.am b/tests/Makefile.am +index 70e77ef..b912dfa 100644 +--- a/tests/Makefile.am ++++ b/tests/Makefile.am +@@ -15,16 +15,22 @@ UNIT_CFLAGS = \ + UNIT_LDADD = \ + $(top_builddir)/gfs2/libgfs2/libgfs2.la \ + @check_LIBS@ ++UNIT_CPPFLAGS = \ ++ -D_FILE_OFFSET_BITS=64 \ ++ -D_LARGEFILE64_SOURCE \ ++ -D_GNU_SOURCE + + check_PROGRAMS = $(UNIT_TESTS) + + check_meta_SOURCES = $(UNIT_SOURCES) check_meta.c + check_meta_CFLAGS = $(UNIT_CFLAGS) + check_meta_LDADD = $(UNIT_LDADD) ++check_meta_CPPFLAGS = $(UNIT_CPPFLAGS) + + check_rgrp_SOURCES = $(UNIT_SOURCES) check_rgrp.c + check_rgrp_CFLAGS = $(UNIT_CFLAGS) + check_rgrp_LDADD = $(UNIT_LDADD) ++check_rgrp_CPPFLAGS = $(UNIT_CPPFLAGS) + endif + + # The `:;' works around a Bash 3.2 bug when the output is not writable. +diff --git a/tests/check_rgrp.c b/tests/check_rgrp.c +index d113846..e5fca8d 100644 +--- a/tests/check_rgrp.c ++++ b/tests/check_rgrp.c +@@ -125,7 +125,7 @@ static Suite * libgfs2_suite(void) + tcase_add_test(tc_rgrp, test_rbm_find_good); + tcase_add_test(tc_rgrp, test_rbm_find_bad); + tcase_add_test(tc_rgrp, test_rbm_find_lastblock); +- tcase_set_timeout(tc_rgrp, 60); ++ tcase_set_timeout(tc_rgrp, 120); + suite_add_tcase(s, tc_rgrp); + + return s; diff --git a/sources b/sources index f4312d3..0605e78 100644 --- a/sources +++ b/sources @@ -1 +1 @@ -9d47c272298c62868dd49c7b1f4bcefc gfs2-utils-3.1.6.tar.gz +b020102a326aecafef458fd1c9ccc5f3 gfs2-utils-3.1.7.tar.gz