Re: [PATCH 1/2] Port gdbserver to GNU/Hurd

2013-09-05 Thread Yue Lu
Hi, This is the my new patch. On Tue, Sep 3, 2013 at 7:11 PM, Pedro Alves wrote: > So my idea would be, instead of adding the new files under gdbserver, > to remove the spurious differences (formatting, reordering, etc.) that > were introduced in the gdbserver copies of the files, eliminating th

Re: [PATCH 03/17] Add proc_set_init_task, make runsystem pid 1

2013-09-05 Thread Justus Winter
Quoting Justus Winter (2013-08-29 12:09:02) > Quoting Samuel Thibault (2013-08-29 01:48:10) > > I think we should keep this in the Debian package, the upstream GNU > > system will a priori still use /hurd/init as reaper. > > Well, it is not my place to question this and maintaining the patches > i

[PATCH 2/2] kern: make the parent of a task available via task_info

2013-09-05 Thread Justus Winter
XXX: this most likely subverts the security of your system. DO NOT APPLY. * include/mach/task_info.h (struct task_basic_info): Add parent_task. * kern/task.c (task_info): Include a port to parent_task. --- include/mach/task_info.h |2 ++ kern/task.c |3 +++ 2 files changed, 5

[PATCH 1/2] kern: track the parent of a task

2013-09-05 Thread Justus Winter
Align Machs model of a task more closely with that of a Unix process by tracking the parent task. XXX: Not sure if it's worth keeping a real reference to the parent since that way we prevent the parent task from being collected if it dies before the child does. * kern/task.h (struct task): Add fi

Hacking gnumach to track parental relationship of tasks

2013-09-05 Thread Justus Winter
Hi, I made two rather small and (as I thought) straight forward changes to gnumach to keep track of a tasks father task and to make this information available. Now I know handing out task ports right and left is not a wise thing to do, but I wanted to do it this way as a first step. Unfortunately

Re: Hacking gnumach to track parental relationship of tasks

2013-09-05 Thread Ludovic Courtès
Justus Winter <4win...@informatik.uni-hamburg.de> skribis: > I made two rather small and (as I thought) straight forward changes to > gnumach to keep track of a tasks father task and to make this > information available. Isn’t that what ‘proc_getpids’ is for? It feels wrong to retrofit POSIX con

[PATCH 5/7] hurd: add notification callbacks for the process management

2013-09-05 Thread Justus Winter
XXX * hurd/process_notify.defs: New file. --- hurd/process_notify.defs | 21 + 1 file changed, 21 insertions(+) create mode 100644 hurd/process_notify.defs diff --git a/hurd/process_notify.defs b/hurd/process_notify.defs new file mode 100644 index 000..7972ef7 --- /dev

Re: [PATCH 1/2] Port gdbserver to GNU/Hurd

2013-09-05 Thread Pedro Alves
On 09/05/2013 11:53 AM, Yue Lu wrote: > Hi, > > This is the my new patch. Thanks. Follows a few comments, by no means an in depth review. We'll probably need to iterate a few times. I'm counting on Thomas and others to help as well! I'm actually very excited to see gdb and gdbserver sharing a

[PATCH 7/7] hurd: add proc_notify_new_processes

2013-09-05 Thread Justus Winter
XXX * hurd/process.defs: Add proc_notify_new_processes. --- hurd/process.defs |5 + 1 file changed, 5 insertions(+) diff --git a/hurd/process.defs b/hurd/process.defs index b7e1775..37add48 100644 --- a/hurd/process.defs +++ b/hurd/process.defs @@ -383,3 +383,8 @@ routine proc_mark_impor

[PATCH 2/7] proc: Remove unused declaration of zombie_list

2013-09-05 Thread Justus Winter
* proc/proc.h (zombie_list): Remove declaration. --- proc/proc.h |2 -- 1 file changed, 2 deletions(-) diff --git a/proc/proc.h b/proc/proc.h index ed47ccc..4369abf 100644 --- a/proc/proc.h +++ b/proc/proc.h @@ -130,8 +130,6 @@ struct exc natural_t thread_state[0]; }; -struct zombie *zo

[PATCH 3/7] libihash: add HURD_IHASH_ITERATE_ITEMS macro

2013-09-05 Thread Justus Winter
Add a macro HURD_IHASH_ITERATE_ITEMS that iterates over all elements in the hash table making both the key and the value available. * libihash/ihash.h (HURD_IHASH_ITERATE_ITEMS): New macro. --- libihash/ihash.h | 17 + 1 file changed, 17 insertions(+) diff --git a/libihash/ihas

[PATCH 1/7] proc: remove unused file exc-reply.defs

2013-09-05 Thread Justus Winter
* proc/exc-reply.defs: Delete file. --- proc/exc-reply.defs | 36 1 file changed, 36 deletions(-) delete mode 100644 proc/exc-reply.defs diff --git a/proc/exc-reply.defs b/proc/exc-reply.defs deleted file mode 100644 index 8a04723..000 --- a/proc/exc-re

Re: [PATCH 1/2] Port gdbserver to GNU/Hurd

2013-09-05 Thread Thomas Schwinge
Hi! Just a very quick one; short on time. On Thu, 05 Sep 2013 20:29:43 +0100, Pedro Alves wrote: > On 09/05/2013 11:53 AM, Yue Lu wrote: > > This is the my new patch. You've received quite some positive feedback, good! :-) > Thanks. Follows a few comments, by no means an in depth review. Th

Re: [PATCH 1/2] Port gdbserver to GNU/Hurd

2013-09-05 Thread Joel Brobecker
> I'm actually very excited to see gdb and gdbserver sharing > a single target backend, even if we still have many wrinkles > in the interfaces to iron out! Mee too! -- Joel

process change notification prototype

2013-09-05 Thread Justus Winter
Hi :) this is an early patch series implementing process notifications in the proc server. [PATCH 1/7] proc: remove unused file exc-reply.defs [PATCH 2/7] proc: Remove unused declaration of zombie_list Cleanups that can be merged independently of the rest. [PATCH 3/7] libihash: add HURD_IHASH_I

[PATCH 6/7] proc: implement proc_notify_new_processes

2013-09-05 Thread Justus Winter
XXX XXX --- proc/Makefile |5 +++-- proc/main.c |1 + proc/mgt.c| 37 + proc/proc.h |6 ++ 4 files changed, 47 insertions(+), 2 deletions(-) diff --git a/proc/Makefile b/proc/Makefile index 2eed13c..4f3be81 100644 --- a/proc/Makefile

[PATCH 4/7] libnotify: add a general notification library

2013-09-05 Thread Justus Winter
XXX --- libnotify/Makefile | 28 ++ libnotify/notify.c | 152 libnotify/notify.h | 50 + 3 files changed, 230 insertions(+) create mode 100644 libnotify/Makefile create mode 100644 libnotify/notify.c create mode 1