Re: [PATCH 8/15] misc: Make vm86 support optional

Andrew Morton
2005-11-12T05:56:13Z

Matt Mackall <mpm@selenic.com> wrote:
>
> Make vm86 support optional
>
> add/remove: 0/14 grow/shrink: 0/5 up/down: 0/-5221 (-5221)
> function old new delta
> do_simd_coprocessor_error 133 132 -1
> irqbits 4 - -4
> irqbits_lock 8 - -8
> release_thread 72 52 -20
> do_debug 212 186 -26
> do_general_protection 475 428 -47
> do_trap 196 140 -56
> release_vm86_irqs 112 - -112
> vm86_irqs 128 - -128
> sys_vm86old 146 - -146
> irq_handler 151 - -151
> mark_screen_rdonly 159 - -159
> sys_vm86 199 - -199
> handle_vm86_trap 231 - -231
> save_v86_state 339 - -339
> do_sys_vm86 379 - -379
> do_vm86_irq_handling 482 - -482
> do_int 508 - -508
> handle_vm86_fault 2225 - -2225

bix:/usr/src/25> grep '#ifdef' patches/tiny-make-vm86-support-optional.patch
+#ifdef CONFIG_VM86
+#ifdef CONFIG_VM86
+#ifdef CONFIG_VM86
+#ifdef CONFIG_VM86
+#ifdef CONFIG_VM86
+#ifdef CONFIG_VM86
+#ifdef CONFIG_VM86
+#ifdef CONFIG_VM86
+#ifdef CONFIG_VM86
+#ifdef CONFIG_VM86

This one has a rather low bytes-to-ifdefs ratio.

I bet you can get most of these benefits by stubbing out handle_vm86_*()
and friends and perhaps setting VM_MASK to zero.