Re: directio

[Valery Kholodkov <valery+nginxru@xxxxxxxxxxx>]

Прошу прощения, я был недостаточно многолословен: O_DIRECT требует ещё 
чтобы размер буфера и указатель в файле были выравнены:
Breakpoint 1, ngx_read_file (file=0x80b9e90, buf=0xb7df9200 "", 
size=131072, offset=44626744)
    at src/os/unix/ngx_files.c:16
16          ngx_log_debug4(NGX_LOG_DEBUG_CORE, file->log, 0,
(gdb) n
21          n = pread(file->fd, buf, size, offset);
(gdb) n
23          if (n == -1) {
(gdb) n
24              ngx_log_error(NGX_LOG_CRIT, file->log, ngx_errno,
(gdb) n
26              return NGX_ERROR;

Соответственно, с этим патчем если я файл запрашиваю с начала, то всё 
работает, за исключением последнего блока. А если я файл запрашиваю 
частями с невыровненных смещений, то результат такой же как в предыдущем 
письме.
Идея здесь, как я понимаю, в том, что ОС не занимается копированием, а 
только трансляцией виртуальных адресов в физические. Данные 
подкладываются в нужное место физической памяти DMA.
Igor Sysoev wrote:
> On Fri, Jul 25, 2008 at 09:11:41PM +0200, Valery Kholodkov wrote:
>
> > Скажите, патч предназначен исключительно для FreeBSD?
> >
> > Потому как на линуксе 2.6.10 я справедливо получаю:
> >
> > 2008/07/25 20:58:08 [crit] 4600#0: *1 pread() failed, file 
> > "/usr/local/nginx/html/Synovya.i.Lubovniki.1.of.2.avi" (22: Invalid 
> > argument) while sending response to client, client: 192.168.1.13, 
> > server: 192.168.1.1, request: "GET /Synovya.i.Lubovniki.1.of.2.avi 
> > HTTP/1.1", host: "192.168.1.1"
> > по очевидной причине -- приемный буфер не выровнен на границу дискового 
> > блока:
> > Breakpoint 1, ngx_read_file (file=0x80b9ec4, buf=0x80f3970 "", 
> > size=32768, offset=0)
> >     at src/os/unix/ngx_files.c:16
> > 16          ngx_log_debug4(NGX_LOG_DEBUG_CORE, file->log, 0,
> > (gdb) n
> > 21          n = pread(file->fd, buf, size, offset);
> > (gdb) n
> > 23          if (n == -1) {
> > (gdb) n
> > 24              ngx_log_error(NGX_LOG_CRIT, file->log, ngx_errno,
> > (gdb) n
> > 26              return NGX_ERROR;
> > (gdb) n
> > 54      }
> Новый патч, в котором буфера выровнены по размеру сектора.
--
Regards,
Valery Kholodkov