From: Eric Dumazet <[EMAIL PROTECTED]> Date: Tue, 01 Jan 2008 16:33:33 +0100
> We use a percpu variable named flow_hash_info, which holds 12 bytes. > > It is currently marked as ____cacheline_aligned, which makes linker > skip space to properly align this variable. > > Before : > c065cc90 D per_cpu__softnet_data > c065cd00 d per_cpu__flow_tables > <Here, hole of 124 bytes> > c065cd80 d per_cpu__flow_hash_info > <Here, hole of 116 bytes> > c065ce00 d per_cpu__flow_flush_tasklets > c065ce14 d per_cpu__rt_cache_stat > > > This alignement is quite unproductive, and removing it reduces the > size of percpu data (by 240 bytes on my x86 machine), and improves > performance (flow_tables & flow_hash_info can share a single cache > line) > > After patch : > c065cc04 D per_cpu__softnet_data > c065cc4c d per_cpu__flow_tables > c065cc50 d per_cpu__flow_hash_info > c065cc5c d per_cpu__flow_flush_tasklets > c065cc70 d per_cpu__rt_cache_stat > > Signed-off-by: Eric Dumazet <[EMAIL PROTECTED]> Also applied, thanks a lot for these patches Eric. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
