[ros-diffs] [hpoussin] 32301: Use dynamic traces Some cleanup

hpoussin at svn.reactos.org hpoussin at svn.reactos.org
Mon Feb 11 17:54:06 CET 2008


Author: hpoussin
Date: Mon Feb 11 19:54:05 2008
New Revision: 32301

URL: http://svn.reactos.org/svn/reactos?rev=32301&view=rev
Log:
Use dynamic traces
Some cleanup

Modified:
    trunk/reactos/drivers/filesystems/npfs_new/create.c
    trunk/reactos/drivers/filesystems/npfs_new/finfo.c
    trunk/reactos/drivers/filesystems/npfs_new/fsctrl.c
    trunk/reactos/drivers/filesystems/npfs_new/npfs.c
    trunk/reactos/drivers/filesystems/npfs_new/npfs.h
    trunk/reactos/drivers/filesystems/npfs_new/npfs.rbuild
    trunk/reactos/drivers/filesystems/npfs_new/rw.c
    trunk/reactos/drivers/filesystems/npfs_new/volume.c

Modified: trunk/reactos/drivers/filesystems/npfs_new/create.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/npfs_new/create.c?rev=32301&r1=32300&r2=32301&view=diff
==============================================================================
--- trunk/reactos/drivers/filesystems/npfs_new/create.c (original)
+++ trunk/reactos/drivers/filesystems/npfs_new/create.c Mon Feb 11 19:54:05 2008
@@ -10,42 +10,43 @@
 
 #include "npfs.h"
 
-//#define NDEBUG
-#include <debug.h>
-
 /* FUNCTIONS *****************************************************************/
 
 NTSTATUS NTAPI
-NpfsCreate(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsCreate(IN PDEVICE_OBJECT DeviceObject,
+           IN PIRP Irp)
 {
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "IRP_MJ_CREATE\n");
+	TRACE_(NPFS, "IRP_MJ_CREATE\n");
 
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 	return STATUS_SUCCESS;
 }
 
 NTSTATUS NTAPI
-NpfsCreateNamedPipe(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsCreateNamedPipe(IN PDEVICE_OBJECT DeviceObject,
+                    IN PIRP Irp)
 {
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "IRP_MJ_CREATE Named Pipe\n");
+	TRACE_(NPFS, "IRP_MJ_CREATE Named Pipe\n");
 
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 	return STATUS_SUCCESS;
 }
 
 NTSTATUS NTAPI
-NpfsCleanup(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsCleanup(IN PDEVICE_OBJECT DeviceObject,
+            IN PIRP Irp)
 {
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "Cleanup\n");
+	TRACE_(NPFS, "Cleanup\n");
 
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 	return STATUS_SUCCESS;
 }
 
 NTSTATUS NTAPI
-NpfsClose(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsClose(IN PDEVICE_OBJECT DeviceObject,
+          IN PIRP Irp)
 {
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "IRP_MJ_CLOSE\n");
+	TRACE_(NPFS, "IRP_MJ_CLOSE\n");
 
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 	return STATUS_SUCCESS;

Modified: trunk/reactos/drivers/filesystems/npfs_new/finfo.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/npfs_new/finfo.c?rev=32301&r1=32300&r2=32301&view=diff
==============================================================================
--- trunk/reactos/drivers/filesystems/npfs_new/finfo.c (original)
+++ trunk/reactos/drivers/filesystems/npfs_new/finfo.c Mon Feb 11 19:54:05 2008
@@ -10,19 +10,17 @@
 
 #include "npfs.h"
 
-//#define NDEBUG
-#include <debug.h>
-
 /* FUNCTIONS *****************************************************************/
 
 NTSTATUS NTAPI
-NpfsQueryInformation(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsQueryInformation(IN PDEVICE_OBJECT DeviceObject,
+                     IN PIRP Irp)
 {
+	TRACE_(NPFS, "NpfsQueryInformation()\n");
+
 	FsRtlEnterFileSystem();
 
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "NpfsQueryInformation()\n");
-
-	/* TODO: Implement */
+	UNIMPLEMENTED;
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 
 	FsRtlExitFileSystem();
@@ -31,13 +29,14 @@
 }
 
 NTSTATUS NTAPI
-NpfsSetInformation(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsSetInformation(IN PDEVICE_OBJECT DeviceObject,
+                   IN PIRP Irp)
 {
+	TRACE_(NPFS, "NpfsSetInformation()\n");
+
 	FsRtlEnterFileSystem();
 
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "NpfsSetInformation()\n");
-
-	/* TODO: Implement */
+	UNIMPLEMENTED;
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 
 	FsRtlExitFileSystem();

Modified: trunk/reactos/drivers/filesystems/npfs_new/fsctrl.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/npfs_new/fsctrl.c?rev=32301&r1=32300&r2=32301&view=diff
==============================================================================
--- trunk/reactos/drivers/filesystems/npfs_new/fsctrl.c (original)
+++ trunk/reactos/drivers/filesystems/npfs_new/fsctrl.c Mon Feb 11 19:54:05 2008
@@ -10,19 +10,17 @@
 
 #include "npfs.h"
 
-//#define NDEBUG
-#include <debug.h>
-
 /* FUNCTIONS *****************************************************************/
 
 NTSTATUS NTAPI
-NpfsDirectoryControl(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsDirectoryControl(IN PDEVICE_OBJECT DeviceObject,
+                     IN PIRP Irp)
 {
+	TRACE_(NPFS, "NpfsDirectoryControl()\n");
+
 	FsRtlEnterFileSystem();
 
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "NpfsDirectoryControl()\n");
-
-	/* TODO: Implement */
+	UNIMPLEMENTED;
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 
 	FsRtlExitFileSystem();
@@ -31,13 +29,14 @@
 }
 
 NTSTATUS NTAPI
-NpfsFileSystemControl(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsFileSystemControl(IN PDEVICE_OBJECT DeviceObject,
+                      IN PIRP Irp)
 {
+	TRACE_(NPFS, "NpfsFileSystemControl()\n");
+
 	FsRtlEnterFileSystem();
 
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "NpfsFileSystemControl()\n");
-
-	/* TODO: Implement */
+	UNIMPLEMENTED;
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 
 	FsRtlExitFileSystem();

Modified: trunk/reactos/drivers/filesystems/npfs_new/npfs.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/npfs_new/npfs.c?rev=32301&r1=32300&r2=32301&view=diff
==============================================================================
--- trunk/reactos/drivers/filesystems/npfs_new/npfs.c (original)
+++ trunk/reactos/drivers/filesystems/npfs_new/npfs.c Mon Feb 11 19:54:05 2008
@@ -10,25 +10,34 @@
 
 #include "npfs.h"
 
-//#define NDEBUG
-#include <debug.h>
-
-ULONG NpfsDebugLevel = NPFS_DL_API_TRACE;
-
-static VOID NpfsInitializeVcb(PNPFS_VCB Vcb);
-
 /* FUNCTIONS *****************************************************************/
 
+static VOID
+NpfsInitializeVcb(OUT PNPFS_VCB Vcb)
+{
+	/* Initialize Volume Control Block, it's one per whole named pipe
+	   file system. */
+
+	TRACE_(NPFS, "NpfsInitializeVcb(), Vcb = %p\n", Vcb);
+
+	/* Zero-init whole VCB */
+	RtlZeroMemory(Vcb, sizeof(NPFS_VCB));
+
+	/* Initialize the common header */
+	Vcb->NodeTypeCode = NPFS_NODETYPE_VCB;
+	Vcb->NodeByteSize = sizeof(NPFS_VCB);
+}
+
 NTSTATUS NTAPI
-DriverEntry(PDRIVER_OBJECT DriverObject,
-			PUNICODE_STRING RegistryPath)
+DriverEntry(IN PDRIVER_OBJECT DriverObject,
+            IN PUNICODE_STRING RegistryPath)
 {
 	PNPFS_DEVICE_EXTENSION DeviceExtension;
 	PDEVICE_OBJECT DeviceObject;
-	UNICODE_STRING DeviceName;
+	UNICODE_STRING DeviceName = RTL_CONSTANT_STRING(L"\\Device\\NamedPipe");
 	NTSTATUS Status;
 
-	DPRINT("Named Pipe File System Driver v0.1\n");
+	TRACE_(NPFS, "DriverEntry(%p, %wZ)", DriverObject, RegistryPath);
 
 	ASSERT (sizeof(NPFS_CONTEXT) <= FIELD_OFFSET(IRP, Tail.Overlay.DriverContext));
 	ASSERT (sizeof(NPFS_WAITER_ENTRY) <= FIELD_OFFSET(IRP, Tail.Overlay.DriverContext));
@@ -50,7 +59,6 @@
 
 	DriverObject->DriverUnload = NULL;
 
-	RtlInitUnicodeString(&DeviceName, L"\\Device\\NamedPipe");
 	Status = IoCreateDevice(DriverObject,
 		sizeof(NPFS_DEVICE_EXTENSION),
 		&DeviceName,
@@ -60,7 +68,7 @@
 		&DeviceObject);
 	if (!NT_SUCCESS(Status))
 	{
-		DPRINT("Failed to create named pipe device! (Status %x)\n", Status);
+		WARN_(NPFS, "Failed to create named pipe device! (Status 0x%08x)\n", Status);
 		return Status;
 	}
 
@@ -85,36 +93,4 @@
 	return STATUS_SUCCESS;
 }
 
-static VOID
-NpfsInitializeVcb(PNPFS_VCB Vcb)
-{
-	/* Initialize Volume Control Block, it's one per whole named pipe
-	   file system. */
-
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "NpfsInitializeVcb(), Vcb = %p\n", Vcb);
-
-	/* Zero-init whole VCB */
-	RtlZeroMemory(Vcb, sizeof(NPFS_VCB));
-
-	/* Initialize the common header */
-	Vcb->NodeTypeCode = NPFS_NODETYPE_VCB;
-	Vcb->NodeByteSize = sizeof(NPFS_VCB);
-}
-
-VOID NTAPI
-NpfsDbgPrint(ULONG Level, char *fmt, ...)
-{
-	va_list args;
-	char str[300];
-
-	if (!(NpfsDebugLevel & Level))
-		return;
-
-	va_start(args, fmt);
-	vsnprintf(str, sizeof(str), fmt, args);
-	va_end(args);
-	DbgPrint ("%4.4d: NPFS %s", PsGetCurrentProcessId(), str);
-}
-
-
 /* EOF */

Modified: trunk/reactos/drivers/filesystems/npfs_new/npfs.h
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/npfs_new/npfs.h?rev=32301&r1=32300&r2=32301&view=diff
==============================================================================
--- trunk/reactos/drivers/filesystems/npfs_new/npfs.h (original)
+++ trunk/reactos/drivers/filesystems/npfs_new/npfs.h Mon Feb 11 19:54:05 2008
@@ -6,10 +6,9 @@
 
 #include <stdarg.h>
 #include <stdio.h>
+#include <debug.h>
 
-/* Debug Levels */
-#define NPFS_DL_NONE      0x00000000
-#define NPFS_DL_API_TRACE 0x00000001
+#define DPFLTR_NPFS_ID DPFLTR_FASTFAT_ID
 
 /* Node type codes for NPFS */
 #define NPFS_NODETYPE_VCB 0x401
@@ -125,29 +124,24 @@
 
 #define PAGE_ROUND_UP(x) ( (((ULONG_PTR)x)%PAGE_SIZE) ? ((((ULONG_PTR)x)&(~(PAGE_SIZE-1)))+PAGE_SIZE) : ((ULONG_PTR)x) )
 
-NTSTATUS NTAPI NpfsCreate(PDEVICE_OBJECT DeviceObject, PIRP Irp);
-NTSTATUS NTAPI NpfsCreateNamedPipe(PDEVICE_OBJECT DeviceObject, PIRP Irp);
-NTSTATUS NTAPI NpfsCleanup(PDEVICE_OBJECT DeviceObject, PIRP Irp);
-NTSTATUS NTAPI NpfsClose(PDEVICE_OBJECT DeviceObject, PIRP Irp);
+DRIVER_DISPATCH NpfsCreate;
+DRIVER_DISPATCH NpfsCreateNamedPipe;
+DRIVER_DISPATCH NpfsCleanup;
+DRIVER_DISPATCH NpfsClose;
 
-NTSTATUS NTAPI NpfsRead(PDEVICE_OBJECT DeviceObject, PIRP Irp);
-NTSTATUS NTAPI NpfsWrite(PDEVICE_OBJECT DeviceObject, PIRP Irp);
+DRIVER_DISPATCH NpfsRead;
+DRIVER_DISPATCH NpfsWrite;
 
-NTSTATUS NTAPI NpfsFlushBuffers(PDEVICE_OBJECT DeviceObject, PIRP Irp);
+DRIVER_DISPATCH NpfsFlushBuffers;
 
-NTSTATUS NTAPI NpfsFileSystemControl(PDEVICE_OBJECT DeviceObject, PIRP Irp);
-NTSTATUS NTAPI NpfsDirectoryControl(PDEVICE_OBJECT DeviceObject, PIRP Irp);
+DRIVER_DISPATCH NpfsFileSystemControl;
+DRIVER_DISPATCH NpfsDirectoryControl;
 
-NTSTATUS NTAPI NpfsQueryInformation(PDEVICE_OBJECT DeviceObject, PIRP Irp);
-NTSTATUS NTAPI NpfsSetInformation(PDEVICE_OBJECT DeviceObject, PIRP Irp);
+DRIVER_DISPATCH NpfsQueryInformation;
+DRIVER_DISPATCH NpfsSetInformation;
 
-NTSTATUS NTAPI NpfsQueryVolumeInformation (PDEVICE_OBJECT DeviceObject, PIRP Irp);
+DRIVER_DISPATCH NpfsQueryVolumeInformation;
 
-NTSTATUS NTAPI
-DriverEntry(PDRIVER_OBJECT DriverObject,
-			PUNICODE_STRING RegistryPath);
-
-VOID NTAPI
-NpfsDbgPrint(ULONG Level, char *fmt, ...);
+DRIVER_INITIALIZE DriverEntry;
 
 #endif /* __DRIVERS_FS_NP_NPFS_H */

Modified: trunk/reactos/drivers/filesystems/npfs_new/npfs.rbuild
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/npfs_new/npfs.rbuild?rev=32301&r1=32300&r2=32301&view=diff
==============================================================================
--- trunk/reactos/drivers/filesystems/npfs_new/npfs.rbuild (original)
+++ trunk/reactos/drivers/filesystems/npfs_new/npfs.rbuild Mon Feb 11 19:54:05 2008
@@ -1,7 +1,6 @@
 <?xml version="1.0"?>
 <!DOCTYPE module SYSTEM "../../../tools/rbuild/project.dtd">
 <module name="npfs" type="kernelmodedriver" installbase="system32/drivers" installname="npfs.sys">
-	<include base="npfs">.</include>
 	<define name="__NO_CTYPE_INLINES" />
 	<library>ntoskrnl</library>
 	<library>hal</library>

Modified: trunk/reactos/drivers/filesystems/npfs_new/rw.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/npfs_new/rw.c?rev=32301&r1=32300&r2=32301&view=diff
==============================================================================
--- trunk/reactos/drivers/filesystems/npfs_new/rw.c (original)
+++ trunk/reactos/drivers/filesystems/npfs_new/rw.c Mon Feb 11 19:54:05 2008
@@ -10,18 +10,16 @@
 
 #include "npfs.h"
 
-//#define NDEBUG
-#include <debug.h>
-
 /* FUNCTIONS *****************************************************************/
 
 #ifndef NDEBUG
-VOID HexDump(PUCHAR Buffer, ULONG Length)
+VOID HexDump(IN PUCHAR Buffer,
+             IN SIZE_T Length)
 {
 	CHAR Line[65];
 	UCHAR ch;
 	const char Hex[] = "0123456789ABCDEF";
-	int i, j;
+	ULONG i, j;
 
 	DbgPrint("---------------\n");
 
@@ -44,13 +42,14 @@
 #endif
 
 NTSTATUS NTAPI
-NpfsRead(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsRead(IN PDEVICE_OBJECT DeviceObject,
+         IN PIRP Irp)
 {
+	TRACE_(NPFS, "NpfsRead()\n");
+
 	FsRtlEnterFileSystem();
 
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "NpfsRead()\n");
-
-	/* TODO: Implement */
+	UNIMPLEMENTED;
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 
 	FsRtlExitFileSystem();
@@ -59,13 +58,14 @@
 }
 
 NTSTATUS NTAPI
-NpfsWrite(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsWrite(IN PDEVICE_OBJECT DeviceObject,
+          IN PIRP Irp)
 {
+	TRACE_(NPFS, "NpfsWrite()\n");
+
 	FsRtlEnterFileSystem();
 
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "NpfsWrite()\n");
-
-	/* TODO: Implement */
+	UNIMPLEMENTED;
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 
 	FsRtlExitFileSystem();
@@ -74,13 +74,14 @@
 }
 
 NTSTATUS NTAPI
-NpfsFlushBuffers(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsFlushBuffers(IN PDEVICE_OBJECT DeviceObject,
+                 IN PIRP Irp)
 {
+	TRACE_(NPFS, "NpfsFlushBuffers()\n");
+
 	FsRtlEnterFileSystem();
 
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "NpfsFlushBuffers()\n");
-
-	/* TODO: Implement */
+	UNIMPLEMENTED;
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 
 	FsRtlExitFileSystem();

Modified: trunk/reactos/drivers/filesystems/npfs_new/volume.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/filesystems/npfs_new/volume.c?rev=32301&r1=32300&r2=32301&view=diff
==============================================================================
--- trunk/reactos/drivers/filesystems/npfs_new/volume.c (original)
+++ trunk/reactos/drivers/filesystems/npfs_new/volume.c Mon Feb 11 19:54:05 2008
@@ -10,19 +10,17 @@
 
 #include "npfs.h"
 
-//#define NDEBUG
-#include <debug.h>
-
 /* FUNCTIONS ****************************************************************/
 
 NTSTATUS NTAPI
-NpfsQueryVolumeInformation(PDEVICE_OBJECT DeviceObject, PIRP Irp)
+NpfsQueryVolumeInformation(IN PDEVICE_OBJECT DeviceObject,
+                           IN PIRP Irp)
 {
+	TRACE_(NPFS, "NpfsQueryVolumeInformation()\n");
+
 	FsRtlEnterFileSystem();
 
-	NpfsDbgPrint(NPFS_DL_API_TRACE, "NpfsQueryVolumeInformation()\n");
-
-	/* TODO: Implement */
+	UNIMPLEMENTED;
 	IoCompleteRequest(Irp, IO_NO_INCREMENT);
 
 	FsRtlExitFileSystem();




More information about the Ros-diffs mailing list