netdev - Re: [PATCH 2.6.24]S2io: Enhance device error/alarm handling

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <46C137D9.7050504@garzik.org>
Date:	Tue, 14 Aug 2007 01:04:25 -0400
From:	Jeff Garzik <jeff@...zik.org>
To:	ram.vepa@...erion.com
CC:	netdev@...r.kernel.org, support@...erion.com
Subject: Re: [PATCH 2.6.24]S2io: Enhance device error/alarm handling

Ramkrishna Vepa wrote:
> - Added support to poll for entire set of device errors and alarms.
> - Optimized interrupt routine fast path.
> - Removed the unused variable, intr_type, in device private structure.
> 
> Signed-off-by: Santosh Rastapur <santosh.rastapur@...erion.com>
> Signed-off-by: Ramkrishna Vepa <ram.vepa@...erion.com>

please split this patch up into multiple patches.

In general it's OK to combine several small, minor changes together, but 
this patch was pretty big.  at that point, you need to start looking 
into splitting up patches into separate logical changes (intr_type clean 
in its own patch, irq handler optimization in its own patch, etc.).

More patches means that each patch is smaller, making it easier to 
review.  It also means that automated tools such as git-bisect can 
easily identify which change created a regression, should one appear.

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html