Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762271AbXKHSJm (ORCPT ); Thu, 8 Nov 2007 13:09:42 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1760887AbXKHSJd (ORCPT ); Thu, 8 Nov 2007 13:09:33 -0500 Received: from mail-gw3.adaptec.com ([162.62.93.58]:57480 "EHLO mail-gw3.adaptec.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760490AbXKHSJb convert rfc822-to-8bit (ORCPT ); Thu, 8 Nov 2007 13:09:31 -0500 X-MimeOLE: Produced By Microsoft Exchange V6.5 Content-class: urn:content-classes:message MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT Subject: RE: [PATCH 1/1] aacraid: don't assign cpu_to_le32(int) to u8 Date: Thu, 8 Nov 2007 13:09:29 -0500 Message-ID: In-Reply-To: <20071108172747.GA4054@infradead.org> X-MS-Has-Attach: X-MS-TNEF-Correlator: Thread-Topic: [PATCH 1/1] aacraid: don't assign cpu_to_le32(int) to u8 Thread-Index: AcgiLLNkBuWnu8zJRLawS+fD74g9IwABOfAw References: <20071107173307.GB25560@infradead.org> <20071108172747.GA4054@infradead.org> From: "Salyzyn, Mark" To: "Christoph Hellwig" Cc: "Andreas Schwab" , "Stephen Rothwell" , , "LKML" Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1858 Lines: 45 Resounding ACK. I just finished *exactly* the same set of changes, composed the patch and was about to hit send when this one came over the wire from you! There was absolutely no differences between our patches (save for the fact I did not place the AIF ones in as they are already in the queue, one is already on -mm). I am going to return to this at some future date and figure out the problems surrounding the context imbalances that are present, making code that determines which context it is called from (sysfs, error recovery or from the background thread) and plays with the various locks confuses sparse. Rewriting so that the contexts are less programmatic is in order... Sincerely -- Mark Salyzyn > -----Original Message----- > From: Christoph Hellwig [mailto:hch@infradead.org] > Sent: Thursday, November 08, 2007 12:28 PM > To: Salyzyn, Mark > Cc: Christoph Hellwig; Andreas Schwab; Stephen Rothwell; > linux-scsi@vger.kernel.org; LKML > Subject: Re: [PATCH 1/1] aacraid: don't assign cpu_to_le32(int) to u8 > > On Wed, Nov 07, 2007 at 01:51:44PM -0500, Salyzyn, Mark wrote: > > Christoph Hellwig [mailto:hch@infradead.org] sez: > > > Did anyone run the driver through sparse to see if we have > > > more issues like this? > > > > There are some warnings from sparse, none like this one. I will deal > > with the warnings ... > > Actually there are a lot of endianess warnings, fortunately > most of them > harmless. The patch below fixes all of them up (including the ones in > the patch I replied to), except for aac_init_adapter which is > really odd > and I don't know what to do. > - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/