[ros-kernel] Re: Recent changes to vfatfs broken

[David W. <we...@cw...>]

On Sun, Nov 04, 2001 at 09:13:48PM -0500, Phillip Susi wrote:
> It seems that the recent changes to vfatfs to allow async IO are 
> broken.  When attempting to mount the filesystem, VfatBuildRequest() calls 
> FsRtlEnterFileSystem() which is a #define to KeEnterCriticalRegion, which 
> disables APCs.  This causes VfatFileSystemControl to be called with APCs 
> disabled, which prevents the read IO issued to read the boot sector to not 
> complete.
> 
> What is the purpose of calling FsRtlEnterFileSystem(), and I seem to 
> remember reading that IO completions use special APCs that can not be 
> disabled, so the APC should be delivered anyway.  Is it then the case that 
> we need to use these special APCs to allow delivery even though APCs have 
> been disabled for some reason?
> 
I think it would be enough to remove the test for KeApcDisable from 
KeInsertQueueApc. At the moment I'm not sure exactly what 'normal' kernel 
mode APCs are used for and nothing does use them so it should be safe to 
ignore Ke[Leave,Enter]CriticalSection.

====================================================
= To remove yourself from this mailing list, go to =
= http://www.reactos.com/home/mailing.html         =
====================================================