* [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 @ 2015-11-05 8:30 Peter Rosin 2015-11-05 8:30 ` [Buildroot] [PATCH 1/2] rpcbind: backport adjustment to changes in libtirpc 1.0.1 Peter Rosin ` (2 more replies) 0 siblings, 3 replies; 16+ messages in thread From: Peter Rosin @ 2015-11-05 8:30 UTC (permalink / raw) To: buildroot From: Peter Rosin <peda@axentia.se> Hi! I'd like to bump libtirpc to 1.0.1. However there is an api incompatibility introduced in 0.3.2 which requires a patch to rpcbind, which I took from upstream. The rpcbind patch make rpcbind work with either tirpc 0.3.2 or 1.0.1, so I added that patch first in the series even if it might seem backwards. Note that I have not tested the rpcbind patch with more than a compile test. I know nothing about rpcbind. That also goes for the changes to libtirpc patch 0007-Disable-DES-..., which removes DES support, since apparently musl and uclibc do not provide DES. I know nothing about musl and uclibc and have not tested if there are more newfangled DES things that needs to be removed... Cheers, Peter Peter Rosin (2): rpcbind: backport adjustment to changes in libtirpc 1.0.1 libtirpc: bump to version 1.0.1 ...able-parts-of-TIRPC-requiring-NIS-support.patch | 14 ++-- ...0007-Disable-DES-authentification-support.patch | 85 +++++++++++++++++--- ...d-missing-rwlock_unlocks-in-xprt_register.patch | 63 +++++++++++++++ package/libtirpc/libtirpc.hash | 4 +- package/libtirpc/libtirpc.mk | 2 +- ...ly-Don-t-use-the-xp_auth-pointer-directly.patch | 42 ++++++++++ 6 files changed, 191 insertions(+), 19 deletions(-) create mode 100644 package/libtirpc/0008-Add-missing-rwlock_unlocks-in-xprt_register.patch create mode 100755 package/rpcbind/0002-handle_reply-Don-t-use-the-xp_auth-pointer-directly.patch -- 1.7.10.4 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 1/2] rpcbind: backport adjustment to changes in libtirpc 1.0.1 2015-11-05 8:30 [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 Peter Rosin @ 2015-11-05 8:30 ` Peter Rosin 2015-11-06 8:27 ` Peter Rosin 2015-11-05 8:30 ` [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 Peter Rosin 2015-11-05 9:05 ` [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 Thomas Petazzoni 2 siblings, 1 reply; 16+ messages in thread From: Peter Rosin @ 2015-11-05 8:30 UTC (permalink / raw) To: buildroot From: Peter Rosin <peda@axentia.se> Signed-off-by: Peter Rosin <peda@axentia.se> --- ...ly-Don-t-use-the-xp_auth-pointer-directly.patch | 42 ++++++++++++++++++++ 1 file changed, 42 insertions(+) create mode 100755 package/rpcbind/0002-handle_reply-Don-t-use-the-xp_auth-pointer-directly.patch diff --git a/package/rpcbind/0002-handle_reply-Don-t-use-the-xp_auth-pointer-directly.patch b/package/rpcbind/0002-handle_reply-Don-t-use-the-xp_auth-pointer-directly.patch new file mode 100755 index 0000000..6145514 --- /dev/null +++ b/package/rpcbind/0002-handle_reply-Don-t-use-the-xp_auth-pointer-directly.patch @@ -0,0 +1,42 @@ +From 9194122389f2a56b1cd1f935e64307e2e963c2da Mon Sep 17 00:00:00 2001 +From: Steve Dickson <steved@redhat.com> +Date: Mon, 2 Nov 2015 17:05:18 -0500 +Subject: [PATCH] handle_reply: Don't use the xp_auth pointer directly + +In the latest libtirpc version to access the xp_auth +one must use the SVC_XP_AUTH macro. To be backwards +compatible a couple ifdefs were added to use the +macro when it exists. + +Signed-off-by: Steve Dickson <steved@redhat.com> +[peda at axentia.se: backport from upstream] +Signed-off-by: Peter Rosin <peda@axentia.se> +--- + src/rpcb_svc_com.c | 7 +++++++ + 1 file changed, 7 insertions(+) + +diff --git a/src/rpcb_svc_com.c b/src/rpcb_svc_com.c +index 4ae93f1..22d6c84 100644 +--- a/src/rpcb_svc_com.c ++++ b/src/rpcb_svc_com.c +@@ -1295,10 +1295,17 @@ handle_reply(int fd, SVCXPRT *xprt) + a.rmt_localvers = fi->versnum; + + xprt_set_caller(xprt, fi); ++#if defined(SVC_XP_AUTH) ++ SVC_XP_AUTH(xprt) = svc_auth_none; ++#else + xprt->xp_auth = &svc_auth_none; ++#endif + svc_sendreply(xprt, (xdrproc_t) xdr_rmtcall_result, (char *) &a); ++#if !defined(SVC_XP_AUTH) + SVCAUTH_DESTROY(xprt->xp_auth); + xprt->xp_auth = NULL; ++#endif ++ + done: + if (buffer) + free(buffer); +-- +2.5.3 + -- 1.7.10.4 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 1/2] rpcbind: backport adjustment to changes in libtirpc 1.0.1 2015-11-05 8:30 ` [Buildroot] [PATCH 1/2] rpcbind: backport adjustment to changes in libtirpc 1.0.1 Peter Rosin @ 2015-11-06 8:27 ` Peter Rosin 0 siblings, 0 replies; 16+ messages in thread From: Peter Rosin @ 2015-11-06 8:27 UTC (permalink / raw) To: buildroot On 2015-11-05 09:30, Peter Rosin wrote: > From: Peter Rosin <peda@axentia.se> > > Signed-off-by: Peter Rosin <peda@axentia.se> > --- > ...ly-Don-t-use-the-xp_auth-pointer-directly.patch | 42 ++++++++++++++++++++ > 1 file changed, 42 insertions(+) > create mode 100755 package/rpcbind/0002-handle_reply-Don-t-use-the-xp_auth-pointer-directly.patch > > diff --git a/package/rpcbind/0002-handle_reply-Don-t-use-the-xp_auth-pointer-directly.patch b/package/rpcbind/0002-handle_reply-Don-t-use-the-xp_auth-pointer-directly.patch > new file mode 100755 Ouch, should be file mode 100644. Should I resend a -v2 series fixing this glitch? Cheers, Peter ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 2015-11-05 8:30 [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 Peter Rosin 2015-11-05 8:30 ` [Buildroot] [PATCH 1/2] rpcbind: backport adjustment to changes in libtirpc 1.0.1 Peter Rosin @ 2015-11-05 8:30 ` Peter Rosin 2015-11-08 13:30 ` Brendan Heading 2015-11-24 18:43 ` Thomas Petazzoni 2015-11-05 9:05 ` [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 Thomas Petazzoni 2 siblings, 2 replies; 16+ messages in thread From: Peter Rosin @ 2015-11-05 8:30 UTC (permalink / raw) To: buildroot From: Peter Rosin <peda@axentia.se> Adjust patch 0001 to trivial changes. Adjust patch 0007 to remove new DES things. Add patch 0008 directly from upstream to fix a late problem in the 1.0.1 release. Signed-off-by: Peter Rosin <peda@axentia.se> --- ...able-parts-of-TIRPC-requiring-NIS-support.patch | 14 ++-- ...0007-Disable-DES-authentification-support.patch | 85 +++++++++++++++++--- ...d-missing-rwlock_unlocks-in-xprt_register.patch | 63 +++++++++++++++ package/libtirpc/libtirpc.hash | 4 +- package/libtirpc/libtirpc.mk | 2 +- 5 files changed, 149 insertions(+), 19 deletions(-) create mode 100644 package/libtirpc/0008-Add-missing-rwlock_unlocks-in-xprt_register.patch diff --git a/package/libtirpc/0001-Disable-parts-of-TIRPC-requiring-NIS-support.patch b/package/libtirpc/0001-Disable-parts-of-TIRPC-requiring-NIS-support.patch index 7f58ab3..e51c5a4 100644 --- a/package/libtirpc/0001-Disable-parts-of-TIRPC-requiring-NIS-support.patch +++ b/package/libtirpc/0001-Disable-parts-of-TIRPC-requiring-NIS-support.patch @@ -11,6 +11,8 @@ Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> [joerg.krause at embedded.rocks: update for 0.3.2] Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks> +[peda at axentia.se: update for 1.0.1] +Signed-off-by: Peter Rosin <peda@axentia.se> --- src/Makefile.am | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) @@ -19,23 +21,23 @@ diff --git a/src/Makefile.am b/src/Makefile.am index 6cc567a..9834f9a 100644 --- a/src/Makefile.am +++ b/src/Makefile.am -@@ -51,7 +51,7 @@ libtirpc_la_SOURCES = auth_none.c auth_unix.c authunix_prot.c bindresvport.c cln - rpc_callmsg.c rpc_generic.c rpc_soc.c rpcb_clnt.c rpcb_prot.c \ +@@ -24,7 +24,7 @@ libtirpc_la_SOURCES = auth_none.c auth_unix.c authunix_prot.c bindresvport.c cln rpcb_st_xdr.c svc.c svc_auth.c svc_dg.c svc_auth_unix.c svc_auth_none.c \ + svc_auth_des.c \ svc_generic.c svc_raw.c svc_run.c svc_simple.c svc_vc.c getpeereid.c \ - auth_time.c auth_des.c authdes_prot.c debug.c + auth_des.c authdes_prot.c debug.c ## XDR - libtirpc_la_SOURCES += xdr.c xdr_rec.c xdr_array.c xdr_float.c xdr_mem.c xdr_reference.c xdr_stdio.c -@@ -68,8 +68,8 @@ if GSS + libtirpc_la_SOURCES += xdr.c xdr_rec.c xdr_array.c xdr_float.c xdr_mem.c xdr_reference.c xdr_stdio.c xdr_sizeof.c +@@ -41,8 +41,8 @@ if GSS libtirpc_la_CFLAGS = -DHAVE_RPCSEC_GSS $(GSSAPI_CFLAGS) endif -libtirpc_la_SOURCES += key_call.c key_prot_xdr.c getpublickey.c --libtirpc_la_SOURCES += netname.c netnamer.c rtime.c +-libtirpc_la_SOURCES += netname.c netnamer.c rpcdname.c rtime.c +#libtirpc_la_SOURCES += key_call.c key_prot_xdr.c getpublickey.c -+#libtirpc_la_SOURCES += netname.c netnamer.c rtime.c ++#libtirpc_la_SOURCES += netname.c netnamer.c rpcdname.c rtime.c CLEANFILES = cscope.* *~ DISTCLEANFILES = Makefile.in diff --git a/package/libtirpc/0007-Disable-DES-authentification-support.patch b/package/libtirpc/0007-Disable-DES-authentification-support.patch index c251811..3f13716 100644 --- a/package/libtirpc/0007-Disable-DES-authentification-support.patch +++ b/package/libtirpc/0007-Disable-DES-authentification-support.patch @@ -9,6 +9,8 @@ Content-Transfer-Encoding: 8bit uClibc and musl does not provide DES authentication. Signed-off-by: J?rg Krause <joerg.krause@embedded.rocks> +[peda at axentia.se: update for 1.0.1] +Signed-off-by: Peter Rosin <peda@axentia.se> --- src/Makefile.am | 2 +- src/rpc_soc.c | 32 -------------------------------- @@ -18,20 +20,35 @@ diff --git a/src/Makefile.am b/src/Makefile.am index 960a522..3a88e31 100644 --- a/src/Makefile.am +++ b/src/Makefile.am -@@ -51,7 +51,7 @@ libtirpc_la_SOURCES = auth_none.c auth_unix.c authunix_prot.c bindresvport.c cln +@@ -22,9 +22,8 @@ libtirpc_la_SOURCES = auth_none.c auth_unix.c authunix_prot.c bindresvport.c cln + pmap_prot.c pmap_prot2.c pmap_rmt.c rpc_prot.c rpc_commondata.c \ rpc_callmsg.c rpc_generic.c rpc_soc.c rpcb_clnt.c rpcb_prot.c \ rpcb_st_xdr.c svc.c svc_auth.c svc_dg.c svc_auth_unix.c svc_auth_none.c \ +- svc_auth_des.c \ svc_generic.c svc_raw.c svc_run.c svc_simple.c svc_vc.c getpeereid.c \ - auth_des.c authdes_prot.c debug.c + debug.c ## XDR - libtirpc_la_SOURCES += xdr.c xdr_rec.c xdr_array.c xdr_float.c xdr_mem.c xdr_reference.c xdr_stdio.c + libtirpc_la_SOURCES += xdr.c xdr_rec.c xdr_array.c xdr_float.c xdr_mem.c xdr_reference.c xdr_stdio.c xdr_sizeof.c +diff --git a/src/svc_auth.c b/src/svc_auth.c +--- a/src/svc_auth.c ++++ b/src/svc_auth.c +@@ -114,9 +114,6 @@ _gss_authenticate(rqst, msg, no_dispatch) + case AUTH_SHORT: + dummy = _svcauth_short(rqst, msg); + return (dummy); +- case AUTH_DES: +- dummy = _svcauth_des(rqst, msg); +- return (dummy); + #ifdef HAVE_RPCSEC_GSS + case RPCSEC_GSS: + dummy = _svcauth_gss(rqst, msg, no_dispatch); diff --git a/src/rpc_soc.c b/src/rpc_soc.c index e146ed4..161a1ec 100644 --- a/src/rpc_soc.c +++ b/src/rpc_soc.c -@@ -521,38 +521,6 @@ clnt_broadcast(prog, vers, proc, xargs, argsp, xresults, resultsp, eachresult) +@@ -522,86 +522,6 @@ clnt_broadcast(prog, vers, proc, xargs, argsp, xresults, resultsp, eachresult) } /* @@ -45,7 +62,6 @@ index e146ed4..161a1ec 100644 - struct sockaddr *syncaddr; /* optional hostaddr to sync with */ - des_block *ckey; /* optional conversation key to use */ -{ -- AUTH *dummy; - AUTH *nauth; - char hostname[NI_MAXHOST]; - @@ -54,19 +70,68 @@ index e146ed4..161a1ec 100644 - * Change addr to hostname, because that is the way - * new interface takes it. - */ -- if (getnameinfo(syncaddr, sizeof(syncaddr), hostname, -- sizeof hostname, NULL, 0, 0) != 0) -- goto fallback; -- +- switch (syncaddr->sa_family) { +- case AF_INET: +- if (getnameinfo(syncaddr, sizeof(struct sockaddr_in), hostname, +- sizeof hostname, NULL, 0, 0) != 0) +- goto fallback; +- break; +- case AF_INET6: +- if (getnameinfo(syncaddr, sizeof(struct sockaddr_in6), hostname, +- sizeof hostname, NULL, 0, 0) != 0) +- goto fallback; +- break; +- default: +- goto fallback; +- } - nauth = authdes_seccreate(servername, window, hostname, ckey); - return (nauth); - } -fallback: -- dummy = authdes_seccreate(servername, window, NULL, ckey); -- return (dummy); +- return authdes_seccreate(servername, window, NULL, ckey); -} - -/* +- * Create the client des authentication object. Obsoleted by +- * authdes_pk_seccreate(). +- */ +-extern AUTH *authdes_pk_seccreate(const char *, netobj *, u_int, const char *, +- const des_block *, nis_server *); +- +-AUTH * +-authdes_pk_create(servername, pkey, window, syncaddr, ckey) +- char *servername; /* network name of server */ +- netobj *pkey; /* public key */ +- u_int window; /* time to live */ +- struct sockaddr *syncaddr; /* optional hostaddr to sync with */ +- des_block *ckey; /* optional conversation key to use */ +-{ +- AUTH *nauth; +- char hostname[NI_MAXHOST]; +- +- if (syncaddr) { +- /* +- * Change addr to hostname, because that is the way +- * new interface takes it. +- */ +- switch (syncaddr->sa_family) { +- case AF_INET: +- if (getnameinfo(syncaddr, sizeof(struct sockaddr_in), hostname, +- sizeof hostname, NULL, 0, 0) != 0) +- goto fallback; +- break; +- default: +- goto fallback; +- } +- nauth = authdes_pk_seccreate(servername, pkey, window, hostname, ckey, NULL); +- return (nauth); +- } +-fallback: +- return authdes_pk_seccreate(servername, pkey, window, NULL, ckey, NULL); +-} +- +- +-/* * Create a client handle for a unix connection. Obsoleted by clnt_vc_create() */ CLIENT * diff --git a/package/libtirpc/0008-Add-missing-rwlock_unlocks-in-xprt_register.patch b/package/libtirpc/0008-Add-missing-rwlock_unlocks-in-xprt_register.patch new file mode 100644 index 0000000..56b5bf8 --- /dev/null +++ b/package/libtirpc/0008-Add-missing-rwlock_unlocks-in-xprt_register.patch @@ -0,0 +1,63 @@ +From 4f1503e84b2f7bd229a097335e52fb8203f5bb0b Mon Sep 17 00:00:00 2001 +From: Michael Forney <mforney@mforney.org> +Date: Wed, 4 Nov 2015 13:58:06 -0500 +Subject: [PATCH] Add missing rwlock_unlocks in xprt_register + +It looks like in b2c9430f46c4ac848957fb8adaac176a3f6ac03f when svc_run +switched to poll, an early return was added, but the rwlock was not +unlocked. + +I observed that rpcbind built against libtirpc-1.0.1 would handle only +one request before hanging, and tracked it down to a missing +rwlock_unlock here. + +Fixes: b2c9430f46c4 ('Use poll() instead of select() in svc_run()') +Signed-off-by: Michael Forney <mforney@mforney.org> +Signed-off-by: Steve Dickson <steved@redhat.com> +[peda at axentia.se: backport from upstream] +Signed-off-by: Peter Rosin <peda@axentia.se> +--- + src/svc.c | 7 ++++--- + 1 file changed, 4 insertions(+), 3 deletions(-) + +diff --git a/src/svc.c b/src/svc.c +index 9c41445..b59467b 100644 +--- a/src/svc.c ++++ b/src/svc.c +@@ -99,7 +99,7 @@ xprt_register (xprt) + { + __svc_xports = (SVCXPRT **) calloc (_rpc_dtablesize(), sizeof (SVCXPRT *)); + if (__svc_xports == NULL) +- return; ++ goto unlock; + } + if (sock < _rpc_dtablesize()) + { +@@ -120,14 +120,14 @@ xprt_register (xprt) + svc_pollfd[i].fd = sock; + svc_pollfd[i].events = (POLLIN | POLLPRI | + POLLRDNORM | POLLRDBAND); +- return; ++ goto unlock; + } + + new_svc_pollfd = (struct pollfd *) realloc (svc_pollfd, + sizeof (struct pollfd) + * (svc_max_pollfd + 1)); + if (new_svc_pollfd == NULL) /* Out of memory */ +- return; ++ goto unlock; + svc_pollfd = new_svc_pollfd; + ++svc_max_pollfd; + +@@ -135,6 +135,7 @@ xprt_register (xprt) + svc_pollfd[svc_max_pollfd - 1].events = (POLLIN | POLLPRI | + POLLRDNORM | POLLRDBAND); + } ++unlock: + rwlock_unlock (&svc_fd_lock); + } + +-- +2.5.3 + diff --git a/package/libtirpc/libtirpc.hash b/package/libtirpc/libtirpc.hash index f171870..cd65164 100644 --- a/package/libtirpc/libtirpc.hash +++ b/package/libtirpc/libtirpc.hash @@ -1,4 +1,4 @@ # From sourceforge's info on download page: -sha1 af9b74d0c4d1499a7b1a43e396e5b7d62180ea65 libtirpc-0.3.2.tar.bz2 +sha1 8da1636f98b5909c0d587e7534bc1e91f5c1a970 libtirpc-1.0.1.tar.bz2 # Locally computed -sha256 2008a379f37f2c5d5a87a568b06707422cc3e4f5da305f7fb71f3f4c6d473ffc libtirpc-0.3.2.tar.bz2 +sha256 5156974f31be7ccbc8ab1de37c4739af6d9d42c87b1d5caf4835dda75fcbb89e libtirpc-1.0.1.tar.bz2 diff --git a/package/libtirpc/libtirpc.mk b/package/libtirpc/libtirpc.mk index 8f49356..70a31ac 100644 --- a/package/libtirpc/libtirpc.mk +++ b/package/libtirpc/libtirpc.mk @@ -4,7 +4,7 @@ # ################################################################################ -LIBTIRPC_VERSION = 0.3.2 +LIBTIRPC_VERSION = 1.0.1 LIBTIRPC_SOURCE = libtirpc-$(LIBTIRPC_VERSION).tar.bz2 LIBTIRPC_SITE = http://downloads.sourceforge.net/project/libtirpc/libtirpc/$(LIBTIRPC_VERSION) LIBTIRPC_LICENSE = BSD-3c -- 1.7.10.4 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 2015-11-05 8:30 ` [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 Peter Rosin @ 2015-11-08 13:30 ` Brendan Heading 2015-11-08 13:37 ` Thomas Petazzoni 2015-11-24 18:43 ` Thomas Petazzoni 1 sibling, 1 reply; 16+ messages in thread From: Brendan Heading @ 2015-11-08 13:30 UTC (permalink / raw) To: buildroot On 5 November 2015 at 08:30, Peter Rosin <peda@lysator.liu.se> wrote: > From: Peter Rosin <peda@axentia.se> > > Adjust patch 0001 to trivial changes. > Adjust patch 0007 to remove new DES things. > Add patch 0008 directly from upstream to fix a late problem in the > 1.0.1 release. Thomas, Peter, FYI the new 1.0.1 version of libtirpc contains my patch to remove the calls to atomic operations that were causing a lot of autobuild failures on SPARC (and probably some other lesser-used arches too). Took a bit of negotiation to get the patch into a form they were happy to accept, but we got there in the end. So when the bump to 1.0.1 goes in we can expect this class of autobuild issues to go away. regards Brendan ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 2015-11-08 13:30 ` Brendan Heading @ 2015-11-08 13:37 ` Thomas Petazzoni 2015-11-15 7:43 ` Peter Rosin 0 siblings, 1 reply; 16+ messages in thread From: Thomas Petazzoni @ 2015-11-08 13:37 UTC (permalink / raw) To: buildroot Dear Brendan Heading, On Sun, 8 Nov 2015 13:30:45 +0000, Brendan Heading wrote: > FYI the new 1.0.1 version of libtirpc contains my patch to remove the > calls to atomic operations that were causing a lot of autobuild > failures on SPARC (and probably some other lesser-used arches too). > Took a bit of negotiation to get the patch into a form they were happy > to accept, but we got there in the end. > > So when the bump to 1.0.1 goes in we can expect this class of > autobuild issues to go away. Excellent, thanks a lot for working on this topic! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 2015-11-08 13:37 ` Thomas Petazzoni @ 2015-11-15 7:43 ` Peter Rosin 2015-11-16 22:08 ` Thomas Petazzoni 0 siblings, 1 reply; 16+ messages in thread From: Peter Rosin @ 2015-11-15 7:43 UTC (permalink / raw) To: buildroot On 2015-11-08 14:37, Thomas Petazzoni wrote: > Dear Brendan Heading, > > On Sun, 8 Nov 2015 13:30:45 +0000, Brendan Heading wrote: > >> FYI the new 1.0.1 version of libtirpc contains my patch to remove the >> calls to atomic operations that were causing a lot of autobuild >> failures on SPARC (and probably some other lesser-used arches too). >> Took a bit of negotiation to get the patch into a form they were happy >> to accept, but we got there in the end. >> >> So when the bump to 1.0.1 goes in we can expect this class of >> autobuild issues to go away. > Excellent, thanks a lot for working on this topic! Hi! What are we waiting for anyway? Anything I can do to move this along? I thought I had straightened out all question marks? Cheers, Peter ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 2015-11-15 7:43 ` Peter Rosin @ 2015-11-16 22:08 ` Thomas Petazzoni 0 siblings, 0 replies; 16+ messages in thread From: Thomas Petazzoni @ 2015-11-16 22:08 UTC (permalink / raw) To: buildroot Dear Peter Rosin, On Sun, 15 Nov 2015 08:43:50 +0100, Peter Rosin wrote: > What are we waiting for anyway? Anything I can do to move this along? > > I thought I had straightened out all question marks? We are right now preparing the 2015.11 release, so at this point, no version bump is merged in master. We are merging some of the new features and version bumps in "next" but this tends to happen more slowly than usual since we try to focus a bit our effort on the master branch. But your patch is not forgotten, it is listed in our patch tracking system at http://patchwork.ozlabs.org/project/buildroot/list/. Thanks! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 2015-11-05 8:30 ` [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 Peter Rosin 2015-11-08 13:30 ` Brendan Heading @ 2015-11-24 18:43 ` Thomas Petazzoni 2015-11-24 21:52 ` Peter Rosin 1 sibling, 1 reply; 16+ messages in thread From: Thomas Petazzoni @ 2015-11-24 18:43 UTC (permalink / raw) To: buildroot Dear Peter Rosin, On Thu, 5 Nov 2015 09:30:10 +0100, Peter Rosin wrote: > From: Peter Rosin <peda@axentia.se> > > Adjust patch 0001 to trivial changes. > Adjust patch 0007 to remove new DES things. > Add patch 0008 directly from upstream to fix a late problem in the > 1.0.1 release. > > Signed-off-by: Peter Rosin <peda@axentia.se> This patch breaks the build of libtirpc, with the following error: rpc_soc.c:64:24: fatal error: rpcsvc/nis.h: No such file or directory compilation terminated. Makefile:749: recipe for target 'libtirpc_la-rpc_soc.lo' failed make[4]: *** [libtirpc_la-rpc_soc.lo] Error 1 make[4]: *** Waiting for unfinished jobs.... Makefile:552: recipe for target 'all-recursive' failed make[3]: *** [all-recursive] Error 1 Makefile:413: recipe for target 'all' failed make[2]: *** [all] Error 2 package/pkg-generic.mk:188: recipe for target '/home/thomas/projets/buildroot/output/build/libtirpc-1.0.1/.stamp_built' failed make[1]: *** [/home/thomas/projets/buildroot/output/build/libtirpc-1.0.1/.stamp_built] Error 2 Makefile:36: recipe for target '_all' failed make: *** [_all] Error 2 Without this patch applied, it builds fine. The failing configuration is: BR2_arm=y BR2_TOOLCHAIN_EXTERNAL=y BR2_TOOLCHAIN_EXTERNAL_CUSTOM=y BR2_TOOLCHAIN_EXTERNAL_DOWNLOAD=y BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/br-arm-full-2015.11-rc1-71-g90d1299.tar.bz2" BR2_TOOLCHAIN_EXTERNAL_GCC_4_7=y BR2_TOOLCHAIN_EXTERNAL_HEADERS_3_10=y BR2_TOOLCHAIN_EXTERNAL_LOCALE=y # BR2_TOOLCHAIN_EXTERNAL_HAS_THREADS_DEBUG is not set BR2_TOOLCHAIN_EXTERNAL_INET_RPC=y BR2_TOOLCHAIN_EXTERNAL_CXX=y BR2_INIT_NONE=y BR2_SYSTEM_BIN_SH_NONE=y # BR2_PACKAGE_BUSYBOX is not set BR2_PACKAGE_RPCBIND=y # BR2_TARGET_ROOTFS_TAR is not set Can you look into this? Once fixed, maybe you can resend the series, and fix the permission problem in PATCH 1/2 at the same time? Thanks a lot! Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 2015-11-24 18:43 ` Thomas Petazzoni @ 2015-11-24 21:52 ` Peter Rosin 2015-11-24 22:08 ` Thomas Petazzoni 0 siblings, 1 reply; 16+ messages in thread From: Peter Rosin @ 2015-11-24 21:52 UTC (permalink / raw) To: buildroot Hi Thomas, On 2015-11-24 19:43, Thomas Petazzoni wrote: > Dear Peter Rosin, > > On Thu, 5 Nov 2015 09:30:10 +0100, Peter Rosin wrote: >> From: Peter Rosin <peda@axentia.se> >> >> Adjust patch 0001 to trivial changes. >> Adjust patch 0007 to remove new DES things. >> Add patch 0008 directly from upstream to fix a late problem in the >> 1.0.1 release. >> >> Signed-off-by: Peter Rosin <peda@axentia.se> > This patch breaks the build of libtirpc, with the following error: > > rpc_soc.c:64:24: fatal error: rpcsvc/nis.h: No such file or directory > compilation terminated. > Makefile:749: recipe for target 'libtirpc_la-rpc_soc.lo' failed > make[4]: *** [libtirpc_la-rpc_soc.lo] Error 1 > make[4]: *** Waiting for unfinished jobs.... > Makefile:552: recipe for target 'all-recursive' failed > make[3]: *** [all-recursive] Error 1 > Makefile:413: recipe for target 'all' failed > make[2]: *** [all] Error 2 > package/pkg-generic.mk:188: recipe for target '/home/thomas/projets/buildroot/output/build/libtirpc-1.0.1/.stamp_built' failed > make[1]: *** [/home/thomas/projets/buildroot/output/build/libtirpc-1.0.1/.stamp_built] Error 2 > Makefile:36: recipe for target '_all' failed > make: *** [_all] Error 2 > > Without this patch applied, it builds fine. The failing configuration > is: > > BR2_arm=y > BR2_TOOLCHAIN_EXTERNAL=y > BR2_TOOLCHAIN_EXTERNAL_CUSTOM=y > BR2_TOOLCHAIN_EXTERNAL_DOWNLOAD=y > BR2_TOOLCHAIN_EXTERNAL_URL="http://autobuild.buildroot.org/toolchains/tarballs/br-arm-full-2015.11-rc1-71-g90d1299.tar.bz2" > BR2_TOOLCHAIN_EXTERNAL_GCC_4_7=y > BR2_TOOLCHAIN_EXTERNAL_HEADERS_3_10=y > BR2_TOOLCHAIN_EXTERNAL_LOCALE=y > # BR2_TOOLCHAIN_EXTERNAL_HAS_THREADS_DEBUG is not set > BR2_TOOLCHAIN_EXTERNAL_INET_RPC=y > BR2_TOOLCHAIN_EXTERNAL_CXX=y > BR2_INIT_NONE=y > BR2_SYSTEM_BIN_SH_NONE=y > # BR2_PACKAGE_BUSYBOX is not set > BR2_PACKAGE_RPCBIND=y > # BR2_TARGET_ROOTFS_TAR is not set > > Can you look into this? Once fixed, maybe you can resend the series, > and fix the permission problem in PATCH 1/2 at the same time? The toolchain does not match my arch, so I couldn't actually test it, but the fix looked simple enough (the include of rpcsvc/nis.h was added in the same upstream commit that added a new function that I also had to remove, and now I simply killed that include too). So, there's a good chance this will work better even if I didn't actually test the failing case. Cheers, Peter ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 2015-11-24 21:52 ` Peter Rosin @ 2015-11-24 22:08 ` Thomas Petazzoni 2015-11-24 22:12 ` Peter Rosin 0 siblings, 1 reply; 16+ messages in thread From: Thomas Petazzoni @ 2015-11-24 22:08 UTC (permalink / raw) To: buildroot Dear Peter Rosin, On Tue, 24 Nov 2015 22:52:42 +0100, Peter Rosin wrote: > The toolchain does not match my arch, so I couldn't actually test it, but the fix Just curious, which arch are you running? I think this toolchain is a x86 toolchain, so it should work pretty much on every PC out there. But maybe you're using a different platform? Best regards, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 2015-11-24 22:08 ` Thomas Petazzoni @ 2015-11-24 22:12 ` Peter Rosin 2015-11-25 8:05 ` Thomas Petazzoni 0 siblings, 1 reply; 16+ messages in thread From: Peter Rosin @ 2015-11-24 22:12 UTC (permalink / raw) To: buildroot On 2015-11-24 23:08, Thomas Petazzoni wrote: > Dear Peter Rosin, > > On Tue, 24 Nov 2015 22:52:42 +0100, Peter Rosin wrote: > >> The toolchain does not match my arch, so I couldn't actually test it, but the fix > Just curious, which arch are you running? I think this toolchain is a > x86 toolchain, so it should work pretty much on every PC out there. But > maybe you're using a different platform? The toolchain is x86_64 it seems, and I have a plain old x86 kernel on the system in question. Yes yes, I could have moved everything over somewhere. But didn't. :-) Cheers, Peter ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 2015-11-24 22:12 ` Peter Rosin @ 2015-11-25 8:05 ` Thomas Petazzoni 0 siblings, 0 replies; 16+ messages in thread From: Thomas Petazzoni @ 2015-11-25 8:05 UTC (permalink / raw) To: buildroot Peter, On Tue, 24 Nov 2015 23:12:30 +0100, Peter Rosin wrote: > >> The toolchain does not match my arch, so I couldn't actually test it, but the fix > > Just curious, which arch are you running? I think this toolchain is a > > x86 toolchain, so it should work pretty much on every PC out there. But > > maybe you're using a different platform? > > The toolchain is x86_64 it seems, and I have a plain old x86 kernel on the system > in question. Yes yes, I could have moved everything over somewhere. But didn't. :-) Hum, yes, indeed the pre-built Buildroot toolchains are meant to be used on a x86-64 system, my bad. I'll test your new series. Thanks, Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 2015-11-05 8:30 [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 Peter Rosin 2015-11-05 8:30 ` [Buildroot] [PATCH 1/2] rpcbind: backport adjustment to changes in libtirpc 1.0.1 Peter Rosin 2015-11-05 8:30 ` [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 Peter Rosin @ 2015-11-05 9:05 ` Thomas Petazzoni 2015-11-05 10:44 ` Peter Rosin 2 siblings, 1 reply; 16+ messages in thread From: Thomas Petazzoni @ 2015-11-05 9:05 UTC (permalink / raw) To: buildroot Dear Peter Rosin, On Thu, 5 Nov 2015 09:30:08 +0100, Peter Rosin wrote: > I'd like to bump libtirpc to 1.0.1. However there is an api incompatibility > introduced in 0.3.2 which requires a patch to rpcbind, which I took from > upstream. The rpcbind patch make rpcbind work with either tirpc 0.3.2 or > 1.0.1, so I added that patch first in the series even if it might seem > backwards. Thanks for those patches. Is rpcbind the only user of libtirpc that is affected by the API change ? Thomas -- Thomas Petazzoni, CTO, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 2015-11-05 9:05 ` [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 Thomas Petazzoni @ 2015-11-05 10:44 ` Peter Rosin 2015-11-06 8:06 ` Peter Rosin 0 siblings, 1 reply; 16+ messages in thread From: Peter Rosin @ 2015-11-05 10:44 UTC (permalink / raw) To: buildroot On 2015-11-05 10:05, Thomas Petazzoni wrote: > Dear Peter Rosin, > > On Thu, 5 Nov 2015 09:30:08 +0100, Peter Rosin wrote: > >> I'd like to bump libtirpc to 1.0.1. However there is an api incompatibility >> introduced in 0.3.2 which requires a patch to rpcbind, which I took from oops, introduced in -> introduced since >> upstream. The rpcbind patch make rpcbind work with either tirpc 0.3.2 or >> 1.0.1, so I added that patch first in the series even if it might seem >> backwards. > Thanks for those patches. Is rpcbind the only user of libtirpc that is > affected by the API change ? > > Thomas Good question... I think so, but who knows? Personally I would have kept libtirpc compatible (that was a possibility), but upstream decided to make the switch since they didn't know of any users and wanted to get in line with other tirpc implementations. Then this regression in rpcbind was discovered, which was rather close to home... The changed interface was thought to be only for adding custom authenticators to RPC, something that is thought to be extremely specialized, but the rpcbind use looks like a workaround for problems getting replies out when using libtirpc somewhat strangely (again, I don't know rpcbind and have only had a cursory look at the code). If a package is using libtirpc normally and do no strange things (unlike rpcbind, which probably is quite special) and don't add its own application specific RPC authenticator, it should not be affected. But as I said, who knows? Fixing breakage in libtirpc users should be straightforward if any come up, and another possibility is to add a patch to revert the final change that removed the old interface. However, that revert is touching core areas in a number of places and is probably not fun to keep around for any length of time. It can be added if needed, is that good enough? I can only find the upstream discussion in the mail archive at sf, which has a rather sucky interface, sorry. Cheers, Peter https://sourceforge.net/p/libtirpc/mailman/libtirpc-devel/thread/5630F3AA.6050201%40RedHat.com/#msg34576143 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 2015-11-05 10:44 ` Peter Rosin @ 2015-11-06 8:06 ` Peter Rosin 0 siblings, 0 replies; 16+ messages in thread From: Peter Rosin @ 2015-11-06 8:06 UTC (permalink / raw) To: buildroot On 2015-11-05 11:44, Peter Rosin wrote: > On 2015-11-05 10:05, Thomas Petazzoni wrote: >> Dear Peter Rosin, >> >> On Thu, 5 Nov 2015 09:30:08 +0100, Peter Rosin wrote: >> >>> I'd like to bump libtirpc to 1.0.1. However there is an api incompatibility >>> introduced in 0.3.2 which requires a patch to rpcbind, which I took from > oops, introduced in -> introduced since >>> upstream. The rpcbind patch make rpcbind work with either tirpc 0.3.2 or >>> 1.0.1, so I added that patch first in the series even if it might seem >>> backwards. >> Thanks for those patches. Is rpcbind the only user of libtirpc that is >> affected by the API change ? >> >> Thomas > Good question... So, I grepped for "xp_auth" in the sources of all other packages that sport a dependency on libtirpc (argus, busybox, conntrack-tools, lmbench, ltp-testsuite, nfs-utils, quota and xinetd) and came up empty. Good enough? Cheers, Peter ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2015-11-25 8:05 UTC | newest] Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-11-05 8:30 [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 Peter Rosin 2015-11-05 8:30 ` [Buildroot] [PATCH 1/2] rpcbind: backport adjustment to changes in libtirpc 1.0.1 Peter Rosin 2015-11-06 8:27 ` Peter Rosin 2015-11-05 8:30 ` [Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1 Peter Rosin 2015-11-08 13:30 ` Brendan Heading 2015-11-08 13:37 ` Thomas Petazzoni 2015-11-15 7:43 ` Peter Rosin 2015-11-16 22:08 ` Thomas Petazzoni 2015-11-24 18:43 ` Thomas Petazzoni 2015-11-24 21:52 ` Peter Rosin 2015-11-24 22:08 ` Thomas Petazzoni 2015-11-24 22:12 ` Peter Rosin 2015-11-25 8:05 ` Thomas Petazzoni 2015-11-05 9:05 ` [Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1 Thomas Petazzoni 2015-11-05 10:44 ` Peter Rosin 2015-11-06 8:06 ` Peter Rosin
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.