Re: [devel] asm/page.h

[Led <ledest@gmail.com>]

Tuesday, 19 February 2008 02:18:11 Kirill Maslinsky написав:
> On Tue, Feb 19, 2008 at 03:06:28AM +0300, Dmitry V. Levin wrote:
> > On Tue, Feb 19, 2008 at 01:45:34AM +0200, Led wrote:
> > > > Это не оно?
> > > >
> > > > The user can find the value of PAGE_SIZE by calling getpagesize()
> > > > defined in unistd.h .
> > >
> > > Хотя, наверное, всё же:
> > >
> > > -asm/page.h
> > > +sys/user.h
> >
> > Какой-то сомнительный файл.  Лучше sysconf(_SC_PAGESIZE).
>
> Вот так?
>
> --- a/eciadsl-usermode/pusb-linux.c
> +++ b/eciadsl-usermode/pusb-linux.c
> @@ -29,7 +29,6 @@
>  #include <string.h>
>
>  #include "pusb-linux.h"
> -#include <asm/page.h>
>
>  #include "pusb.h"
>
> @@ -438,8 +437,8 @@ inline int pusb_endpoint_rw(int fd, int ep, unsigned
> char* buf, int size, int ti
>
>                 bulk.len = size;
>
> -               if (size > PAGE_SIZE)
> -                       bulk.len = PAGE_SIZE;
> +               if (size > sysconf(_SC_PAGESIZE))
> +                       bulk.len = sysconf(_SC_PAGESIZE);
>
>                 bulk.timeout = timeout;
>                 bulk.data    = buf;

Зачем дважды вызывать? ИМХО лучше завести переменную page_size.


-- 
Led