Re: test7-pre7 compile error [reiserfs]

From: Paavo Nikula (pnikula@sunpoint.net)
Date: Wed Aug 23 2000 - 14:29:28 EST


Didn't work with reiserfs 3.6.13. here's what it's complaining

***************
*** 162,168 ****
                 // user space buffer is swapped out. At that time
                 // entry can move to somewhere else
                 memcpy (local_buf, d_name, d_reclen);
- if (filldir (dirent, d_name, d_reclen, d_off, d_ino) < 0) {
                     if (local_buf != small_buf) {
                         kfree(local_buf) ;
                     }
--- 162,170 ----
                 // user space buffer is swapped out. At that time
                 // entry can move to somewhere else
                 memcpy (local_buf, d_name, d_reclen);
+ /* MaD's <martin@douda.net> ugly hack to make things compile
+ on -test7-pre3. If it breaks you have to keep both pieces.*/
+ if (filldir (dirent, d_name, d_reclen, d_off, d_ino, DT_UNKNOWN) < 0) {
                     if (local_buf != small_buf) {
                         kfree(local_buf) ;
                     }

> Date: Wed, 23 Aug 2000 21:17:08 +0200 (CEST)
> From: Martin MaD Douda <martin@douda.net>
> To: Paavo Nikula <pnikula@sunpoint.net>
> Subject: Re: test7-pre7 compile error [reiserfs]
>
>
>
> The function filldir has additional argument since test7-pre2. Here
> is
> patch allowing test7-pre3 working with reiserfs 3.6.12. Could you
> please
> test is if it will also help in test7-pre7? (It should work IMHO).
>
> Martin
>
>
> diff -ru linux/fs/reiserfs/dir.c
> linux-test7-pre3.mad/fs/reiserfs/dir.c
> --- linux/fs/reiserfs/dir.c Sun Aug 13 22:06:33 2000
> +++ linux-test7-pre3.mad/fs/reiserfs/dir.c Sun Aug 13 22:02:32 2000
> @@ -162,7 +162,9 @@
> // user space buffer is swapped out. At that time
> // entry can move to somewhere else
> memcpy (local_buf, d_name, d_reclen);
> - if (filldir (dirent, d_name, d_reclen, d_off, d_ino) < 0) {
> + /* MaD's <martin@douda.net> ugly hack to make things compile
> + on -test7-pre3. If it breaks you have to keep both pieces.*/
> + if (filldir (dirent, d_name, d_reclen, d_off, d_ino, DT_UNKNOWN) <
> 0) {
> if (local_buf != small_buf) {
> kfree(local_buf) ;
> }
>
>
>
> On Wed, 23 Aug 2000, Paavo Nikula wrote:
>
>> dir.c: In function `reiserfs_readdir':
>> dir.c:165: too few arguments to function
>> make[2]: *** [dir.o] Error 1
>> make[2]: Leaving directory `/disks/hdb/src/kernel/linux/fs/reiserfs'
>> make[1]: *** [_modsubdir_reiserfs] Error 2
>> make[1]: Leaving directory `/disks/hdb/src/kernel/linux/fs'
>> make: *** [_mod_fs] Error 2
>>
>>
>>
>> Joko muuten olet toga.com etujäsen, saat tarjouksia ja voit voittaa
> Palm V kämmenmikron. Osoite on
> http://www.sunpoint.net/SunAds/click.htm?mode=footer&id=7&jump=http%3
> A%2F%2Fwww.toga.com%2Ffi
>> -
>> To unsubscribe from this list: send the line "unsubscribe
> linux-kernel" in
>> the body of a message to majordomo@vger.kernel.org
>> Please read the FAQ at http://www.tux.org/lkml/
>>
>
> ----------------------------------------------------------------------
> ----------
> Martin "MaD" Douda
> WEB:http://martin.douda.net/ EMAIL:martin@douda.net
> SMS:mad@gate.mobil.cz (up to 160 characters) PHONE:+420603752779
> PGP:ID=0x6FE43023 Fingerprint:E495 11DA EF6E 0DD6 965A 54F3 888E CC9E
> 6FE4 3023
> ----------------------------------------------------------------------
> ----------
> If the automobile had followed the same development cycle as the
> computer, a
> Rolls-Royce today would cost $100, get a million miles to the gallon,
> and
> explode once a year, killing everyone inside.
>
>
>

Liity toga.com etujäseneksi, saat etujäsentarjouksia ja lisäksi voit voittaa Palm V kämmenmikron! http://www.sunpoint.net/SunAds/click.htm?mode=footer&id=6&jump=http%3A%2F%2Fwww.toga.com%2Ffi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Wed Aug 23 2000 - 21:00:09 EST