Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S266324AbUJAVSB (ORCPT ); Fri, 1 Oct 2004 17:18:01 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S266352AbUJAVRr (ORCPT ); Fri, 1 Oct 2004 17:17:47 -0400 Received: from stat16.steeleye.com ([209.192.50.48]:27319 "EHLO hancock.sc.steeleye.com") by vger.kernel.org with ESMTP id S266324AbUJAU6c (ORCPT ); Fri, 1 Oct 2004 16:58:32 -0400 Subject: RE: [PATCH]: megaraid 2.20.4: Fixes a data corruption bug From: James Bottomley To: "Bagalkote, Sreenivas" Cc: "Mukker, Atul" , "'linux-kernel@vger.kernel.org'" , "'linux-scsi@vger.kernel.org'" , "'bunk@fs.tum.de'" , "'Andrew Morton'" , "'Matt_Domsch@dell.com'" , "Ju, Seokmann" In-Reply-To: <0E3FA95632D6D047BA649F95DAB60E570230C985@exa-atlanta> References: <0E3FA95632D6D047BA649F95DAB60E570230C985@exa-atlanta> Content-Type: text/plain Content-Transfer-Encoding: 7bit X-Mailer: Ximian Evolution 1.0.8 (1.0.8-9) Date: 01 Oct 2004 16:58:07 -0400 Message-Id: <1096664298.1766.103.camel@mulgrave> Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1368 Lines: 35 On Fri, 2004-10-01 at 16:08, Bagalkote, Sreenivas wrote: > The submitted previous version of megaraid (2.20.3.1) had > register_ioctl32_conversion & unregister_ioctl32_conversion > defined to empty statements if CONFIG_COMPAT was _not_ > defined. I know that. However, when the empty statements were added to ioctl32.h, those had to be taken out of the megaraid driver. That's this patch in the scsi-misc-2.6 tree: [PATCH] megaraid warning fix The ioctl32 conversion registration stubs are in ioctl32.h now. Signed-off-by: Andrew Morton > But I think the preferred way was to have the occurances of > (un)register_ioctl32_conversion in the code surrounded by > #ifdef CONFIG_COMPAT ... #endif directly. In the kernel source > only register_ioctl32_conversion has these #ifdef .. #endif. The > unregister_ioctl32_conversion doesn't. The current preferred way is to use the empty definitions in linux/ioctl32.h which means there's no necessity for adding the #ifdef CONFIG_COMPAT. The correct thing is to remove the #ifdef CONFIG_COMPAT from around the register_ioctl32... part. James - 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/