From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932419AbWDMBXx (ORCPT ); Wed, 12 Apr 2006 21:23:53 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932418AbWDMBXx (ORCPT ); Wed, 12 Apr 2006 21:23:53 -0400 Received: from smtp.knology.net ([24.214.63.101]:32911 "EHLO smtp.knology.net") by vger.kernel.org with ESMTP id S932416AbWDMBXw (ORCPT ); Wed, 12 Apr 2006 21:23:52 -0400 Message-ID: <443DA830.8030209@thedillows.org> Date: Wed, 12 Apr 2006 21:24:00 -0400 From: Dave Dillow User-Agent: Mozilla Thunderbird 1.0.2 (Windows/20050317) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Ingo Oeser CC: Ingo Oeser , Denis Vlasenko , netdev@vger.kernel.org, "David S. Miller" , linux-kernel@vger.kernel.org, jgarzik@pobox.com Subject: Re: [RFD][PATCH] typhoon and core sample for folding away VLAN stuff References: <200604071628.30486.vda@ilport.com.ua> <200604111502.52302.vda@ilport.com.ua> <200604111517.37215.netdev@axxeo.de> <200604122132.46113.ioe-lkml@rameria.de> In-Reply-To: <200604122132.46113.ioe-lkml@rameria.de> Content-Type: text/plain; charset=KOI8-R; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Ingo Oeser wrote: > Dave Dillow: Is this style clean enough to have it in your driver? Though I'm not real fond of Denis's last patch, I think I prefer it's style to this, solely because it removes more code when VLANs are disabled -- you've left the spin_locks in, and have more #ifdefs. Regardless, I remain opposed to this particular instance of bloat busting. While both patches have improved in style, they remove a useful feature and make the code less clean, for no net gain. > This kind of changes are important, because bloat creeps in byte by byte > of unused features. So I really appreciate your work here Denis. On SMP FC4, typhoon.ko has a text size of 68330, so you need to cut 2794 bytes to see an actual difference in memory usage for a module. Non-SMP it is 67741, so there you only need to cut 2205 bytes to get a win. Every byte counts, except when it doesn't.