[ros-diffs] [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()