Re: [PATCH hurd 28/30] libports: use protected payloads to optimize the object lookup

2014-12-08 Thread Samuel Thibault
Justus Winter, le Thu 04 Dec 2014 15:39:04 +0100, a écrit : > If we now `ports_destroy_right' X, and at the same time the last send > right to X is destroyed, there's a chance that said notification is > already delivered to X. Mmm, isn't that also true for any kind of message? Samuel

Re: [PATCH hurd] include: don't install nonexistent refcount.h

2014-12-08 Thread Samuel Thibault
David Michael, le Sun 07 Dec 2014 23:21:16 -0500, a écrit : > * include/Makefile (installhdrs): Remove refcount.h. Justus seems to have missed it indeed, applied, thanks! Samuel

Re: [PATCH gnumach] Retire procedure `old_mach_port_get_receive_status'

2014-12-08 Thread Samuel Thibault
Justus Winter, le Mon 08 Dec 2014 10:48:19 +0100, a écrit : > Retire the compatibility RPC `old_mach_port_get_receive_status' that > works like `mach_port_get_receive_status' but returns an > `old_mach_port_status' object that lacks the `mps_seqno' field. > > Do not remove the type yet, so we do n

Re: [PATCH gnumach 1/2] kern: provide notifications about new tasks

2014-12-08 Thread Samuel Thibault
Justus Winter, le Mon 08 Dec 2014 13:55:18 +0100, a écrit : > I thought the message carries a reference that we can keep? Ah, right, sorry. I somehow never got used to that asymmetry of references. Then ack! Samuel

Re: [PATCH hurd 27/30] hurd: add intranpayload functions to all hurd types

2014-12-08 Thread Samuel Thibault
David Michael, le Mon 08 Dec 2014 13:45:50 -0500, a écrit : > On Thu, Nov 27, 2014 at 8:19 AM, Justus Winter > <4win...@informatik.uni-hamburg.de> wrote: > > +#ifdef HURD_DEFAULT_PAYLOAD_TO_PORT > > +#if HURD_DEFAULT_PAYLOAD_TO_PORT > > +/* Any non-numeric value will fail this test. If 1 (or any n

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Samuel Thibault
Hello, I investigated a bit and found the issue, which probably explains quite a few other bugs: - I added fprintf(stderr,"%s:%s", __FILE__, __func__); fflush(stderr); basically in all EBUSY places in libdiskfs, libnetfs, libfshelp - I got it from libnetfs/netfs_S_file_set_translator. That di

Re: [PATCH hurd 27/30] hurd: add intranpayload functions to all hurd types

2014-12-08 Thread David Michael
Hi, On Thu, Nov 27, 2014 at 8:19 AM, Justus Winter <4win...@informatik.uni-hamburg.de> wrote: > +#ifdef HURD_DEFAULT_PAYLOAD_TO_PORT > +#if HURD_DEFAULT_PAYLOAD_TO_PORT > +/* Any non-numeric value will fail this test. If 1 (or any number) is > + given, do not inject the default translator funct

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Samuel Thibault
Svante Signell, le Mon 08 Dec 2014 14:17:24 +0100, a écrit : > Aha, so if I had used fprintf to e.g. stderr the built hurd would boot > even with printing enabled? Good to know ;) I do put printfs to stderr in my installed translators essentially all the time, including ext2fs and such. So yes, th

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Svante Signell
On Mon, 2014-12-08 at 14:10 +0100, Samuel Thibault wrote: > Svante Signell, le Mon 08 Dec 2014 14:08:15 +0100, a écrit : > > On Mon, 2014-12-08 at 13:26 +0100, Samuel Thibault wrote: > > > Svante Signell, le Mon 08 Dec 2014 13:24:16 +0100, a écrit : > > > > what is wrong with writing to a file? > >

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Samuel Thibault
Svante Signell, le Mon 08 Dec 2014 14:08:15 +0100, a écrit : > On Mon, 2014-12-08 at 13:26 +0100, Samuel Thibault wrote: > > Svante Signell, le Mon 08 Dec 2014 13:24:16 +0100, a écrit : > > > what is wrong with writing to a file? > > > > Nothing wrong, except that you must be sure that the transla

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Svante Signell
On Mon, 2014-12-08 at 13:26 +0100, Samuel Thibault wrote: > Svante Signell, le Mon 08 Dec 2014 13:24:16 +0100, a écrit : > > what is wrong with writing to a file? > > Nothing wrong, except that you must be sure that the translator > can write to it. Making the root translator write to its own > /

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Justus Winter
Quoting Samuel Thibault (2014-12-08 13:11:22) > fprintf(stderr)+fflush *does* work, I've been using various times. I like to use error (0, 0, "didum"). It's easy to use, prints the servers name, prints a newline and does fflush. The most annoying thing about it is that it doesn't print the messa

Re: [PATCH gnumach 1/2] kern: provide notifications about new tasks

2014-12-08 Thread Justus Winter
Quoting Samuel Thibault (2014-11-21 01:50:15) > Justus Winter, le Wed 12 Nov 2014 16:49:08 +0100, a écrit : > > +kern_return_t > > +register_new_task_notification( > > + const host_t host, > > + ipc_port_t notification) > > +{ > > + if (host == HOST_NULL) > > + return KERN_I

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Samuel Thibault
BTW, one way to check what is actually getting run is using gdb: $ ps -feM | grep auth youpi 19910 19908 0:00.01 /bin/fakeauth /bin/sh -c cd "$1" || exit ; shift $ gdb /bin/fakeauth 19910 ... Reading symbols from /home/youpi/libpthread.so.0.3...(no debugging symbols found)...done. Loaded symbo

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Samuel Thibault
Svante Signell, le Mon 08 Dec 2014 13:24:16 +0100, a écrit : > what is wrong with writing to a file? Nothing wrong, except that you must be sure that the translator can write to it. Making the root translator write to its own /tmp/foobar.log is probably deemed to fail, for instance. Samuel

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Svante Signell
On Mon, 2014-12-08 at 13:11 +0100, Samuel Thibault wrote: > Svante Signell, le Mon 08 Dec 2014 13:09:28 +0100, a écrit : > > On Mon, 2014-12-08 at 12:40 +0100, Samuel Thibault wrote: > > > Svante Signell, le Mon 08 Dec 2014 12:05:13 +0100, a écrit : > > > > Which translators are restarted when inst

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Samuel Thibault
Svante Signell, le Mon 08 Dec 2014 13:09:28 +0100, a écrit : > On Mon, 2014-12-08 at 12:40 +0100, Samuel Thibault wrote: > > Svante Signell, le Mon 08 Dec 2014 12:05:13 +0100, a écrit : > > > Which translators are restarted when installing, > > > > Nothing gets restarted when you install. Only the

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Samuel Thibault
Svante Signell, le Mon 08 Dec 2014 13:04:50 +0100, a écrit : > On Mon, 2014-12-08 at 12:40 +0100, Samuel Thibault wrote: > > Svante Signell, le Mon 08 Dec 2014 12:05:13 +0100, a écrit : > > > Which translators are restarted when installing, > > > > > exec /my_chroot/DEBs/test_fakeroot-hurd/settran

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Svante Signell
On Mon, 2014-12-08 at 12:40 +0100, Samuel Thibault wrote: > Svante Signell, le Mon 08 Dec 2014 12:05:13 +0100, a écrit : > > Which translators are restarted when installing, > > Nothing gets restarted when you install. Only the newer translator that > you start will use the new implementation. >

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Svante Signell
On Mon, 2014-12-08 at 12:40 +0100, Samuel Thibault wrote: > Svante Signell, le Mon 08 Dec 2014 12:05:13 +0100, a écrit : > > Which translators are restarted when installing, > > > exec /my_chroot/DEBs/test_fakeroot-hurd/settrans --chroot \ > > /my_chroot/DEBs/test_fakeroot-hurd/fakeauth \ >

Re: Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Samuel Thibault
Svante Signell, le Mon 08 Dec 2014 12:05:13 +0100, a écrit : > Which translators are restarted when installing, Nothing gets restarted when you install. Only the newer translator that you start will use the new implementation. > and which are when rebooting? Everything gets restarted of course.

Confusing results when trying to debug fakeroot-hurd

2014-12-08 Thread Svante Signell
Hi, Attached is a partial summery trying to debug the problems with fakeroot-hurd when building glibc. Print statements to file are added to: trans/fakeroot.c libnetfs/file-set-translator.c libdiskfs/file-set-trans.c (and other files in libnetfs/, libdiskfs/, but no hits whatsoever for them). S

[PATCH gnumach] Retire procedure `old_mach_port_get_receive_status'

2014-12-08 Thread Justus Winter
Retire the compatibility RPC `old_mach_port_get_receive_status' that works like `mach_port_get_receive_status' but returns an `old_mach_port_status' object that lacks the `mps_seqno' field. Do not remove the type yet, so we do not break anyones build. The RPC stubs currently distributed with the