Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-05-01 11:39:58 +00:00
Code Issues Projects Releases Packages Wiki Activity

[FASTFAT]

Browse source 
Implement statistics counting for create operations.

svn path=/trunk/; revision=75938
This commit is contained in:
Pierre Schweitzer 2017-09-24 09:50:36 +00:00
parent 1fac1ca781
commit 10a7555f9e
2 changed files with 49 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

43
reactos/drivers/filesystems/fastfat/create.c
View file

@ -468,6 +468,8 @@ VfatCreateFile(
return STATUS_CANNOT_DELETE; return STATUS_CANNOT_DELETE;
} }
vfatAddToStat(DeviceExt, Fat.CreateHits, 1);
pFcb = DeviceExt->VolumeFcb; pFcb = DeviceExt->VolumeFcb;
if (pFcb->OpenHandleCount == 0) if (pFcb->OpenHandleCount == 0)
@ -486,6 +488,7 @@ VfatCreateFile(
FALSE); FALSE);
if (!NT_SUCCESS(Status)) if (!NT_SUCCESS(Status))
{ {
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return Status; return Status;
} }
} }
@ -493,6 +496,7 @@ VfatCreateFile(
vfatAttachFCBToFileObject(DeviceExt, pFcb, FileObject); vfatAttachFCBToFileObject(DeviceExt, pFcb, FileObject);
DeviceExt->OpenHandleCount++; DeviceExt->OpenHandleCount++;
pFcb->OpenHandleCount++; pFcb->OpenHandleCount++;
vfatAddToStat(DeviceExt, Fat.SuccessfulCreates, 1);
Irp->IoStatus.Information = FILE_OPENED; Irp->IoStatus.Information = FILE_OPENED;
return STATUS_SUCCESS; return STATUS_SUCCESS;
@ -566,6 +570,8 @@ VfatCreateFile(
/* Try opening the file. */ /* Try opening the file. */
if (!OpenTargetDir) if (!OpenTargetDir)
{ {
vfatAddToStat(DeviceExt, Fat.CreateHits, 1);
Status = VfatOpenFile(DeviceExt, &PathNameU, FileObject, RequestedDisposition, RequestedOptions, &ParentFcb); Status = VfatOpenFile(DeviceExt, &PathNameU, FileObject, RequestedDisposition, RequestedOptions, &ParentFcb);
} }
else else
@ -573,6 +579,8 @@ VfatCreateFile(
PVFATFCB TargetFcb; PVFATFCB TargetFcb;
LONG idx, FileNameLen; LONG idx, FileNameLen;
vfatAddToStat(DeviceExt, Fat.CreateHits, 1);
ParentFcb = (FileObject->RelatedFileObject != NULL) ? FileObject->RelatedFileObject->FsContext : NULL; ParentFcb = (FileObject->RelatedFileObject != NULL) ? FileObject->RelatedFileObject->FsContext : NULL;
if (ParentFcb) if (ParentFcb)
{ {
@ -654,6 +662,7 @@ VfatCreateFile(
if (!NT_SUCCESS(Status)) if (!NT_SUCCESS(Status))
{ {
VfatCloseFile(DeviceExt, FileObject); VfatCloseFile(DeviceExt, FileObject);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return Status; return Status;
} }
} }
@ -672,6 +681,15 @@ VfatCreateFile(
vfatReleaseFCB(DeviceExt, ParentFcb); vfatReleaseFCB(DeviceExt, ParentFcb);
} }
if (NT_SUCCESS(Status))
{
vfatAddToStat(DeviceExt, Fat.SuccessfulCreates, 1);
}
else
{
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
}
return Status; return Status;
} }
@ -689,12 +707,14 @@ VfatCreateFile(
{ {
vfatReleaseFCB(DeviceExt, ParentFcb); vfatReleaseFCB(DeviceExt, ParentFcb);
} }
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return Status; return Status;
} }
if (!NT_SUCCESS(Status) && ParentFcb == NULL) if (!NT_SUCCESS(Status) && ParentFcb == NULL)
{ {
DPRINT1("VfatOpenFile failed for '%wZ', status %x\n", &PathNameU, Status); DPRINT1("VfatOpenFile failed for '%wZ', status %x\n", &PathNameU, Status);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return Status; return Status;
} }
@ -712,6 +732,7 @@ VfatCreateFile(
if (TrailingBackslash) if (TrailingBackslash)
{ {
vfatReleaseFCB(DeviceExt, ParentFcb); vfatReleaseFCB(DeviceExt, ParentFcb);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return STATUS_OBJECT_NAME_INVALID; return STATUS_OBJECT_NAME_INVALID;
} }
Attributes |= FILE_ATTRIBUTE_ARCHIVE; Attributes |= FILE_ATTRIBUTE_ARCHIVE;
@ -726,6 +747,7 @@ VfatCreateFile(
if (!NT_SUCCESS(Status)) if (!NT_SUCCESS(Status))
{ {
vfatReleaseFCB(DeviceExt, pFcb); vfatReleaseFCB(DeviceExt, pFcb);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return Status; return Status;
} }
@ -745,12 +767,14 @@ VfatCreateFile(
} }
else else
{ {
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return Status; return Status;
} }
} }
else else
{ {
vfatReleaseFCB(DeviceExt, ParentFcb); vfatReleaseFCB(DeviceExt, ParentFcb);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return Status; return Status;
} }
} }
@ -769,9 +793,11 @@ VfatCreateFile(
VfatCloseFile(DeviceExt, FileObject); VfatCloseFile(DeviceExt, FileObject);
if (TrailingBackslash && !vfatFCBIsDirectory(pFcb)) if (TrailingBackslash && !vfatFCBIsDirectory(pFcb))
{ {
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return STATUS_OBJECT_NAME_INVALID; return STATUS_OBJECT_NAME_INVALID;
} }
Irp->IoStatus.Information = FILE_EXISTS; Irp->IoStatus.Information = FILE_EXISTS;
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return STATUS_OBJECT_NAME_COLLISION; return STATUS_OBJECT_NAME_COLLISION;
} }
@ -785,6 +811,7 @@ VfatCreateFile(
if (!NT_SUCCESS(Status)) if (!NT_SUCCESS(Status))
{ {
VfatCloseFile(DeviceExt, FileObject); VfatCloseFile(DeviceExt, FileObject);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return Status; return Status;
} }
} }
@ -796,17 +823,20 @@ VfatCreateFile(
vfatFCBIsDirectory(pFcb)) vfatFCBIsDirectory(pFcb))
{ {
VfatCloseFile (DeviceExt, FileObject); VfatCloseFile (DeviceExt, FileObject);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return STATUS_FILE_IS_A_DIRECTORY; return STATUS_FILE_IS_A_DIRECTORY;
} }
if (BooleanFlagOn(RequestedOptions, FILE_DIRECTORY_FILE) && if (BooleanFlagOn(RequestedOptions, FILE_DIRECTORY_FILE) &&
!vfatFCBIsDirectory(pFcb)) !vfatFCBIsDirectory(pFcb))
{ {
VfatCloseFile (DeviceExt, FileObject); VfatCloseFile (DeviceExt, FileObject);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return STATUS_NOT_A_DIRECTORY; return STATUS_NOT_A_DIRECTORY;
} }
if (TrailingBackslash && !vfatFCBIsDirectory(pFcb)) if (TrailingBackslash && !vfatFCBIsDirectory(pFcb))
{ {
VfatCloseFile (DeviceExt, FileObject); VfatCloseFile (DeviceExt, FileObject);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return STATUS_OBJECT_NAME_INVALID; return STATUS_OBJECT_NAME_INVALID;
} }
#ifndef USE_ROS_CC_AND_FS #ifndef USE_ROS_CC_AND_FS
@ -823,6 +853,7 @@ VfatCreateFile(
DPRINT1("%d %d %d\n", Stack->Parameters.Create.SecurityContext->DesiredAccess & FILE_WRITE_DATA, DPRINT1("%d %d %d\n", Stack->Parameters.Create.SecurityContext->DesiredAccess & FILE_WRITE_DATA,
RequestedDisposition == FILE_OVERWRITE, RequestedDisposition == FILE_OVERWRITE_IF); RequestedDisposition == FILE_OVERWRITE, RequestedDisposition == FILE_OVERWRITE_IF);
VfatCloseFile (DeviceExt, FileObject); VfatCloseFile (DeviceExt, FileObject);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return (BooleanFlagOn(RequestedOptions, FILE_DELETE_ON_CLOSE)) ? STATUS_CANNOT_DELETE return (BooleanFlagOn(RequestedOptions, FILE_DELETE_ON_CLOSE)) ? STATUS_CANNOT_DELETE
: STATUS_SHARING_VIOLATION; : STATUS_SHARING_VIOLATION;
} }
@ -844,6 +875,7 @@ VfatCreateFile(
if(!BooleanFlagOn(pFcb->Flags, FCB_IS_PAGE_FILE)) if(!BooleanFlagOn(pFcb->Flags, FCB_IS_PAGE_FILE))
{ {
VfatCloseFile(DeviceExt, FileObject); VfatCloseFile(DeviceExt, FileObject);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return STATUS_INVALID_PARAMETER; return STATUS_INVALID_PARAMETER;
} }
} }
@ -857,6 +889,7 @@ VfatCreateFile(
if (BooleanFlagOn(pFcb->Flags, FCB_IS_PAGE_FILE)) if (BooleanFlagOn(pFcb->Flags, FCB_IS_PAGE_FILE))
{ {
VfatCloseFile(DeviceExt, FileObject); VfatCloseFile(DeviceExt, FileObject);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return STATUS_INVALID_PARAMETER; return STATUS_INVALID_PARAMETER;
} }
} }
@ -881,6 +914,7 @@ VfatCreateFile(
if (!NT_SUCCESS (Status)) if (!NT_SUCCESS (Status))
{ {
VfatCloseFile(DeviceExt, FileObject); VfatCloseFile(DeviceExt, FileObject);
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
return Status; return Status;
} }
} }
@ -938,6 +972,15 @@ VfatCreateFile(
/* FIXME : test write access if requested */ /* FIXME : test write access if requested */
if (NT_SUCCESS(Status))
{
vfatAddToStat(DeviceExt, Fat.SuccessfulCreates, 1);
}
else
{
vfatAddToStat(DeviceExt, Fat.FailedCreates, 1);
}
return Status; return Status;
} }

6
reactos/drivers/filesystems/fastfat/vfat.h
View file

@ -602,6 +602,12 @@ vfatVolumeIsFatX(PDEVICE_EXTENSION DeviceExt)
return BooleanFlagOn(DeviceExt->Flags, VCB_IS_FATX); return BooleanFlagOn(DeviceExt->Flags, VCB_IS_FATX);
} }
#define vfatAddToStat(Vcb, Stat, Inc) \
{ \
PSTATISTICS Stats = &(Vcb)->Statistics[KeGetCurrentProcessorNumber() % VfatGlobalData->NumberProcessors]; \
Stats->Stat += Inc; \
}
/* blockdev.c */ /* blockdev.c */
NTSTATUS NTSTATUS