Re: [devel] asm/page.h

["Kirill A. Shutemov" <k.shutemov@velesys.com>]

[-- Attachment #1: Type: text/plain, Size: 2244 bytes --]

On [Tue, 19.02.2008 04:49], Dmitry V. Levin wrote:
> On Tue, Feb 19, 2008 at 03:40:41AM +0300, Kirill Maslinsky wrote:
> > On Tue, Feb 19, 2008 at 02:23:41AM +0200, Led wrote:
> > > Tuesday, 19 February 2008 02:18:11 Kirill Maslinsky написав:
> > 
> > [...]
> > 
> > > > -               if (size > PAGE_SIZE)
> > > > -                       bulk.len = PAGE_SIZE;
> > > > +               if (size > sysconf(_SC_PAGESIZE))
> > > > +                       bulk.len = sysconf(_SC_PAGESIZE);
> > 
> > > Зачем дважды вызывать? ИМХО лучше завести переменную page_size.
> > 
> > Логично. Вот так будет корректно?
> > 
> > --- 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"
> >  
> > @@ -431,6 +430,7 @@ inline int pusb_endpoint_rw(int fd, int ep, unsigned char* buf, int size, int ti
> >         static struct usbdevfs_bulktransfer bulk;
> >         static int ret;
> >         int received = 0;
> > +       int page_size = sysconf(_SC_PAGESIZE);
> >  
> >         do
> >         {
> > @@ -438,8 +438,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 > page_size)
> > +                       bulk.len = page_size;
> >  
> > Вы меня простите за тупые вопросы, я просто этого языка (C) вообще не знаю, 
> > опасаюсь напортачить. 
> 
> Код синтаксически корректный, но upstream может захотеть соптимизировать
> количество вызовов sysconf.

А проверить возвращаемое из sysconf значение не нужно? А то мало ли что...

-- 
Regards,  Kirill A. Shutemov
 + Belarus, Minsk
 + Velesys Ltd, http://www.velesys.com/
 + ALT Linux Team, http://www.altlinux.com/

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 197 bytes --]