RE: [PATCH]: megaraid 2.20.4: Fixes a data corruption bug

[Bagalkote, Sreenivas]

Without CONFIG_COMPAT around them, I get "unresolved symbol"
for (un)register_ioctl32_conversion while loading the module.

Thanks,
Sreenivas

>-----Original Message-----
>From: James Bottomley [mailto:James.Bottomley@xxxxxxxxxxxx]
>Sent: Friday, October 01, 2004 4:24 PM
>To: Bagalkote, Sreenivas
>Cc: Mukker, Atul; 'linux-kernel@xxxxxxxxxxxxxxx';
>'linux-scsi@xxxxxxxxxxxxxxx'; 'bunk@xxxxxxxxx'; 'Andrew Morton';
>'Matt_Domsch@xxxxxxxx'; Ju, Seokmann
>Subject: RE: [PATCH]: megaraid 2.20.4: Fixes a data corruption bug
>
>
>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.
>
>
>> 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.
>
>Actually, because of the way linux/ioctl32 defines these, the #ifdef
>CONFIG_COMPAT is unnecessary even around register_ioctl32_...
>
>James
>
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/