[ros-kernel] RE: [ros-cvs] CVS Update: reactos

[Hartmut Birr]

Hi,    

the changes in copy.c are not correct. If the mdl is not allocated from
stack, it must be deallocated after the call to IoPageRead or IoPageWrite. I
do not see a reason to allocate the mdl not from stack but from nonpaged
pool. The mdl's are for max 16 pages. That is short enough for an allocation
from stack. 

- Hartmut 


> -----Original Message-----
> From: ros-cvs-bounces at reactos.com 
> [mailto:ros-cvs-bounces at reactos.com] On Behalf Of Vizzini
> Sent: Saturday, June 19, 2004 1:54 AM
> To: ros-cvs at reactos.com
> Subject: [ros-cvs] CVS Update: reactos
> 
> 
> CVSROOT:	/CVS/ReactOS
> Module name:	reactos
> Repository:	reactos/ntoskrnl/ps/
> Changes by:	vizzini at mok.osexperts.com	04/06/19 01:53:36
> 
> Modified files:
> 	reactos/ntoskrnl/cc/: copy.c 
> 	reactos/ntoskrnl/io/: buildirp.c 
> 	reactos/ntoskrnl/mm/: pagefile.c virtual.c 
> 	reactos/ntoskrnl/nt/: profile.c 
> 	reactos/ntoskrnl/ps/: idle.c 
> 
> Log message:
> 	patch by Tom (tmkpl at poczta.onet.pl)
> 	to clean up return val checking of some mdl calls
> 
> _______________________________________________
> Ros-cvs mailing list
> Ros-cvs at reactos.com
> http://reactos.com/mailman/listinfo/ros-cvs
>