Проецирование файла в память с помощью mmap

Question

Я пытаюсь проецировать файл в память для работы с ним. Структуры файлов contais, поэтому я пытаюсь использовать указатель на начало одной структуры, а затем читать ее и изменять некоторую переменную. Проблема в том, что время выполнения велико, и я полагаю, что использование mmap будет меньше. Это код, любое предложение?

#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

Ну, во-первых, расскажите, пожалуйста, что evaluacion определяется как и поставить do там, что совпадает с while; Я предполагаю, что это сразу после строки «int actual».

Во-вторых, похоже, что вы чаще всего вызываете mmap(), чем необходимо; как часто бывает revisanotas() с тем же fd? Сам вызов mmap медленный, например malloc; скорость - это когда вы используете сопоставленный файл, в данном случае данные, на которые указывает buffer.

В-третьих, вычислите datos.st_size/(sizeof(evaluacion)) один раз за пределами цикла и измените положение while, чтобы сравнить его. Текущий код выглядит так, будто он выполняет разделение один раз на итерацию через цикл, а деления медленны.

Посмотрите, поможет ли это.

Answer 2

Требование к read() не является необходимым. Mmap()карты содержимое файла в памяти для вас - поэтому он обычно быстрее, чем чтение всего файла с помощью read(). Вы должны убрать звонок до read(). Однако есть и другие проблемы с вашим кодом.

Если вы хотите, чтобы изменения действительно отражались в файле на диске, тогда вы должны позвонить msync(buffer, dataos.st_size, MS_SYNC). Когда вы закончите, вызовите munmap(buffer, dataos.st_size), чтобы освободить сегмент разделяемой памяти. Подумайте об msync() как эквивалент общей памяти fflush() и munmap() похож на close(). Ключевое различие между munmap() и close() состоит в том, что первое не очищает буферы или не синхронизируется с диском, поэтому вам нужно сделать это самостоятельно.

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]; 
}; 

Там только один FD, только один файл открыт, но многие внутри

Структуры

Answer 4

Я некоторый прогресс, я думаю, но, кажется, я не модифицируя файл. Если я запускаю программу один раз, он обнаруживает 32 переменных измененными, но если я бегу дважды все тот же, когда он должен быть изменен :(

Время выполнения теперь не плохо, я думаю, что

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

Я думаю, что сейчас работает :), код выглядит следующим образом. Прокомментируйте, если вы видите что-то не так:

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); 
} 

Спасибо всем за помощь.