Re: [PATCH 5/5] kern: make struct kmem_cache fit into two cache lines

2014-01-02 Thread Samuel Thibault
Justus Winter, le Thu 02 Jan 2014 20:02:46 +0100, a écrit : > Previously, the size of struct kmem_cache was 136 bytes, just eight > bytes larger than 128 bytes, which is typically two cache lines on > today's CPUs. > > By reducing the size of the name field which holds a human-readable > descripti

Re: [PATCH 4/5] kern: reduce the size of struct task

2014-01-02 Thread Samuel Thibault
Justus Winter, le Thu 02 Jan 2014 20:02:45 +0100, a écrit : > * kern/task.h (struct task): Reduce the size of struct task by > 2 * sizeof boolean_t by using a bit field for the boolean flags. Ack. > --- > kern/task.h | 8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git

Re: [PATCH 3/5] vm: reduce the size of struct vm_page

2014-01-02 Thread Samuel Thibault
Justus Winter, le Thu 02 Jan 2014 20:02:44 +0100, a écrit : > Previously, the bit field left 31 bits unused. By reducing the size > of wire_count by one bit, the size of the whole struct is reduced by > four bytes. > > * vm/vm_page.h (struct vm_page): Reduce the size of wire_count to 15 > bits.

Re: [PATCH 2/5] vm: merge the two bit fields in struct vm_page

2014-01-02 Thread Samuel Thibault
Justus Winter, le Thu 02 Jan 2014 20:02:43 +0100, a écrit : > * vm/vm_page.h (struct vm_page): Merge the two bit fields. Ack. > --- > vm/vm_page.h | 10 +++--- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/vm/vm_page.h b/vm/vm_page.h > index 7392044..339d355 100644 > --

Re: [PATCH 1/5] vm: remove NS32000-specific padding from struct vm_page

2014-01-02 Thread Samuel Thibault
Justus Winter, le Thu 02 Jan 2014 20:02:42 +0100, a écrit : > Apparently, the NS32000 was a 32-bit CPU from the 1990ies. The string > "ns32000" appears nowhere else in the source. > > * vm/vm_page.h (struct vm_page): Remove NS32000-specific padding. Ack. > --- > vm/vm_page.h | 3 --- > 1 file

[PATCH 4/5] kern: reduce the size of struct task

2014-01-02 Thread Justus Winter
* kern/task.h (struct task): Reduce the size of struct task by 2 * sizeof boolean_t by using a bit field for the boolean flags. --- kern/task.h | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/kern/task.h b/kern/task.h index 9bfea57..e852033 100644 --- a/kern/task.h +++

[PATCH 5/5] kern: make struct kmem_cache fit into two cache lines

2014-01-02 Thread Justus Winter
Previously, the size of struct kmem_cache was 136 bytes, just eight bytes larger than 128 bytes, which is typically two cache lines on today's CPUs. By reducing the size of the name field which holds a human-readable description by eight bytes to 24 bytes, the struct kmem_cache can be made fit int

[PATCH 3/5] vm: reduce the size of struct vm_page

2014-01-02 Thread Justus Winter
Previously, the bit field left 31 bits unused. By reducing the size of wire_count by one bit, the size of the whole struct is reduced by four bytes. * vm/vm_page.h (struct vm_page): Reduce the size of wire_count to 15 bits. --- vm/vm_page.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 2/5] vm: merge the two bit fields in struct vm_page

2014-01-02 Thread Justus Winter
* vm/vm_page.h (struct vm_page): Merge the two bit fields. --- vm/vm_page.h | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/vm/vm_page.h b/vm/vm_page.h index 7392044..339d355 100644 --- a/vm/vm_page.h +++ b/vm/vm_page.h @@ -92,11 +92,8 @@ struct vm_page {

[PATCH 1/5] vm: remove NS32000-specific padding from struct vm_page

2014-01-02 Thread Justus Winter
Apparently, the NS32000 was a 32-bit CPU from the 1990ies. The string "ns32000" appears nowhere else in the source. * vm/vm_page.h (struct vm_page): Remove NS32000-specific padding. --- vm/vm_page.h | 3 --- 1 file changed, 3 deletions(-) diff --git a/vm/vm_page.h b/vm/vm_page.h index 4445cb0..