Re: [uml-devel] compile error ...(now tracked down)

Re: [uml-devel] compile error ...(now tracked down)

[Steve Schmidtke]

>blaisorblade_spam@yahoo.it said:
> > # define st_atime st_atim.tv_sec        /* Backward compatibility.  */
>	...
> >                 .st_atime   = src->st_atime,   /* time of last access */
>
>This is just because of the names of the struct uml_stat.  Changing those
>field names should fix the compile problem.

Yup, there are a few ways sub-second file timestamps are handled:

  http://mail.gnu.org/archive/html/bug-gnulib/2003-08/msg00183.html

I'm not going to pick one at this time. :)  I'll send Jeff another
patch that renames the entries and references to ust_*.  I also see a few 
stat() references in the code I can stomp right now.

Steve

_________________________________________________________________
Protect your PC - get McAfee.com VirusScan Online  
http://clinic.mcafee.com/clinic/ibuy/campaign.asp?cid=3963



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

Re: [uml-devel] compile error ...(now tracked down)

[Jeff Dike]

steve_schmidtke@hotmail.com said:
> I'll send Jeff another patch that renames the entries and references
> to ust_*.  I also see a few  stat() references in the code I can stomp
> right now.

Don't worry about it.  I'm halfway through that now.

				Jeff



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

[uml-devel] compile error with 2.4.22-uml4 and 2.4.23-pre4-djc2-4um

[Matthew R. Scott]

With both of these patches I am getting the following error during compile:

gcc-2.95   -Wall -Wstrict-prototypes -Wno-trigraphs -O2 -fno-strict-aliasing 
-fno-common -march=pentium -O3 -pipe -fomit-frame-pointer -U__i386__ -Ui386 
-DUM_FASTCALL -g -D__arch_um__ -DSUBARCH=\"i386\" -D_LARGEFILE64_SOURCE  -I/
home/gldnspud/uml/build/buildkernel-build/linux/arch/um/include -I/home/
gldnspud/uml/build/buildkernel-build/linux/arch/um/kernel/tt/include -I/home/
gldnspud/uml/build/buildkernel-build/linux/arch/um/kernel/skas/include 
-D_GNU_SOURCE -c -o umid.o umid.c
In file included from umid.c:18:
/home/gldnspud/uml/build/buildkernel-build/linux/arch/um/include/os.h:41: 
warning: no semicolon at end of struct or union
/home/gldnspud/uml/build/buildkernel-build/linux/arch/um/include/os.h:41: 
parse error before `.'
/home/gldnspud/uml/build/buildkernel-build/linux/arch/um/include/os.h:42: 
parse error before `.'
/home/gldnspud/uml/build/buildkernel-build/linux/arch/um/include/os.h:43: 
parse error before `.'
make[4]: *** [umid.o] Error 1
make[4]: Leaving directory `/home/gldnspud/uml/build/buildkernel-build/linux/
arch/um/kernel'
make[3]: *** [first_rule] Error 2
make[3]: Leaving directory `/home/gldnspud/uml/build/buildkernel-build/linux/
arch/um/kernel'
make[2]: *** [_dir_arch/um/kernel] Error 2
make[2]: Leaving directory `/home/gldnspud/uml/build/buildkernel-build/linux'
make[1]: *** [buildkernel-build-stamp] Error 2
make[1]: Leaving directory `/home/gldnspud/uml/build'
make: *** [build-stamp] Error 2

When I look at line 41, 42, and 43 in arch/um/include/os.h though, there 
doesn't seem to be any sort of problems.  Those lines are in the middle of a 
perfectly valid-looking struct.


-- 
Matthew R. Scott
OMAjA / http://www.omaja.com/



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

Re: [uml-devel] compile error ...(now tracked down)

[BlaisorBlade]

Alle 20:29, martedì 16 settembre 2003, Matthew R. Scott ha scritto:

> With both of these patches I am getting the following error during compile:
I use the headers from glibc-devel-2.3.1-10mdk(from mandrake 9.1) and I get 
the same errors. Supposed that the patch compiles well somewhere else, it's 
something changed with glibc from the system of who released the patch. See 
below, especially where the code snippets speak about backward compatibility.

> In file included from umid.c:18:
> /home/gldnspud/uml/build/buildkernel-build/linux/arch/um/include/os.h:41:
> warning: no semicolon at end of struct or union
> /home/gldnspud/uml/build/buildkernel-build/linux/arch/um/include/os.h:41:
> parse error before `.'
> /home/gldnspud/uml/build/buildkernel-build/linux/arch/um/include/os.h:42:
> parse error before `.'
> /home/gldnspud/uml/build/buildkernel-build/linux/arch/um/include/os.h:43:
> parse error before `.'
>
> When I look at line 41, 42, and 43 in arch/um/include/os.h though, there
> doesn't seem to be any sort of problems.  Those lines are in the middle of
> a perfectly valid-looking struct.

I agree, but cd to that folder and rerun the exact gcc command, replacing -c 
-o umid.o with -E(to preprocess only the file), then use less(the 
preprocessed output is sent to stdout). On about line 5295(on me, look well), 
you can see that those line, because of a preprocessor #define, get turned 
into something else. The os.h file is good, but in umid.c it's included after 
sys/stat.h(which is the glibc one!). It then includes in turn bits/stat.h:

#ifdef __USE_MISC					
# define st_atime st_atim.tv_sec        /* Backward compatibility.  */
# define st_mtime st_mtim.tv_sec
# define st_ctime st_ctim.tv_sec
...
__USE_MISC comes from features.h; if you define __BSD_SOURCE(to choose this 
dialect of Unix) it gets defined; if you don't define anything, it comes up 
automatically. Anyway we need it in the code below.

I've given this command inside arch/um to find what includes both os.h and 
sys/stat.h
# grep 'os\.h\"' $(grep 'sys/stat\.h' `find . -name '*.[ch]'` -l)
./drivers/chan_user.c:#include "os.h"
./os-Linux/file.c:#include "os.h"
./kernel/umid.c:#include "os.h"
./kernel/initrd_user.c:#include "os.h"

I've tried #undefining them, but they are needed somewhere else(in file.c), so 
I've simply swapped the order of the two includes(so that os.h is included 
with those define's not in place). And yet, it doesn't compile. It stops on 
file.c.

This is the offending code(which was absent from the cleanup of -4um):
static void copy_stat(struct uml_stat* dst, struct stat64* src)
{
        *dst=((struct uml_stat) {
                .st_dev     = src->st_dev,     /* device */
                .st_ino     = src->st_ino,     /* inode */
                .st_mode    = src->st_mode,    /* protection */
                .st_nlink   = src->st_nlink,   /* number of hard links */
                .st_uid     = src->st_uid,     /* user ID of owner */
                .st_gid     = src->st_gid,     /* group ID of owner */
                .st_size    = src->st_size,    /* total size, in bytes */
                .st_blksize = src->st_blksize, /* blocksize for filesys I/O */
                .st_blocks  = src->st_blocks,  /* number of blocks allocated 
*/
                .st_atime   = src->st_atime,   /* time of last access */
                .st_mtime   = src->st_mtime,   /* time of last modification */
                .st_ctime   = src->st_ctime,   /* time of last change */
        });
}

As you can see, on the same line it needs that st_{a,m,c}time is its literal 
value on the right and the original vaue on the left. I am available to 
anyone who wants to fix the bug with help, actual hacking and whatever, but 
I'm not starting because I don't know how... if I #undef the things, then I 
need to patch the above function to use directly st_atim.tv_sec, and then 
maybe things won't work on systems where they worked before.
-- 
cat <<EOSIGN
Paolo Giarrusso, aka Blaisorblade
Linux Kernel 2.4.21/2.6.0-test on an i686; Linux registered user n. 292729
EOSIGN




-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

Re: [uml-devel] compile error ...(now tracked down)

[Jeff Dike]

blaisorblade_spam@yahoo.it said:
> # define st_atime st_atim.tv_sec        /* Backward compatibility.  */
	...
>                 .st_atime   = src->st_atime,   /* time of last access */ 

This is just because of the names of the struct uml_stat.  Changing those
field names should fix the compile problem.

				Jeff



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel