On 03/04/2011 11:22 PM, Bernhard Reutner-Fischer wrote: > Bernd Schmidt <[email protected]> wrote: > > I've cleaned up a few issues in the previous version of these > patches, such as those pointed out by Bernhard, a numer of > whitespace issues, and some dead code. Mark Salter has also pointed > out that the FINISH_BOOTSTRAP_RELOC macro isn't actually needed when > ld.so <http://ld.so> is built with gcc, so I've removed that as > well. Bernd > ------------------------------------------------------------------------ > uClibc mailing list [email protected] > http://lists.busybox.net/mailman/listinfo/uclibc > > > Hi, > > The port is ok but I am not convinced about the xdr change, API-wise.
Thanks. I've committed patches 1-4, 7. As for the xdr change - it fixes incorrect code, and I offer http://www.mail-archive.com/[email protected]/msg05574.html as evidence that not just the C6X port is affected. I don't know much about RPC code. I've had a bit of a grep inside uClinux-dist to find instances of functions that are cast to xdrproc_t. These are in portmap, mysql, and busybox - all of these seem to take two arguments only. Googling suggests that the function that wants three arguments is xdr_string, and for that, xdr_wrapstring exists. What about patch 5, the fallthrough bits? This seems to have been fixed in glibc in commit f0ccf6ea41931f325df4699a4c7fdf81888563ee. Bernd _______________________________________________ uClibc mailing list [email protected] http://lists.busybox.net/mailman/listinfo/uclibc
