| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <486B47A8.3090809@suse.de> Date: Wed, 02 Jul 2008 11:17:28 +0200 From: Hannes Reinecke <hare@...e.de> To: Denys Vlasenko <vda.linux@...glemail.com> Cc: James Bottomley <James.Bottomley@...senpartnership.com>, linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org, Andrew Morton <akpm@...ux-foundation.org> Subject: Re: [PATCH 0/3] aic7xxx: debloat Hi Denys, Denys Vlasenko wrote: > Hi Hannes, > > aic7xxx still contains ~30kb of dead code if pretty printing of registers > is requested. These patches deal with it. > > I did not get any feedback from you whether the approach of adding > "dont_generate_debug_code" marker would be acceptable despite > asking twice. I decided that it's a "Yes". > Well, yes. Sort of. Patches are okay in principle, only it would be best if you could hook the dont_generate_debug_code marker to the AIC79XX_DEBUG_ENABLE config option. This way anyone interested in doing sequencer debugging would still get the full sequencer code (which can be quite helpful at times). And it would be disabled for the unsuspecting user. Apart from this: well done. Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage hare@...e.de +49 911 74053 688 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Markus Rex, HRB 16746 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists