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

[Andrew Morton]

Matt Mackall <mpm@xxxxxxxxxxx> 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.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/