From: Stephane Wirtel <stephane.wirtel@belgacom.net>
To: linux-kernel@vger.kernel.org
Subject: Re: Linux 2.4.20-pre3
Date: Sat, 17 Aug 2002 12:22:59 +0200 [thread overview]
Message-ID: <20020817102259.GA4174@debian> (raw)
In-Reply-To: <Pine.LNX.4.44.0208170117360.8089-100000@freak.distro.conectiva>
about the patch of devfs, why don't add directly this patch in the new 2.4.20-pre3
?
On Sat, Aug 17, 2002 at 01:18:17AM -0300, Marcelo Tosatti wrote:
>
>
> On Sat, 17 Aug 2002, Skidley wrote:
>
> > check.c: In function `devfs_register_disc':
> > check.c:328: structure has no member named `number'
> > check.c:329: structure has no member named `number'
> > check.c: In function `devfs_register_partitions':
> > check.c:361: structure has no member named `number'
> > make[3]: *** [check.o] Error 1
> > make[3]: Leaving directory
> > `/home/skidley/kernel/linux-2.4.20-pre3/fs/partitions'
> > make[2]: *** [first_rule] Error 2
> > make[2]: Leaving directory
> > `/home/skidley/kernel/linux-2.4.20-pre3/fs/partitions'
> > make[1]: *** [_subdir_partitions] Error 2
> > make[1]: Leaving directory `/home/skidley/kernel/linux-2.4.20-pre3/fs'
> > make: *** [_dir_fs] Error 2
>
> Yeah, I forgot to apply the fix to this one, sorry.
>
> Here it is:
>
> Subject: [PATCH] fix current BK tree compilation with devfs enabled
>
>
> Not that I care for devfs, but there was at least one report on lkml.
>
> I tried to also put the devfs_handle_t under CONFIG_DEVFS_FS, but the
> devfs wrappers require it. And yes, I'm seriously pissed that devfs
> puts wordsize objects everywhere even if not enabled.
>
>
> --- 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;
>
> -
> 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/
--
Stephane Wirtel <stephane.wirtel@belgacom.net>
Web : www.linux-mons.be "Linux Is Not UniX !!!"
next prev parent reply other threads:[~2002-08-17 10:19 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-08-17 4:59 Linux 2.4.20-pre3 Skidley
2002-08-17 4:18 ` Marcelo Tosatti
2002-08-17 10:22 ` Stephane Wirtel [this message]
-- strict thread matches above, loose matches on Subject: below --
2002-08-18 6:21 Skidley
2002-08-17 1:31 Marcelo Tosatti
2002-08-17 8:59 ` Adrian Bunk
2002-08-17 11:43 ` Mel
2001-11-02 8:29 ` Pavel Machek
2002-08-17 14:12 ` Geert Uytterhoeven
2002-08-17 14:57 ` Christoph Hellwig
2002-08-17 15:34 ` Geert Uytterhoeven
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20020817102259.GA4174@debian \
--to=stephane.wirtel@belgacom.net \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox