[hpoussin] 21548: Add more documentation about ntoskrnl/io/pnpmgr.c

Author: hpoussin
Date: Tue Apr 11 17:00:08 2006
New Revision: 21548
URL: http://svn.reactos.ru/svn/reactos?rev=21548&view=rev
Log:
Add more documentation about ntoskrnl/io/pnpmgr.c
Modified:
    trunk/documentation/audit/ntoskrnl/io/pnpmgr.txt
Modified: trunk/documentation/audit/ntoskrnl/io/pnpmgr.txt
URL:
http://svn.reactos.ru/svn/reactos/trunk/documentation/audit/ntoskrnl/io/pnp…
==============================================================================
--- trunk/documentation/audit/ntoskrnl/io/pnpmgr.txt (original)
+++ trunk/documentation/audit/ntoskrnl/io/pnpmgr.txt Tue Apr 11 17:00:08 2006
@@ -11,8 +11,11 @@
 Result: Clean (<5 lines)
 IopQueryDeviceCapabilities()
-Info: Google reports only references to ReactOS's source code, no results in msdn
search too
-Result: Author needs to be contacted, however function implementation seems to be rather
small and has a few comments
+Info:
+       Google reports only references to ReactOS's source code, no results in msdn
search too.
+       IRP_MN_QUERY_CAPABILITIES and DEVICE_CAPABILITIES structure are described here:
http://www.osronline.com/DDKx/kmarch/k112_22r6.htm
+       Uses IopInitiatePnpIrp to send the IRP_MJ_PNP Irp
+Result: Clean
 IoInvalidateDeviceRelations()
 Result: Clean (<5 lines)
@@ -34,8 +37,10 @@
 Result: <5 lines of actual code -> Clean
 IopGetBusTypeGuidIndex()
-Info: Only ReactOS source code got in results of Google search. Function is heavily
commented. There is a goto usage, however it's obvious and doesn't seem to come
-        from reverse engineering.
+Info:
+       Searches BusTypeGuid entry in the IopBusTypeGuidList list. If it is not found,
create a new entry in the list and return its index
+       Only ReactOS source code got in results of Google search. Function is heavily
commented. There is a goto usage, however it's obvious and doesn't seem to come
+       from reverse engineering.
 Result: ?
 IopCreateDeviceNode() / IopFreeDeviceNode()
@@ -43,8 +48,11 @@
 Result: ?
 IopInitiatePnpIrp()
-Info: Google finds only ReactOS results. I haven't checked other information sources
-Result: ?
+Info:
+       Google finds only ReactOS results.
+       Sends a IRP_MJ_PNP Irp to a device stack.
+       Method to send a synchronous Irp is well documented on the Web, and all used
methods are documented
+Result: Clean
 IopTraverseDeviceTreeNode() / IopTraverseDeviceTree()
 Info: Google finds only ReactOS results. I haven't checked other information sources
@@ -59,16 +67,25 @@
 Result: ?
 IopAssignDeviceResources()
-Info: Google finds only ReactOS results. I haven't checked other information sources.
This function seems to be absolutely ReactOS-specific (especially the irq-hacks/etc)
-Result: Probably Clean?
+Info:
+       Google finds only ReactOS results.
+       Creates a allocated resource list by using the resource requirements list. It
doesn't do resource arbitration.
+       Structures are documented at http://www.osronline.com/DDKx/kmarch/k112_5quq.htm
and http://www.osronline.com/DDKx/kmarch/k112_92k2.htm
+       There is a goto usage, to prevent duplication of resources cleanup code
+Result: Clean
 IopGetParentIdPrefix()
-Info: Google finds only ReactOS results. I haven't checked other information sources
-Result: ?
+Info:
+       Google finds only ReactOS results.
+       Creates a prefix for the device instance string, when it may be not unique on the
whole computer.
+       The prefix is the crc32 of the instance path of the parent device node.
+Result: Clean
 IopActionInterrogateDeviceStack()
-Info: Google finds only ReactOS results. I haven't checked other information sources
-Result: ?
+Info:
+       Google finds only ReactOS results. I haven't checked other information sources
+       Sends lots of IRP_MJ_PNP Irps to fill the device node structure
+Result: Probably clean
 IopActionConfigureChildServices() / IopActionInitChildServices()
 Info: Google finds only ReactOS results. I haven't checked other information sources
@@ -84,7 +101,7 @@
 Result: Clean (<5 LOC)
 IopInvalidateDeviceRelations()
-Info: hpoussin worked on it, and also I remember there are sources of documentation for
it
+Info: Fireball remembers there are sources of documentation for it
 Result: probably Clean
 IopSetRootDeviceInstanceData()