Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753024AbYKMJik (ORCPT ); Thu, 13 Nov 2008 04:38:40 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751626AbYKMJi3 (ORCPT ); Thu, 13 Nov 2008 04:38:29 -0500 Received: from gw-ca.panasas.com ([66.104.249.162]:23689 "EHLO laguna.int.panasas.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750728AbYKMJi2 (ORCPT ); Thu, 13 Nov 2008 04:38:28 -0500 Message-ID: <491BF5A3.9070700@panasas.com> Date: Thu, 13 Nov 2008 11:38:43 +0200 From: Boaz Harrosh User-Agent: Thunderbird/3.0a2 (X11; 2008072418) MIME-Version: 1.0 To: Randy Dunlap CC: James Bottomley , Andrew Morton , =?UTF-8?B?SsO2cm4gRW5nZWw=?= , open-osd development , Mike Christie , FUJITA Tomonori , Jeff Garzik , linux-scsi , Sami.Iren@seagate.com, linux-kernel Subject: Re: [PATCH 03/18 ver3] libosd: OSDv1 Headers References: <491073BB.4000900@panasas.com> <1225817046-5946-1-git-send-email-bharrosh@panasas.com> <4916F934.4090205@panasas.com> <20081110092940.57b4c169.randy.dunlap@oracle.com> <491AD697.9030305@panasas.com> <20081112105945.f4ec0089.randy.dunlap@oracle.com> In-Reply-To: <20081112105945.f4ec0089.randy.dunlap@oracle.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 13 Nov 2008 09:37:05.0078 (UTC) FILETIME=[63911D60:01C94573] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 663 Lines: 33 Randy Dunlap wrote: > > ugh > >> + } __packed attrs_cdb; >> +/*52*/ u8 get_set_attributes_parameters[80-52]; > > ugh > > > more of the same > > > ditto. nice comment ;) > > > > Thanks for the fixes. > Yes Yes that was version 3 where I was asking if I might convince you for its merits. As I said in that reply I will change it. Just did not have the chance to send out version 4 out yet. Thanks Boaz -- 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/