linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* devfs compile breaks on 2.4.20-pre2
@ 2002-08-13  4:20 John Kim
  2002-08-13  8:42 ` Adrian Bunk
  0 siblings, 1 reply; 2+ messages in thread
From: John Kim @ 2002-08-13  4:20 UTC (permalink / raw)
  To: linux-kernel; +Cc: drew


devfs is seems to be broken for 2.4.20-pre2 because include/linux/genhd.h
has been modified from 2.4.20-pre1.  I see that 'number' member has been
removed from hd_struct in include/linux/genhd.h.

Functions devfs_register_disc and devfs_register_partitions in
fs/partitions/check.c is still expecting "number" it to be there.

Unfortunately I don't know enough about devfs (or kernel in general) to
fix this.  My apologies if this mailing list wrong place to report
problems like this.


John Kim


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: devfs compile breaks on 2.4.20-pre2
  2002-08-13  4:20 devfs compile breaks on 2.4.20-pre2 John Kim
@ 2002-08-13  8:42 ` Adrian Bunk
  0 siblings, 0 replies; 2+ messages in thread
From: Adrian Bunk @ 2002-08-13  8:42 UTC (permalink / raw)
  To: John Kim; +Cc: linux-kernel, drew

On Tue, 13 Aug 2002, John Kim wrote:

> has been modified from 2.4.20-pre1.  I see that 'number' member has been
> removed from hd_struct in include/linux/genhd.h.
>
> Functions devfs_register_disc and devfs_register_partitions in
> fs/partitions/check.c is still expecting "number" it to be there.
>...


The following patch made by Christoph Hellwig fixes it:


--- linux-2.4.20-bk-20020810/include/linux/genhd.h	Sat Aug 10 14:37:16 2002
+++ linux/include/linux/genhd.h	Mon Aug 12 23:40:37 2002
@@ -62,7 +62,9 @@ struct hd_struct {
 	unsigned long start_sect;
 	unsigned long nr_sects;
 	devfs_handle_t de;              /* primary (master) devfs entry  */
-
+#ifdef CONFIG_DEVFS_FS
+	int number;
+#endif /* CONFIG_DEVFS_FS */
 #ifdef CONFIG_BLK_STATS
 	/* Performance stats: */
 	unsigned int ios_in_flight;

> John Kim

cu
Adrian

-- 

You only think this is a free country. Like the US the UK spends a lot of
time explaining its a free country because its a police state.
								Alan Cox


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2002-08-13  8:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2002-08-13  4:20 devfs compile breaks on 2.4.20-pre2 John Kim
2002-08-13  8:42 ` Adrian Bunk

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).