Re: [PATCH] Clean up multipath linking

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Le Thu, 2 Apr 2009 23:19:36 -0500,
Benjamin Marzinski <bmarzins@xxxxxxxxxx> a écrit :

> This is a patch to fix up the linking. It does two things.  First, it
> makes libmultipath.so install to /lib/ just like a normal shared
> library, so you don't have to use -rpath to link to it.  Second, and
> more importantly, it moves the libaio linking into
> libcheckdirectio.so, where it belongs.  Since libcheckdirectio.so is
> a dynamic shared object, multipath and multipathd don't know what
> functions they need to link in from libaio. This fixes the directio
> lockup for me.
> 
How grateful I am. I was so focused on finding a bug in the async code,
I missed this linking regression.

> Signed-off-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx>
> ---
>  libmultipath/Makefile          |    7 ++++---
>  libmultipath/checkers/Makefile |    3 +++
>  multipath/Makefile             |    5 ++---
>  multipathd/Makefile            |    4 ++--
>  4 files changed, 11 insertions(+), 8 deletions(-)
> 
> Index: multipath-tools-090402/libmultipath/Makefile
> ===================================================================
> --- multipath-tools-090402.orig/libmultipath/Makefile
> +++ multipath-tools-090402/libmultipath/Makefile
> @@ -23,14 +23,15 @@ endif
>  all: $(LIBS)
>  
>  $(LIBS): $(OBJS)
> -	$(CC) $(SHARED_FLAGS) $(CFLAGS) -o $@ $(OBJS)
> +	$(CC) $(SHARED_FLAGS) -Wl,-soname,$@ $(CFLAGS) -o $@ $(OBJS)
>  
>  install:
> +	$(INSTALL_PROGRAM) -d $(DESTDIR)$(prefix)/lib
> +	$(INSTALL_PROGRAM) -m 755 $(LIBS)
> $(DESTDIR)$(prefix)/lib/$(LIBS) $(INSTALL_PROGRAM) -m 755 -d
> $(DESTDIR)$(libdir)
> -	$(INSTALL_PROGRAM) -m 755 $(LIBS) $(DESTDIR)$(libdir)/$(LIBS)
>  
>  uninstall:
> -	rm -f $(DESTDIR)$(libdir)/$(LIBS)
> +	rm -f $(DESTDIR)$(prefix)/lib/$(LIBS)
>  
>  clean:
>  	rm -f core *.a *.o *.gz *.so
> Index: multipath-tools-090402/multipath/Makefile
> ===================================================================
> --- multipath-tools-090402.orig/multipath/Makefile
> +++ multipath-tools-090402/multipath/Makefile
> @@ -6,9 +6,8 @@ include ../Makefile.inc
>  
>  OBJS = main.o
>  
> -CFLAGS += -I$(multipathdir) -Wl,-rpath,$(libdir)
> -LDFLAGS += -lpthread -ldevmapper -laio -ldl \
> -	   -lmultipath -L$(multipathdir)
> +CFLAGS += -I$(multipathdir)
> +LDFLAGS += -lpthread -ldevmapper -ldl -lmultipath -L$(multipathdir)
>  
>  EXEC = multipath
>  
> Index: multipath-tools-090402/multipathd/Makefile
> ===================================================================
> --- multipath-tools-090402.orig/multipathd/Makefile
> +++ multipath-tools-090402/multipathd/Makefile
> @@ -5,8 +5,8 @@ include ../Makefile.inc
>  #
>  # basic flags setting
>  #
> -CFLAGS += -I$(multipathdir) -Wl,-rpath,$(libdir)
> -LDFLAGS += -lpthread -ldevmapper -lreadline -lncurses -laio -ldl \
> +CFLAGS += -I$(multipathdir)
> +LDFLAGS += -lpthread -ldevmapper -lreadline -lncurses -ldl \
>  	   -lmultipath -L$(multipathdir)
>  
>  #
> Index: multipath-tools-090402/libmultipath/checkers/Makefile
> ===================================================================
> --- multipath-tools-090402.orig/libmultipath/checkers/Makefile
> +++ multipath-tools-090402/libmultipath/checkers/Makefile
> @@ -17,6 +17,9 @@ CFLAGS += -I..
>  
>  all: $(LIBS)
>  
> +libcheckdirectio.so: libsg.o directio.o
> +	$(CC) $(SHARED_FLAGS) -o $@ $^ -laio
> +
>  libcheck%.so: libsg.o %.o
>  	$(CC) $(SHARED_FLAGS) -o $@ $^
>  
> 
> --
> dm-devel mailing list
> dm-devel@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/dm-devel

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

[Index of Archives]     [DM Crypt]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Packaging]     [Fedora SELinux]     [Yosemite Discussion]     [KDE Users]     [Fedora Docs]

  Powered by Linux