Re: [devel] asm/page.h

[Kirill Maslinsky <kirill@altlinux.org>]

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) вообще не знаю, 
опасаюсь напортачить. 


-- 
Kirill Maslinsky
ALT Linux Team