Re: [PATCH]: make highmem only things enclosed in the right #ifdef

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 19 Dec 2002, Juan Quintela wrote:

> What do you think of this new one?

 Well, you could remove the line below:

>  	         sizeof(pgd_t ) * USER_PTRS_PER_PGD);
>  
> -	pgd_base = swapper_pg_dir;
>  
>  #ifdef CONFIG_HIGHMEM

but that's nitpicking (and I may fix it up if Ralf applies the patch as
is) -- I've pointed you out the problem of spacing more to bring your
attention to such details than to object this particular change.  The
patch looks semantically OK. 

-- 
+  Maciej W. Rozycki, Technical University of Gdansk, Poland   +
+--------------------------------------------------------------+
+        e-mail: macro@ds2.pg.gda.pl, PGP key available        +



[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux