Message-ID: <EE83E551E08D1D43AD52D50B9F511092E114A7@ntserver2>
From: Gregory Giguashvili <Gregoryg@ParadigmGeo.com>
To: "'Andi Kleen'" <ak@suse.de>
Cc: linux-kernel@vger.kernel.org
Subject: RE: Problem with msync system call
Date: Tue, 23 Jul 2002 21:36:26 +0200
MIME-Version: 1.0
Content-Type: multipart/mixed;
	boundary="----_=_NextPart_000_01C23280.3BCAE2F0"
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 3915
Lines: 133

This message is in MIME format. Since your mail reader does not understand
this format, some or all of this message may not be legible.

------_=_NextPart_000_01C23280.3BCAE2F0
Content-Type: text/plain;
	charset="iso-8859-1"

It seems to finally work both between two Linux machines and
between Linux and other OSes! Thank you all for your help!

I attach the working source to whoever that cares...

>I think the problem in your case is that you have the pages mmaped.
>NFS uses invalidate_inode_pages() to throw away the cache, but that
>doesn't work when the pages are mapped. It may work to munmap/mmap
>around the locking.
Now, I think I understand what the problem is.

Can we make msync call with MS_INVALIDATE flag temporarily unmap the 
file, invalidate the cache and remap the file again? It sounds like 
a hell of an overhead, but users don't expect msync call to be a 
light one.

Anyway, this would be better than the current behavior, which in fact does 
nothing for the mapped files. Also, the documentation for msync call is 
extremely vague, which only adds to the confusion.

Best,
Giga


------_=_NextPart_000_01C23280.3BCAE2F0
Content-Type: application/octet-stream;
	name="mmap.cc"
Content-Transfer-Encoding: quoted-printable
Content-Disposition: attachment;
	filename="mmap.cc"

#include <stdio.h>=0A=
#include <unistd.h>=0A=
#include <sys/mman.h>=0A=
#include <string.h>=0A=
#include <sys/types.h>=0A=
#include <sys/stat.h>=0A=
#include <fcntl.h>=0A=
=0A=
int main (int argc, char* argv []) =0A=
{=0A=
    char buffer [BUFSIZ];=0A=
    char* pMap;=0A=
=0A=
    int fd =3D open ("MAPPED.FILE", O_CREAT | O_RDWR | O_SYNC, =
0666);=0A=
    if (fd =3D=3D -1) {=0A=
	perror ("open");=0A=
	return -1;=0A=
    }=0A=
=0A=
    if (lseek (fd, BUFSIZ - 1, SEEK_SET) =3D=3D -1) {=0A=
	perror ("lseek");=0A=
	return -1;=0A=
    }=0A=
    write (fd, "\0", 1);=0A=
    if (lseek (fd, 0, SEEK_SET) =3D=3D -1) {=0A=
	perror ("lseek");=0A=
	return -1;=0A=
    }=0A=
=0A=
    pMap =3D (char*) mmap (NULL, BUFSIZ, PROT_READ | PROT_WRITE, =0A=
			 MAP_SHARED, fd, 0);=0A=
    if ((size_t) pMap =3D=3D -1) {=0A=
	perror ("mmap");=0A=
	return -1;=0A=
    }=0A=
    =0A=
    while (1) {=0A=
	fprintf (stderr, "<Press ENTER to read> OR <Enter string to write> $ =
");=0A=
	fgets (buffer, sizeof (buffer) - 1, stdin);=0A=
=0A=
	struct flock lck =3D { =0A=
	    F_WRLCK,  /* Type of lock: F_RDLCK, F_WRLCK, or F_UNLCK    */=0A=
	    SEEK_SET, /* Where `l_start' is relative to (like `lseek') */=0A=
	    0,        /* Offset where the lock begins                  */=0A=
	    0,        /* Size of the locked area; zero means until EOF */=0A=
	    getpid () /* Process holding the lock                      */=0A=
	};=0A=
=0A=
	if (munmap (pMap, BUFSIZ)) {=0A=
	    perror ("munmap");=0A=
	    return -1;=0A=
	}=0A=
=0A=
	if (fcntl (fd, F_SETLK, &lck)) {=0A=
	    perror ("fcntl");=0A=
	    return -1;=0A=
	}=0A=
=0A=
	pMap =3D (char*) mmap (NULL, BUFSIZ, PROT_READ | PROT_WRITE, =0A=
				   MAP_SHARED, fd, 0);=0A=
	if ((size_t) pMap =3D=3D -1) {=0A=
	    perror ("mmap");=0A=
	    return -1;=0A=
	}=0A=
=0A=
	if (buffer [0] =3D=3D '\n') {=0A=
	    fprintf (stderr, "Mapped file contents: %s\n", pMap);=0A=
	} else {=0A=
	    strncpy (pMap, buffer, strlen (buffer) - 1);=0A=
	    fprintf (stderr, "Written to mapped file: %s\n", pMap);=0A=
	}=0A=
=0A=
 	if (msync (pMap, BUFSIZ, MS_SYNC | MS_INVALIDATE)) {=0A=
 	    perror ("msync");=0A=
 	    return -1;=0A=
 	}=0A=
=0A=
	lck.l_type =3D F_UNLCK;=0A=
	if (fcntl (fd, F_SETLK, &lck)) {=0A=
	    perror ("fcntl");=0A=
	    return -1;=0A=
	}=0A=
    }=0A=
=0A=
    return 0;=0A=
}=0A=

------_=_NextPart_000_01C23280.3BCAE2F0--
-
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/