>> +    NULL,                   /* fasync */
>> +    NULL,                   /* check_media_change */
>> +    NULL                    /* revalidate */
>>  };
>
>Do you really need these? I thought GCC always packed out empty structure
>entries like that automatically...

Yeah.  I put them in because hd.c actually does have a revalidate routine - it 
just turns out to be arranged slightly wrong at the moment.

And, duh, I meant to send that to linux-kernel not linux-arm.  For the ARM 
there is another problem if you've applied Russell's patch; you need something
like this too.

@@ -80,6 +73,8 @@
 static int reset;
 static int hd_error;
 
+#define SUBSECTOR(block) (CURRENT->current_nr_sectors > 0)
+
 /*
  *  This struct defines the HD's and their types.
  */

p.


unsubscribe: body of `unsubscribe linux-arm' to [EMAIL PROTECTED]

Reply via email to