[Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1

[Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1

[Peter Rosin]

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

[Buildroot] [PATCH 1/2] rpcbind: backport adjustment to changes in libtirpc 1.0.1

[Peter Rosin]

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

[Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1

[Peter Rosin]

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

[Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1

[Thomas Petazzoni]

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

[Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1

[Peter Rosin]

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

[Buildroot] [PATCH 0/2] bump libtirpc to 1.0.1

[Peter Rosin]

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

[Buildroot] [PATCH 1/2] rpcbind: backport adjustment to changes in libtirpc 1.0.1

[Peter Rosin]

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

[Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1

[Brendan Heading]

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

[Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1

[Thomas Petazzoni]

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

[Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1

[Peter Rosin]

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

[Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1

[Thomas Petazzoni]

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

[Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1

[Thomas Petazzoni]

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

[Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1

[Peter Rosin]

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

[Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1

[Thomas Petazzoni]

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

[Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1

[Peter Rosin]

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

[Buildroot] [PATCH 2/2] libtirpc: bump to version 1.0.1

[Thomas Petazzoni]

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