shared snapshot LVM patch compile error

[devicemapper niko <devicemapper.niko@xxxxxxxxx>]

less arguments of lv_info in lvm-shared-merge-dont-delete.patch caused this error,

origin_only should be 0 ? I didn't look inside the code,so ask you guys

 

in lvm-shared-merge-dont-delete.patch

Index: LVM2.2.02.73/lib/metadata/lv_manip.c
===================================================================
--- LVM2.2.02.73.orig/lib/metadata/lv_manip.c   2010-09-10 19:15:20.000000000 +0200
+++ LVM2.2.02.73/lib/metadata/lv_manip.c        2010-09-10 19:15:33.000000000 +0200
@@ -2354,6 +2354,38 @@ int lv_remove_with_dependencies(struct c
                struct logical_volume *origin = origin_from_cow(lv);
                /* Remove shared snapshot's virtual snapshots first */
                if (lv_is_multisnap_origin(origin)) {
+                       if (!level) {
+                               int origin_active, merging;
+                               struct lvinfo info;
+                               if (!lv_info(cmd, origin, &info, 0, 0)) {                            -----------------> here
+                                       log_error("Check for existence of snapshot "
+                                                 "origin '%s' failed.", origin->name);
+                                       return 0;
+                               }

 

 

gcc -c -I. -I../include -DHAVE_CONFIG_H -DDM_IOCTLS -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -fPIC -Wall -Wundef -Wshadow -Wcast-align -Wwrite-strings -Wmissing-prototypes -Wmissing-declarations -Wnested-externs -Winline -Wmissing-noreturn -Wformat-security -Wredundant-decls -O2 -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -fPIC -Wall -Wundef -Wshadow -Wcast-align -Wwrite-strings -Wmissing-prototypes -Wmissing-declarations -Wnested-externs -Winline -Wmissing-noreturn -Wformat-security -Wredundant-decls -O2 locking/no_locking.c -o locking/no_locking.o
gcc -c -I. -I../include -DHAVE_CONFIG_H -DDM_IOCTLS -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -fPIC -Wall -Wundef -Wshadow -Wcast-align -Wwrite-strings -Wmissing-prototypes -Wmissing-declarations -Wnested-externs -Winline -Wmissing-noreturn -Wformat-security -Wredundant-decls -O2 -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -fPIC -Wall -Wundef -Wshadow -Wcast-align -Wwrite-strings -Wmissing-prototypes -Wmissing-declarations -Wnested-externs -Winline -Wmissing-noreturn -Wformat-security -Wredundant-decls -O2 log/log.c -o log/log.o
gcc -c -I. -I../include -DHAVE_CONFIG_H -DDM_IOCTLS -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -fPIC -Wall -Wundef -Wshadow -Wcast-align -Wwrite-strings -Wmissing-prototypes -Wmissing-declarations -Wnested-externs -Winline -Wmissing-noreturn -Wformat-security -Wredundant-decls -O2 -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic -fPIC -Wall -Wundef -Wshadow -Wcast-align -Wwrite-strings -Wmissing-prototypes -Wmissing-declarations -Wnested-externs -Winline -Wmissing-noreturn -Wformat-security -Wredundant-decls -O2 metadata/lv_manip.c -o metadata/lv_manip.o
metadata/lv_manip.c: In function 'lv_remove_with_dependencies':
metadata/lv_manip.c:2360: warning: passing argument 3 of 'lv_info' makes integer from pointer without a cast
../include/activate.h:80: note: expected 'unsigned int' but argument is of type 'struct lvinfo *'
metadata/lv_manip.c:2360: error: too few arguments to function 'lv_info'
make[1]: *** [metadata/lv_manip.o] Error 1
make[1]: Leaving directory `/root/rpmbuild/BUILD/LVM2.2.02.72/lib'
make: *** [lib] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.RpT3fj (%build)

--
dm-devel mailing list
dm-devel@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/dm-devel