Re: Contributing to GNU Hurd

Hi! On Thu, 5 Dec 2013 13:16:31 -0800, kishore kumar wrote: > This is Kishore. I am very much interested in Operating Systems. I have a > bit experience in both Linux and Solaris. Please let me know how can I > start contributing to GNU Hurd project. Welcome! There are some pointers given on <

Re: RFC: [PATCH] SCM_CREDS support

On Thu, 2013-10-24 at 18:24 +0200, Svante Signell wrote: > On Thu, 2013-10-24 at 18:15 +0200, Samuel Thibault wrote: > > Svante Signell, le Thu 24 Oct 2013 18:14:19 +0200, a écrit : > > > Sure, but again, what is the relation between that and having both > > SCM_RIGHT and SCM_CREDS in the same mes

[PATCH 35/66] i386/i386at/kd.c: simplify code

Variable char_byte_width is always zero. Use this to simplify the code. * i386/i386at/kd.c (char_byte_width): Remove variable. (j): Remove variable. (mask): Remove assignment. (from): Simplify assignment. Remove for loop. (j): Remove variable. Remove for loop. (byte): Remove variable. Remove for l

[PATCH 61/66] kern/refcount.h: remove unnecessary include

File kern/refcount.h includes itself. Remove this include. * kern/refcount.h: Don't include kern/refcount.h. --- kern/refcount.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/kern/refcount.h b/kern/refcount.h index 7fd6cdf..58e1cc4 100644 --- a/kern/refcount.h +++ b/kern/refcount.h @@ -29

[PATCH 39/66] i386/i386at/kd.c: use boolean instead of an unsigned int

* i386/i386at/kd.c (kd_bellstate): Use boolean instead of an unsigned int. --- i386/i386at/kd.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/i386/i386at/kd.c b/i386/i386at/kd.c index 1425f74..1bf6fe1 100644 --- a/i386/i386at/kd.c +++ b/i386/i386at/kd.c @@ -1215,7 +121

[PATCH 16/66] device/kmsg.c: use boolean instead of an int

* device/kmsg.c (kmsg_in_use): Use boolean instead of an int. --- device/kmsg.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/device/kmsg.c b/device/kmsg.c index c8bd897..0e18725 100644 --- a/device/kmsg.c +++ b/device/kmsg.c @@ -42,7 +42,7 @@ static int kmsg_read_of

[PATCH 37/66] i386/i386at/kd.c: simplify code

Variable font_byte_width is always zero. Use this to simplify the code. * i386/i386at/kd.c (font_byte_width): Remove variable. (i, from): Remove variables. Remove for loop. * i386/i386at/kdsoft.h (font_byte_width): Remove declaration. --- i386/i386at/kd.c | 7 --- i386/i386at/kdsoft.h |

[PATCH 66/66] kern/processor.h: remove ifdef

This code is used even if KERNEL is not defined. Remove the ifdef. * kern/processor.h [KERNEL]: Remove ifdef. --- kern/processor.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/kern/processor.h b/kern/processor.h index 7bd29cf..2000192 100644 --- a/kern/processor.h +++ b/kern/processor.h

[PATCH 17/66] device/kmsg.c: use boolean instead of an int

* device/kmsg.c (kmsg_init_done): Use boolean instead of an int. --- device/kmsg.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/device/kmsg.c b/device/kmsg.c index 0e18725..7034bfc 100644 --- a/device/kmsg.c +++ b/device/kmsg.c @@ -46,7 +46,7 @@ static boolean_t kmsg_in

Contributing to GNU Hurd

Hi, This is Kishore. I am very much interested in Operating Systems. I have a bit experience in both Linux and Solaris. Please let me know how can I start contributing to GNU Hurd project. Thanks. Best, Kishore

[PATCH 38/66] i386/i386at/kd.c: use boolean instead of an int

* i386/i386at/kd.c (mouse_in_use): Use boolean instead of an int. Remove duplicate declaration. --- i386/i386at/kd.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/i386/i386at/kd.c b/i386/i386at/kd.c index 1a2d9fd..3299e70 100644 --- a/i386/i386at/kd.c +++ b/i386/i386at

[PATCH 43/66] i386/i386at/rtc.c: use boolean instead of an int

* i386/i386at/rtc.c (first_rtcopen_ever): Use boolean instead of an int. --- i386/i386at/rtc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/i386/i386at/rtc.c b/i386/i386at/rtc.c index 3112830..2115f23 100644 --- a/i386/i386at/rtc.c +++ b/i386/i386at/rtc.c @@ -53,7 +53

[PATCH 08/66] ddb/db_output.h: add comment

Add comment saying that kdbprintf is an alternate name for db_printf(). * ddb/db_output.h: Add comment. --- ddb/db_output.h | 1 + 1 file changed, 1 insertion(+) diff --git a/ddb/db_output.h b/ddb/db_output.h index 8d74f88..35c39a6 100644 --- a/ddb/db_output.h +++ b/ddb/db_output.h @@ -35,6 +35

[PATCH 62/66] kern/syscall_sw.c: use boolean instead of an int

* kern/syscall_sw.c (kern_invalid_debug): Use boolean instead of an int. --- kern/syscall_sw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kern/syscall_sw.c b/kern/syscall_sw.c index 607d843..856b12f 100644 --- a/kern/syscall_sw.c +++ b/kern/syscall_sw.c @@ -56,7 +56,7 @@

[PATCH 11/66] device/cirbuf.c: use boolean instead of an int

* device/cirbuf.c (cb_check_enable): Use boolean instead of an int. --- device/cirbuf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/device/cirbuf.c b/device/cirbuf.c index 7af2d7a..292b888 100644 --- a/device/cirbuf.c +++ b/device/cirbuf.c @@ -42,7 +42,7 @@ /* if c_cl ==

[PATCH 41/66] i386/i386at/kd.h: remove ifdef

Variable kb_mode is used even if KERNEL is not defined. Remove ifdef. * i386/i386at/kd.h [KERNEL]: Remove ifdef. --- i386/i386at/kd.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/i386/i386at/kd.h b/i386/i386at/kd.h index 4d1ab7f..ed11cbb 100644 --- a/i386/i386at/kd.h +++ b/i386/i386at/kd

[PATCH 13/66] device/dev_lookup.c: remove unused lock

* device/dev_lookup.c (dev_port_lock): Remove unused lock. --- device/dev_lookup.c | 4 1 file changed, 4 deletions(-) diff --git a/device/dev_lookup.c b/device/dev_lookup.c index 01317b9..0c2198e 100644 --- a/device/dev_lookup.c +++ b/device/dev_lookup.c @@ -242,8 +242,6 @@ mach_device_dea

[PATCH 21/66] i386/i386/db_interface.c: remove dead assignment

* i386/i386/db_interface.c (i386_nested_saved_state): Remove variable. (i386_nested_saved_state): Remove dead assignment. --- i386/i386/db_interface.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/i386/i386/db_interface.c b/i386/i386/db_interface.c index 23a66dd..ac75e03 100644 --- a/i386/

[PATCH 57/66] kern/lock_mon.c: remove dead assignments

* kern/lock_mon.c (scurval, ssum, sli): Remove variables. (scurval, ssum, sli): Remove dead assignments. --- kern/lock_mon.c | 5 - 1 file changed, 5 deletions(-) diff --git a/kern/lock_mon.c b/kern/lock_mon.c index d126c69..cc8c36c 100644 --- a/kern/lock_mon.c +++ b/kern/lock_mon.c @@ -172,

[PATCH 63/66] vm/vm_page.h: remove unused variables

* vm/vm_page.h (first_phys_addr, last_phys_addr): Remove unused variables. --- vm/vm_page.h | 5 - 1 file changed, 5 deletions(-) diff --git a/vm/vm_page.h b/vm/vm_page.h index 4536d1c..f5681d7 100644 --- a/vm/vm_page.h +++ b/vm/vm_page.h @@ -156,11 +156,6 @@ extern queue_head_t vm_page_

[PATCH 36/66] i386/i386at/kd.c: simplify code

Variable fb_byte_width is always zero. Use this to simplify the code. * i386/i386at/kd.c (fb_byte_width): Remove variable. (to): Remove variable. (bit2fbptr): Remove assignment. (to): Likewise. (to, fp, i): Remove variables. (bit2fbptr): Remove assignments. Remove for loop. (count): Remove assignm

[PATCH 65/66] kern/act.h: remove unnecessary include

File kern/act.h includes itself. Remove this include. * kern/act.h: Don't include kern/act.h. --- kern/act.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/kern/act.h b/kern/act.h index 9175211..5ae4b9d 100644 --- a/kern/act.h +++ b/kern/act.h @@ -40,8 +40,6 @@ #include #include -#in

[PATCH 12/66] device/cons.h: remove ifdef

Field constab is used even if KERNEL is not defined. Remove the ifdef. * device/cons.h [KERNEL]: Remove ifdef. --- device/cons.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/device/cons.h b/device/cons.h index 8ac796c..d569a8f 100644 --- a/device/cons.h +++ b/device/cons.h @@ -43,9 +43,

[PATCH 64/66] Cleanup of the prototypes

* device/cirbuf.h (nqdb): Remove prototype without definition. * device/conf.h (dev_set_indirect): Likewise. * kern/boot_script.h (boot_script_read_file): Likewise. * kern/eventcount.h (evc_notify_abort): Remove duplicate prototype. * kern/thread.h (thread_set_own_priority): Likewise. * kern/threa

[PATCH 07/66] ddb/db_output.c: remove duplicate function

Function kdbprintf() and db_printf() are the same function. Remove kdbprintf() and define kdbprintf to db_printf. * ddb/db_output.c (kdbprintf): Remove function. * ddb/db_output.h: Define kdbprintf to db_printf. (kdbprintf): Remove prototype. --- ddb/db_output.c | 12 ddb/db_output.

[PATCH 23/66] i386/i386/ipl.h: remove ifdef and add ifndef

This code is used even if KERNEL is not defined. Remove the ifdef. * i386/i386/ipl.h [KERNEL]: Remove ifdef. [_I386_IPL_H_]: Add ifndef. --- i386/i386/ipl.h | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/i386/i386/ipl.h b/i386/i386/ipl.h index 8f729e1..7503ac3 100644 -

[PATCH 56/66] kern/lock_mon.c: use spl_t instead of an int

Variable curr_ipl is in other files declared as spl_t. * kern/lock_mon.c (curr_ipl): Use spl_t instead of an int. --- kern/lock_mon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kern/lock_mon.c b/kern/lock_mon.c index 0f86735..d126c69 100644 --- a/kern/lock_mon.c +++ b/ke

[PATCH 58/66] kern/pc_sample.c: remove unused variables

* kern/pc_sample.c (pc_sampling_enabled, pc_sampling_lock): Remove unused variables. --- kern/pc_sample.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/kern/pc_sample.c b/kern/pc_sample.c index 05c08b8..cdf8e95 100644 --- a/kern/pc_sample.c +++ b/kern/pc_sample.c @@ -43,9 +43,6 @@ typ

[PATCH 47/66] ipc/ipc_init.c: qualify constant with const

* ipc/ipc_init.c (ipc_kernel_map_size): Qualify constant with const. --- ipc/ipc_init.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ipc/ipc_init.c b/ipc/ipc_init.c index ca7e791..a7e9ec1 100644 --- a/ipc/ipc_init.c +++ b/ipc/ipc_init.c @@ -54,7 +54,7 @@ static struct vm

[PATCH 52/66] kern/debug.c: initialize the lock

* kern/debug.c (Assert_print_lock): Initialize the lock. --- kern/debug.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/kern/debug.c b/kern/debug.c index 33e64f3..bdc87e2 100644 --- a/kern/debug.c +++ b/kern/debug.c @@ -45,7 +45,7 @@ extern int db_breakpoints_inserted; #e

[PATCH 51/66] kern/act.h: remove prototype without definition

* kern/act.h (act_create_kernel): Remove prototype without definition. --- kern/act.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/kern/act.h b/kern/act.h index cddf6c7..9175211 100644 --- a/kern/act.h +++ b/kern/act.h @@ -174,9 +174,7 @@ voidact_init(void); /* Exported t

[PATCH 50/66] kern/act.c, act.h: comment unused variable

* kern/act.c (null_act): Comment unused variable. * kern/act.h (null_act): Likewise. --- kern/act.c | 2 +- kern/act.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/kern/act.c b/kern/act.c index 321ff98..61b982e 100644 --- a/kern/act.c +++ b/kern/act.c @@ -61,7 +61,7 @@ s

[PATCH 55/66] kern/lock_mon.c: remove unused variable

* kern/lock_mon.c (default_lock_stack): Remove unused variable. --- kern/lock_mon.c | 1 - 1 file changed, 1 deletion(-) diff --git a/kern/lock_mon.c b/kern/lock_mon.c index 80c50b5..0f86735 100644 --- a/kern/lock_mon.c +++ b/kern/lock_mon.c @@ -83,7 +83,6 @@ struct lock_info_bucket { struct

[PATCH 46/66] i386/intel/pmap.c: remove unused variable

* i386/intel/pmap.c (end): Remove unused variable. --- i386/intel/pmap.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/i386/intel/pmap.c b/i386/intel/pmap.c index 69a58ed..d3e77ed 100644 --- a/i386/intel/pmap.c +++ b/i386/intel/pmap.c @@ -395,8 +395,6 @@ int ptes_per_vm_page;

[PATCH 40/66] i386/i386at/kd.h: remove ifdef

Variable key_map is used even if KERNEL is not defined. Remove ifdef. * i386/i386at/kd.h [KERNEL]: Remove ifdef. --- i386/i386at/kd.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/i386/i386at/kd.h b/i386/i386at/kd.h index b29a5bf..4d1ab7f 100644 --- a/i386/i386at/kd.h +++ b/i386/i386at/kd

[PATCH 32/66] i386/i386at/com.c: remove dead assignment

* i386/i386at/com.c (comtimer_state): Remove variable. (comtimer_state): Remove dead assignment. --- i386/i386at/com.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/i386/i386at/com.c b/i386/i386at/com.c index 62aa85c..2df6682 100644 --- a/i386/i386at/com.c +++ b/i386/i386at/com.c @@ -64,7

[PATCH 42/66] i386/i386at/lpr.c: remove uninitialized variable

Variable lpr_alive is never initialized. I suggest a removal of the code that deals with this variable. But, on the other hand, maybe the variable should be initialized to all zero. It's not so clear from the code. * i386/i386at/lpr.c (lpr_alive): Remove variable. Remove the if. --- i386/i386a

[PATCH 44/66] i386/i386at/rtc.c: remove unused variable

* i386/i386at/rtc.c (tz): Remove unused variable. --- i386/i386at/rtc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/i386/i386at/rtc.c b/i386/i386at/rtc.c index 2115f23..98f157f 100644 --- a/i386/i386at/rtc.c +++ b/i386/i386at/rtc.c @@ -103,7 +103,6 @@ unsigned char *regs; extern

[PATCH 45/66] i386/intel/pmap.c: use boolean instead of an int

* i386/intel/pmap.c (pmap_debug): Use boolean instead of an int. --- i386/intel/pmap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/i386/intel/pmap.c b/i386/intel/pmap.c index 3e2e6ba..69a58ed 100644 --- a/i386/intel/pmap.c +++ b/i386/intel/pmap.c @@ -384,7 +384,7 @@ pmap_t

[PATCH 60/66] kern/rbtree.h: remove unnecessary include

File kern/rbtree.h includes itself. Remove this include. * kern/rbtree.h: Don't include kern/rbtree.h. --- kern/rbtree.h | 1 - 1 file changed, 1 deletion(-) diff --git a/kern/rbtree.h b/kern/rbtree.h index 5a65d1e..f577f7e 100644 --- a/kern/rbtree.h +++ b/kern/rbtree.h @@ -32,7 +32,6 @@ #incl

[PATCH 33/66] i386/i386at/immc.c, kd.c: use boolean instead of an int

* i386/i386at/immc.c (immediate_console_enable): Use boolean instead of an int. * i386/i386at/kd.c (immediate_console_enable): Likewise. --- i386/i386at/immc.c | 2 +- i386/i386at/kd.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/i386/i386at/immc.c b/i386/i386at/immc

[PATCH 28/66] i386/i386/trap.c: remove unused variable

* i386/i386/trap.c (brb): Remove unused variable. --- i386/i386/trap.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/i386/i386/trap.c b/i386/i386/trap.c index 89d1f3d..ffb23bb 100644 --- a/i386/i386/trap.c +++ b/i386/i386/trap.c @@ -151,9 +151,6 @@ char *trap_name(unsigned int trapnum)

[PATCH 54/66] kern/debug.c: remove unused variable

* kern/debug.c (__stack_chk_guard): Remove unused variable. --- kern/debug.c | 7 --- 1 file changed, 7 deletions(-) diff --git a/kern/debug.c b/kern/debug.c index 8cd3f02..d3ae4fa 100644 --- a/kern/debug.c +++ b/kern/debug.c @@ -205,13 +205,6 @@ log(int level, const char *fmt, ...)

[PATCH 31/66] i386/i386at/com.c: simplify code

Variable comtimer_active is used uninitialized. What's the chance of this variable getting the value FALSE? I suggest a removal of the code that deals with this variable. But, on the other hand, this may not be a good solution. Maybe the variable should be initialized to FALSE, meaning that the co

[PATCH 48/66] ipc/mach_port.c: use boolean instead of an int

* ipc/mach_port.c (mach_port_deallocate_debug): Use boolean instead of an int. --- ipc/mach_port.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ipc/mach_port.c b/ipc/mach_port.c index 46cb4de..e1d9419 100644 --- a/ipc/mach_port.c +++ b/ipc/mach_port.c @@ -555,7 +555,7 @@ ma

[PATCH 24/66] i386/i386/mp_desc.c: remove unused variable

* i386/i386/mp_desc.c (avail_start): Remove unused variable. --- i386/i386/mp_desc.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/i386/i386/mp_desc.c b/i386/i386/mp_desc.c index efdc3ee..5ede8f1 100644 --- a/i386/i386/mp_desc.c +++ b/i386/i386/mp_desc.c @@ -65,12 +65,6 @@ char

[PATCH 27/66] i386/i386/pit.c: remove unused variables

* i386/i386/pit.c (pitctr1_port, pitctr2_port): Remove unused variables. --- i386/i386/pit.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/i386/i386/pit.c b/i386/i386/pit.c index 4f156d8..85929e9 100644 --- a/i386/i386/pit.c +++ b/i386/i386/pit.c @@ -57,8 +57,6 @@ WITH THE USE OR PERFORMA

[PATCH 26/66] i386/i386/pic.c: remove unused variables

* i386/i386/pic.c (nintr, npics): Remove unused variables. --- i386/i386/pic.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/i386/i386/pic.c b/i386/i386/pic.c index 50ca150..75cb835 100644 --- a/i386/i386/pic.c +++ b/i386/i386/pic.c @@ -62,9 +62,6 @@ int curr_pic_mask; intiunit[N

[PATCH 20/66] i386/i386/db_interface.c: remove dead assignment

* i386/i386/db_interface.c (i386_last_saved_statep): Remove variable. (i386_last_saved_statep): Remove dead assignment. --- i386/i386/db_interface.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/i386/i386/db_interface.c b/i386/i386/db_interface.c index a59ed37..23a66dd 100644 --- a/i386/i3

[PATCH 29/66] i386/i386/trap.c: remove unused variables

* i386/i386/trap.c (v86_assist_on, v86_unsafe_ok, v86_do_sti_cli, v86_do_sti_immediate, cli_count, sti_count): Remove unused variables. --- i386/i386/trap.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/i386/i386/trap.c b/i386/i386/trap.c index ffb23bb..239c853 100644 --- a/i38

[PATCH 53/66] kern/debug.c: initialize panicstr

Variable that keeps the panic string. Initialize to NULL, meaning that the initial condition is that there is no panic string. * kern/debug.c (panicstr): Initialize to NULL. --- kern/debug.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kern/debug.c b/kern/debug.c index bdc

[PATCH 25/66] i386/i386/mp_desc.c: remove dead assignment

* i386/i386/mp_desc.c (mp_desc_table): Remove variable. (mp_desc_table): Remove dead assignment. * i386/i386/mp_desc.h (mp_desc_table): Remove declaration. --- i386/i386/mp_desc.c | 6 -- i386/i386/mp_desc.h | 5 - 2 files changed, 11 deletions(-) diff --git a/i386/i386/mp_desc.c b/i386/

[PATCH 49/66] ipc/port.h: update comment

This is ipc/port.h. Not ipc/ipc_port.h. * ipc/port.h: Update comment. --- ipc/port.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ipc/port.h b/ipc/port.h index 6e9f77b..d6fb59b 100644 --- a/ipc/port.h +++ b/ipc/port.h @@ -29,7 +29,7 @@ /* */ /* - * File: ipc/ipc_

[PATCH 59/66] kern/priority.c: update comment

This is priority.c. Not clock_prim.c. * kern/priority.c: Update comment. --- kern/priority.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kern/priority.c b/kern/priority.c index ddb8fd6..d9ded85 100644 --- a/kern/priority.c +++ b/kern/priority.c @@ -27,7 +27,7 @@ * the r

[PATCH 34/66] i386/i386at/model_dep.c, kd.c: use boolean instead of an int

* i386/i386at/kd.c (rebootflag): Use boolean instead of an int. Remove duplicate variable declaration. * i386/i386at/model_dep.c (rebootflag): Use boolean instead of an int. --- i386/i386at/kd.c| 3 +-- i386/i386at/model_dep.c | 4 ++-- 2 files changed, 3 insertions(+), 4 deletions(-) di

[PATCH 06/66] ddb/db_output.c: remove call to nonexistent db_printf_enter()

* ddb/db_output.c [db_printf_enter] (db_printf_enter): Remove call to nonexistent function. --- ddb/db_output.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/ddb/db_output.c b/ddb/db_output.c index c08db6f..268de69 100644 --- a/ddb/db_output.c +++ b/ddb/db_output.c @@ -210,9 +210,6 @@ db

[PATCH 19/66] i386/i386/db_interface.c: use boolean instead of an int

* i386/i386/db_interface.c (kernel_dr): Use boolean instead of an int. --- i386/i386/db_interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/i386/i386/db_interface.c b/i386/i386/db_interface.c index a8ac52a..a59ed37 100644 --- a/i386/i386/db_interface.c +++ b/i386

[PATCH 18/66] device/tty.h, chario.h: move prototype

Now that we have device/chario.h move the chario_init() prototype from device/tty.h to device/chario.h. * device/chario.h (chario_init): Add prototype. * device/device_init.c: Include device/chario.h. * device/tty.h (chario_init): Remove prototype. --- device/chario.h | 2 ++ device/device_

[PATCH 22/66] i386/i386/db_interface.c: remove dead assignment

* i386/i386/db_interface.c (i386_last_kdb_sp): Remove variable. (i386_last_kdb_sp): Remove dead assignment. --- i386/i386/db_interface.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/i386/i386/db_interface.c b/i386/i386/db_interface.c index ac75e03..568c855 100644 --- a/i386/i386/db_inter

[PATCH 15/66] device/if_ether.h: remove ifdef

Function ether_sprintf() is used even if KERNEL is not defined. Avoid implicit declaration in that case by removing the ifdef. * device/if_ether.h [KERNEL]: Remove ifdef. --- device/if_ether.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/device/if_ether.h b/device/if_ether.h index e368fb

[PATCH 10/66] device/chario.c: use boolean instead of an int

* device/chario.c (pdma_default): Use boolean instead of an int. --- device/chario.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/device/chario.c b/device/chario.c index 387428a..8d68857 100644 --- a/device/chario.c +++ b/device/chario.c @@ -82,7 +82,7 @@ struct ldisc_switc

[PATCH 30/66] i386/i386/user_ldt.c: remove unused variable

* i386/i386/user_ldt.c (acc_type): Remove unused variable. --- i386/i386/user_ldt.c | 12 1 file changed, 12 deletions(-) diff --git a/i386/i386/user_ldt.c b/i386/i386/user_ldt.c index 5c3d323..07be0a0 100644 --- a/i386/i386/user_ldt.c +++ b/i386/i386/user_ldt.c @@ -43,18 +43,6 @@

[PATCH 05/66] ddb/db_command.h: remove duplicate variable declaration

* ddb/db_command.h (db_recover): Remove duplicate variable declaration. --- ddb/db_command.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/ddb/db_command.h b/ddb/db_command.h index 2517a90..be1431e 100644 --- a/ddb/db_command.h +++ b/ddb/db_command.h @@ -53,8 +53,6 @@ extern db_addr_t

[PATCH 09/66] device/blkio.c: remove function

Function block_io_mmap() is never a dev op, therefore it is unused and it can be written again easily if needed. I suggest a removal of this function. * device/blkio.c (block_io_mmap): Remove function. * device/dev_pager.c (block_io_mmap): Remove forward declaration. Remove the code in the case b

[PATCH 14/66] device/if_ether.h: remove unused variable

* device/if_ether.h (etherbroadcastaddr): Remove unused variable. * device/subrs.c (etherbroadcastaddr): Remove initialization. --- device/if_ether.h | 2 -- device/subrs.c| 5 - 2 files changed, 7 deletions(-) diff --git a/device/if_ether.h b/device/if_ether.h index dbdd818..e368fbf 10

[PATCH 03/66] chips/busses.h: remove ifdef

This code is used even if KERNEL is not defined. Remove the ifdef. * chips/busses.h [KERNEL]: Remove ifdef. --- chips/busses.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/chips/busses.h b/chips/busses.h index 49c0e44..aebf04c 100644 --- a/chips/busses.h +++ b/chips/busses.h @@ -140,7 +

[PATCH 04/66] ddb/db_command.c: remove forward declaration

* ddb/db_command.c (db_skip_to_eol): Remove forward declaration. --- ddb/db_command.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/ddb/db_command.c b/ddb/db_command.c index 1299cfa..c3e1962 100644 --- a/ddb/db_command.c +++ b/ddb/db_command.c @@ -297,8 +297,6 @@ db_command_list(last_cmdp,

[PATCH 02/66] i386/i386/setjmp.h: remove ifdef

Function _longjmp() is used even if __GNUC__ is not defined. Avoid implicit declaration in that case by removing the ifdef. * i386/i386/setjmp.h [__GNUC__] (_longjmp): Remove ifdef. --- i386/i386/setjmp.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/i386/i386/setjmp.h b/i386/i386/setjmp.

[PATCH 01/66] kern/machine.c: remove __volatile__

Shutdown can be optimised. Remove __volatile__. * kern/machine.c [__GNUC__] (processor_doshutdown): Remove volatile function qualifier. --- kern/machine.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/kern/machine.c b/kern/machine.c index 3dadeb5..8a33327 100644 --- a/kern/machine.c +++

Regression: truncated I/O, probably after having spawned a lot of processes

Hi! I have not yet gotten a chance to catch up with the last months worth of Hurd development (but did notice that the "screen logout issue" is fixed, hooray!), so please bear with me if this has already been discussed. Said months of Hurd development have just found their way into my Hurd system