Projection de fichier en mémoire à l'aide de mmap

Question

J'essaie de projeter un fichier en mémoire pour pouvoir l'utiliser. Le fichier contais structs donc j'essaye d'utiliser un pointeur au début d'une structure puis de le lire et de modifier une variable. Le problème est que le temps d'exécution est élevé et je suppose qu'en utilisant mmap le temps sera moindre. Ceci est le code, une suggestion?

#include <unistd.h> 
#include <sys/stat.h> 
#include <stdio.h> 
#include <stdlib.h> 
#include <sys/mman.h> 

int revisanotas(int fd) 
{ 
int nbytes=1; 
int nbytese=0; 
int i=0; 
int n=0; 
struct stat datos; 
fstat(fd, &datos); 
evaluacion buf; 
evaluacion* buffer=mmap(0,datos.st_size, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0); 
int actual = read(fd,buffer,datos.st_size); 
{ 
i++; 
if (buffer[i].notamedia >= 4.5 && buffer[i].notamedia < 5) 
{ 
n=n+1; 
printf("Notamedia = %f\n",buffer[i].notamedia); 

buffer[i].notamedia=5; 
} 

}while (i<(datos.st_size/(sizeof(evaluacion)))); 
return 

Answer 1

Eh bien, tout d'abord, s'il vous plaît nous dire ce que evaluacion est défini comme, et mis le do là-dedans qui correspond à la while; Je suppose que c'est juste après la ligne "int réelle". Deuxièmement, il semble que vous appeliez mmap() plus souvent que nécessaire; à quelle fréquence est revisanotas() appelé avec le même fd? L'appel mmap lui-même est lent, comme malloc; la vitesse est lorsque vous utilisez le fichier mappé, dans ce cas, les données pointées par buffer. Troisièmement, calculez datos.st_size/(sizeof(evaluacion)) une fois en dehors de la boucle et modifiez la clause while pour la comparer à celle-ci. Le code actuel semble effectuer la division une fois par itération à travers la boucle, et les divisions sont lentes.

Voir si cela aide tout.

Answer 2

L'appel à read() est inutile. Mmap()cartes le contenu du fichier en mémoire pour vous - c'est pourquoi il est généralement plus rapide que la lecture du fichier entier en utilisant read(). Vous devriez être en mesure de supprimer l'appel à read() tout à fait. Cependant, il y a d'autres problèmes avec votre code.

Si vous souhaitez que les modifications soient reflétées dans le fichier disque, vous devez appeler le msync(buffer, dataos.st_size, MS_SYNC). Lorsque vous avez terminé, appelez munmap(buffer, dataos.st_size) pour libérer le segment de mémoire partagée. Pensez à msync() comme la mémoire partagée équivalente à fflush() et munmap() est similaire à close(). La principale différence entre munmap() et close() est que le premier ne vide pas les tampons ou ne se synchronise pas sur le disque, donc vous devez le faire vous-même.

Answer 3

Thanks mike for the answer, the struct is something like this and it's defined in a header file: 

struct evaluacion 
{ char id[16]; 
char apellido1[32]; 
char apellido2[32]; 
char name[32]; float nota1p; 
float nota2p; 
float notamedia; 
char photofilename[20]; 
int photosize; 
char photodata[16000]; 
}; 

Il n'y a qu'une seule fd, un seul fichier ouvert, mais beaucoup struct à l'intérieur

Answer 4

Je fais quelques progrès, je pense, mais il semble que je ne suis pas modifier le fichier. Si je lance le programme une fois, il détecte 32 variables modifiées, mais si je cours deux fois toujours le même, quand le Il est censé être modifié :(

Le temps d'exécution est maintenant pas mal je pense

int revisanotas(int fd) 
{ 
int i=0; 
int n=0; 
struct stat datos; 
fstat(fd, &datos); 
int num=datos.st_size/(sizeof(evaluacion)); 
evaluacion buf; 
evaluacion* buffer=mmap(0,datos.st_size, PROT_READ | PROT_WRITE, MAP_PRIVATE, fd, 0); 
do 
{ 
i++; 
if (buffer[i].notamedia >= 4.5 && buffer[i].notamedia < 5) 
{ 
n=n+1; 

buffer[i].notamedia=5; 
} 
msync(&buffer[i],sizeof(buffer[i]),MS_SYNC); 
}while (i<(num)); 
munmap(buffer,datos.st_size); 
return(n); 
} 

Answer 5

Je pense maintenant travaille :), le code est le suivant. S'il vous plaît commenter si vous voyez quelque chose de mal:

int revisanotas(int fd) 
{ 
int i=0; 
int n=0; 
struct stat datos; 
fstat(fd, &datos); 
int num=datos.st_size/(sizeof(evaluacion)); 

evaluacion* buffer=mmap(0,datos.st_size, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0); 
do 
{ 
i++; 
if (buffer[i].notamedia >= 4.5 && buffer[i].notamedia < 5) 
{ 
n=n+1; 
buffer[i].notamedia=5; 
} 

}while (i<(num)); 
int r=munmap(&buffer[0],datos.st_size); 

return(n); 
} 

Merci à tous pour l'aide.