Ros-diffs

ros-diffs@reactos.org

81390 discussions

[janderwald] 55353: [HIDPARSER] - Rewrite hidparser so that it support multiple top level collections - Add support for flat collection context meaning all context structures are aligned in after e...
by janderwald＠svn.reactos.org 31 Jan '12

31 Jan '12

Author: janderwald Date: Tue Jan 31 16:19:33 2012 New Revision: 55353 URL: http://svn.reactos.org/svn/reactos?rev=55353&view=rev Log: [HIDPARSER] - Rewrite hidparser so that it support multiple top level collections - Add support for flat collection context meaning all context structures are aligned in after each other. The reason this is required is that usermode hid.dll will access the same structure and thus can't use kernel pointers. User mode hid.dll is used to support keyboard with custom keys (for example search key, email key etc) Added: branches/usb-bringup-trunk/lib/drivers/hidparser/context.c (with props) Modified: branches/usb-bringup-trunk/drivers/hid/hidparse/hidparse.c branches/usb-bringup-trunk/drivers/usb/usbd/usbd.c branches/usb-bringup-trunk/lib/drivers/hidparser/CMakeLists.txt branches/usb-bringup-trunk/lib/drivers/hidparser/api.c branches/usb-bringup-trunk/lib/drivers/hidparser/hidparser.c branches/usb-bringup-trunk/lib/drivers/hidparser/hidparser.h branches/usb-bringup-trunk/lib/drivers/hidparser/parser.c branches/usb-bringup-trunk/lib/drivers/hidparser/parser.h Modified: branches/usb-bringup-trunk/drivers/hid/hidparse/hidparse.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/hid/h… ============================================================================== --- branches/usb-bringup-trunk/drivers/hid/hidparse/hidparse.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/hid/hidparse/hidparse.c [iso-8859-1] Tue Jan 31 16:19:33 2012 @@ -93,7 +93,7 @@ // // init parser // - HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, NULL, &Parser); + HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, &Parser); // // free collection @@ -114,12 +114,12 @@ // // init parser // - HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, PreparsedData, &Parser); + HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, &Parser); // // get caps // - return HidParser_GetCaps(&Parser, Capabilities); + return HidParser_GetCaps(&Parser, PreparsedData, Capabilities); } NTSTATUS @@ -130,30 +130,17 @@ IN POOL_TYPE PoolType, OUT PHIDP_DEVICE_DESC DeviceDescription) { - PHID_PARSER Parser; - HIDPARSER_STATUS Status; - - // - // first allocate the parser - // - Status = HidParser_AllocateParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, &Parser); - if (Status != HIDPARSER_STATUS_SUCCESS) - { - // - // not enough memory - // - return STATUS_INSUFFICIENT_RESOURCES; - } + HID_PARSER Parser; + + // + // init parser + // + HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, &Parser); // // get description; // - Status = HidParser_GetCollectionDescription(Parser, ReportDesc, DescLength, PoolType, DeviceDescription); - - // - // FIXME parser memory leak - // - return Status; + return HidParser_GetCollectionDescription(&Parser, ReportDesc, DescLength, PoolType, DeviceDescription); } HIDAPI @@ -174,13 +161,13 @@ // // init parser // - HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, PreparsedData, &Parser); + HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, &Parser); // // get usage length // - return HidParser_MaxUsageListLength(&Parser, ReportType, UsagePage); + return HidParser_MaxUsageListLength(&Parser, PreparsedData, ReportType, UsagePage); } HIDAPI @@ -205,12 +192,12 @@ // // init parser // - HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, PreparsedData, &Parser); + HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, &Parser); // // get value caps // - return HidParser_GetSpecificValueCaps(&Parser, ReportType, UsagePage, LinkCollection, Usage, ValueCaps, ValueCapsLength); + return HidParser_GetSpecificValueCaps(&Parser, PreparsedData, ReportType, UsagePage, LinkCollection, Usage, ValueCaps, ValueCapsLength); } HIDAPI @@ -236,12 +223,12 @@ // // init parser // - HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, PreparsedData, &Parser); + HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, &Parser); // // get usages // - return HidParser_GetUsages(&Parser, ReportType, UsagePage, LinkCollection, UsageList, UsageLength, Report, ReportLength); + return HidParser_GetUsages(&Parser, PreparsedData, ReportType, UsagePage, LinkCollection, UsageList, UsageLength, Report, ReportLength); } @@ -311,12 +298,12 @@ // // init parser // - HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, PreparsedData, &Parser); + HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, &Parser); // // get scaled usage value // - return HidParser_GetScaledUsageValue(&Parser, ReportType, UsagePage, LinkCollection, Usage, UsageValue, Report, ReportLength); + return HidParser_GetScaledUsageValue(&Parser, PreparsedData, ReportType, UsagePage, LinkCollection, Usage, UsageValue, Report, ReportLength); } HIDAPI @@ -335,7 +322,7 @@ // // init parser // - HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, NULL, &Parser); + HidParser_InitParser(AllocFunction, FreeFunction, ZeroFunction, CopyFunction, DebugFunction, &Parser); // // translate usage pages Modified: branches/usb-bringup-trunk/drivers/usb/usbd/usbd.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/usb/u… ============================================================================== --- branches/usb-bringup-trunk/drivers/usb/usbd/usbd.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/usb/usbd/usbd.c [iso-8859-1] Tue Jan 31 16:19:33 2012 @@ -324,8 +324,8 @@ InterfaceList[InterfaceCount].InterfaceDescriptor != NULL; InterfaceCount++) { - UrbSize += sizeof(USBD_INTERFACE_INFORMATION); - UrbSize += (InterfaceList[InterfaceCount].InterfaceDescriptor->bNumEndpoints - 1) * sizeof(USBD_PIPE_INFORMATION); + UrbSize += FIELD_OFFSET(USBD_INTERFACE_INFORMATION, Pipes); + UrbSize += (InterfaceList[InterfaceCount].InterfaceDescriptor->bNumEndpoints) * sizeof(USBD_PIPE_INFORMATION); } UrbSize += sizeof(URB) + sizeof(USBD_INTERFACE_INFORMATION); Modified: branches/usb-bringup-trunk/lib/drivers/hidparser/CMakeLists.txt URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/lib/drivers/h… ============================================================================== --- branches/usb-bringup-trunk/lib/drivers/hidparser/CMakeLists.txt [iso-8859-1] (original) +++ branches/usb-bringup-trunk/lib/drivers/hidparser/CMakeLists.txt [iso-8859-1] Tue Jan 31 16:19:33 2012 @@ -4,6 +4,7 @@ -DNDEBUG=1) list(APPEND SOURCE + context.c hidparser.c parser.c api.c) Modified: branches/usb-bringup-trunk/lib/drivers/hidparser/api.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/lib/drivers/h… ============================================================================== --- branches/usb-bringup-trunk/lib/drivers/hidparser/api.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/lib/drivers/hidparser/api.c [iso-8859-1] Tue Jan 31 16:19:33 2012 @@ -31,143 +31,9 @@ 150,158,159,128,136,177,178,176,142,152,173,140 }; - - -ULONG -HidParser_NumberOfTopCollections( - IN PHID_PARSER Parser) -{ - PHID_PARSER_CONTEXT ParserContext; - - // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity checks - // - ASSERT(ParserContext); - ASSERT(ParserContext->RootCollection); - ASSERT(ParserContext->RootCollection->NodeCount); - - // - // number of top collections - // - return ParserContext->RootCollection->NodeCount; -} - -PHID_COLLECTION -HidParser_GetCollection( - IN PHID_PARSER Parser, - IN ULONG CollectionNumber) -{ - PHID_PARSER_CONTEXT ParserContext; - - // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity checks - // - ASSERT(ParserContext); - ASSERT(ParserContext->RootCollection); - ASSERT(ParserContext->RootCollection->NodeCount); - - // - // is collection index out of bounds - // - if (CollectionNumber < ParserContext->RootCollection->NodeCount) - { - // - // valid collection - // - return ParserContext->RootCollection->Nodes[CollectionNumber]; - } - - // - // no such collection - // - Parser->Debug("HIDPARSE] No such collection %lu\n", CollectionNumber); - return NULL; -} - -PHID_REPORT -HidParser_GetReportInCollection( - PHID_COLLECTION Collection, - IN UCHAR ReportType) -{ - ULONG Index; - PHID_REPORT Report; - - // - // search in local array - // - for(Index = 0; Index < Collection->ReportCount; Index++) - { - if (Collection->Reports[Index]->Type == ReportType) - { - // - // found report - // - return Collection->Reports[Index]; - } - } - - // - // search in local array - // - for(Index = 0; Index < Collection->NodeCount; Index++) - { - Report = HidParser_GetReportInCollection(Collection->Nodes[Index], ReportType); - if (Report) - { - // - // found report - // - return Report; - } - } - - // - // not found - // - return NULL; -} - -PHID_REPORT -HidParser_GetReportByType( - IN PHID_PARSER Parser, - IN ULONG CollectionIndex, - IN UCHAR ReportType) -{ - PHID_COLLECTION Collection; - - // - // find collection - // - Collection = HidParser_GetCollection(Parser, CollectionIndex); - if (!Collection) - { - // - // no such collection - // - ASSERT(FALSE); - return NULL; - } - - // - // search report - // - return HidParser_GetReportInCollection(Collection, ReportType); -} - HIDPARSER_STATUS HidParser_GetCollectionUsagePage( - IN PHID_PARSER Parser, - IN ULONG CollectionIndex, + IN PVOID CollectionContext, OUT PUSHORT Usage, OUT PUSHORT UsagePage) { @@ -176,7 +42,7 @@ // // find collection // - Collection = HidParser_GetCollection(Parser, CollectionIndex); + Collection = HidParser_GetCollectionFromContext(CollectionContext); if (!Collection) { // @@ -195,33 +61,16 @@ ULONG HidParser_GetReportLength( - IN PHID_PARSER Parser, - IN ULONG CollectionIndex, - IN ULONG ReportType) -{ - PHID_PARSER_CONTEXT ParserContext; + IN PVOID CollectionContext, + IN UCHAR ReportType) +{ PHID_REPORT Report; ULONG ReportLength; // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity checks - // - ASSERT(ParserContext); - - // - // FIXME support multiple top collecions - // - ASSERT(ParserContext->RootCollection->NodeCount == 1); - - // // get first report // - Report = HidParser_GetReportByType(Parser, CollectionIndex, ReportType); + Report = HidParser_GetReportInCollection(CollectionContext, ReportType); if (!Report) { // @@ -249,56 +98,17 @@ return ReportLength; } -UCHAR -HidParser_IsReportIDUsed( - IN PHID_PARSER Parser) -{ - PHID_PARSER_CONTEXT ParserContext; - - // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity checks - // - ASSERT(ParserContext); - - // - // return flag - // - return ParserContext->UseReportIDs; -} - ULONG HidParser_GetReportItemCountFromReportType( - IN PHID_PARSER Parser, - IN ULONG CollectionIndex, - IN ULONG ReportType) -{ - PHID_PARSER_CONTEXT ParserContext; + IN PVOID CollectionContext, + IN UCHAR ReportType) +{ PHID_REPORT Report; // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity checks - // - ASSERT(ParserContext); - - // - // FIXME support multiple top collecions - // - ASSERT(ParserContext->RootCollection->NodeCount == 1); - - // // get report // - Report = HidParser_GetReportByType(Parser, CollectionIndex, ReportType); + Report = HidParser_GetReportInCollection(CollectionContext, ReportType); if (!Report) { // @@ -316,35 +126,18 @@ ULONG HidParser_GetReportItemTypeCountFromReportType( - IN PHID_PARSER Parser, - IN ULONG CollectionIndex, - IN ULONG ReportType, + IN PVOID CollectionContext, + IN UCHAR ReportType, IN ULONG bData) { - PHID_PARSER_CONTEXT ParserContext; ULONG Index; PHID_REPORT Report; ULONG ItemCount = 0; // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity checks - // - ASSERT(ParserContext); - - // - // FIXME support multiple top collecions - // - ASSERT(ParserContext->RootCollection->NodeCount == 1); - - // // get report // - Report = HidParser_GetReportByType(Parser, CollectionIndex, ReportType); + Report = HidParser_GetReportInCollection(CollectionContext, ReportType); if (!Report) { // @@ -361,14 +154,14 @@ // // check item type // - if (Report->Items[Index]->HasData && bData == TRUE) + if (Report->Items[Index].HasData && bData == TRUE) { // // found data item // ItemCount++; } - else if (Report->Items[Index]->HasData == FALSE && bData == FALSE) + else if (Report->Items[Index].HasData == FALSE && bData == FALSE) { // // found value item @@ -383,85 +176,6 @@ return ItemCount; } -ULONG -HidParser_GetContextSize( - IN PHID_PARSER Parser, - IN ULONG CollectionIndex) -{ - // - // FIXME the context must contain all parsed info - // - return sizeof(HID_PARSER_CONTEXT); -} - -VOID -HidParser_FreeContext( - IN PHID_PARSER Parser, - IN PUCHAR Context, - IN ULONG ContextLength) -{ - // - // FIXME implement freeing of parsed info - // -} - -HIDPARSER_STATUS -HidParser_AllocateParser( - IN PHIDPARSER_ALLOC_FUNCTION AllocFunction, - IN PHIDPARSER_FREE_FUNCTION FreeFunction, - IN PHIDPARSER_ZERO_FUNCTION ZeroFunction, - IN PHIDPARSER_COPY_FUNCTION CopyFunction, - IN PHIDPARSER_DEBUG_FUNCTION DebugFunction, - OUT PHID_PARSER *OutParser) -{ - PHID_PARSER Parser; - PHID_PARSER_CONTEXT ParserContext; - - // - // allocate - // - Parser = (PHID_PARSER)AllocFunction(sizeof(HID_PARSER)); - if (!Parser) - { - // - // no memory - // - return HIDPARSER_STATUS_INSUFFICIENT_RESOURCES; - } - - // - // allocate parser context - // - ParserContext = (PHID_PARSER_CONTEXT)AllocFunction(sizeof(HID_PARSER_CONTEXT)); - if (!ParserContext) - { - // - // no memory - // - FreeFunction(Parser); - return HIDPARSER_STATUS_INSUFFICIENT_RESOURCES; - } - - - // - // init parser - // - Parser->Alloc = AllocFunction; - Parser->Free = FreeFunction; - Parser->Zero = ZeroFunction; - Parser->Copy = CopyFunction; - Parser->Debug = DebugFunction; - Parser->ParserContext = ParserContext; - - // - // store result - // - *OutParser = Parser; - // - // success - // - return HIDPARSER_STATUS_SUCCESS; -} VOID HidParser_InitParser( @@ -470,7 +184,6 @@ IN PHIDPARSER_ZERO_FUNCTION ZeroFunction, IN PHIDPARSER_COPY_FUNCTION CopyFunction, IN PHIDPARSER_DEBUG_FUNCTION DebugFunction, - IN PVOID ParserContext, OUT PHID_PARSER Parser) { Parser->Alloc = AllocFunction; @@ -478,85 +191,23 @@ Parser->Zero = ZeroFunction; Parser->Copy = CopyFunction; Parser->Debug = DebugFunction; - Parser->ParserContext = ParserContext; -} - -ULONG -HidParser_GetCollectionCount( - IN PHID_COLLECTION Collection) -{ - ULONG Index; - ULONG Count = Collection->NodeCount; - - for(Index = 0; Index < Collection->NodeCount; Index++) - { - // - // count collection for sub nodes - // - Count += HidParser_GetCollectionCount(Collection->Nodes[Index]); - } - - // - // done - // - return Count; -} - -ULONG -HidParser_GetTotalCollectionCount( - IN PHID_PARSER Parser) -{ - PHID_PARSER_CONTEXT ParserContext; - - // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity check - // - ASSERT(ParserContext); - ASSERT(ParserContext->RootCollection); - - // - // count collections - // - return HidParser_GetCollectionCount(ParserContext->RootCollection); } ULONG HidParser_GetMaxUsageListLengthWithReportAndPage( - IN PHID_PARSER Parser, - IN ULONG CollectionIndex, - IN ULONG ReportType, + IN PVOID CollectionContext, + IN UCHAR ReportType, IN USAGE UsagePage OPTIONAL) { - PHID_PARSER_CONTEXT ParserContext; ULONG Index; PHID_REPORT Report; ULONG ItemCount = 0; USHORT CurrentUsagePage; // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity checks - // - ASSERT(ParserContext); - - // - // FIXME support multiple top collecions - // - ASSERT(ParserContext->RootCollection->NodeCount == 1); - - // // get report // - Report = HidParser_GetReportByType(Parser, CollectionIndex, ReportType); + Report = HidParser_GetReportInCollection(CollectionContext, ReportType); if (!Report) { // @@ -570,8 +221,8 @@ // // check usage page // - CurrentUsagePage = (Report->Items[Index]->UsageMinimum >> 16); - if (CurrentUsagePage == UsagePage && Report->Items[Index]->HasData) + CurrentUsagePage = (Report->Items[Index].UsageMinimum >> 16); + if (CurrentUsagePage == UsagePage && Report->Items[Index].HasData) { // // found item @@ -589,14 +240,13 @@ HIDPARSER_STATUS HidParser_GetSpecificValueCapsWithReport( IN PHID_PARSER Parser, - IN ULONG CollectionIndex, - IN ULONG ReportType, + IN PVOID CollectionContext, + IN UCHAR ReportType, IN USHORT UsagePage, IN USHORT Usage, OUT PHIDP_VALUE_CAPS ValueCaps, IN OUT PULONG ValueCapsLength) { - PHID_PARSER_CONTEXT ParserContext; ULONG Index; PHID_REPORT Report; ULONG ItemCount = 0; @@ -604,24 +254,9 @@ USHORT CurrentUsage; // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity checks - // - ASSERT(ParserContext); - - // - // FIXME support multiple top collecions - // - ASSERT(ParserContext->RootCollection->NodeCount == 1); - - // // get report // - Report = HidParser_GetReportByType(Parser, CollectionIndex, ReportType); + Report = HidParser_GetReportInCollection(CollectionContext, ReportType); if (!Report) { // @@ -635,8 +270,8 @@ // // check usage page // - CurrentUsagePage = (Report->Items[Index]->UsageMinimum >> 16); - CurrentUsage = (Report->Items[Index]->UsageMinimum & 0xFFFF); + CurrentUsagePage = (Report->Items[Index].UsageMinimum >> 16); + CurrentUsage = (Report->Items[Index].UsageMinimum & 0xFFFF); if ((Usage == CurrentUsage && UsagePage == CurrentUsagePage) || (Usage == 0 && UsagePage == CurrentUsagePage) || (Usage == CurrentUsage && UsagePage == 0) || (Usage == 0 && UsagePage == 0)) { @@ -655,10 +290,10 @@ // ValueCaps[ItemCount].UsagePage = CurrentUsagePage; ValueCaps[ItemCount].ReportID = Report->ReportID; - ValueCaps[ItemCount].LogicalMin = Report->Items[Index]->Minimum; - ValueCaps[ItemCount].LogicalMax = Report->Items[Index]->Maximum; - ValueCaps[ItemCount].IsAbsolute = !Report->Items[Index]->Relative; - ValueCaps[ItemCount].BitSize = Report->Items[Index]->BitCount; + ValueCaps[ItemCount].LogicalMin = Report->Items[Index].Minimum; + ValueCaps[ItemCount].LogicalMax = Report->Items[Index].Maximum; + ValueCaps[ItemCount].IsAbsolute = !Report->Items[Index].Relative; + ValueCaps[ItemCount].BitSize = Report->Items[Index].BitCount; // // FIXME: FILLMEIN @@ -695,15 +330,14 @@ HIDPARSER_STATUS HidParser_GetUsagesWithReport( IN PHID_PARSER Parser, - IN ULONG CollectionIndex, - IN ULONG ReportType, + IN PVOID CollectionContext, + IN UCHAR ReportType, IN USAGE UsagePage, OUT USAGE *UsageList, IN OUT PULONG UsageLength, IN PCHAR ReportDescriptor, IN ULONG ReportDescriptorLength) { - PHID_PARSER_CONTEXT ParserContext; ULONG Index; PHID_REPORT Report; ULONG ItemCount = 0; @@ -713,24 +347,9 @@ ULONG Data; // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity checks - // - ASSERT(ParserContext); - - // - // FIXME support multiple top collecions - // - ASSERT(ParserContext->RootCollection->NodeCount == 1); - - // // get report // - Report = HidParser_GetReportByType(Parser, CollectionIndex, ReportType); + Report = HidParser_GetReportInCollection(CollectionContext, ReportType); if (!Report) { // @@ -752,7 +371,7 @@ // // get report item // - ReportItem = Report->Items[Index]; + ReportItem = &Report->Items[Index]; // // does it have data @@ -843,15 +462,14 @@ HIDPARSER_STATUS HidParser_GetScaledUsageValueWithReport( IN PHID_PARSER Parser, - IN ULONG CollectionIndex, - IN ULONG ReportType, + IN PVOID CollectionContext, + IN UCHAR ReportType, IN USAGE UsagePage, IN USAGE Usage, OUT PLONG UsageValue, IN PCHAR ReportDescriptor, IN ULONG ReportDescriptorLength) { - PHID_PARSER_CONTEXT ParserContext; ULONG Index; PHID_REPORT Report; USHORT CurrentUsagePage; @@ -859,24 +477,9 @@ ULONG Data; // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // sanity checks - // - ASSERT(ParserContext); - - // - // FIXME support multiple top collecions - // - ASSERT(ParserContext->RootCollection->NodeCount == 1); - - // // get report // - Report = HidParser_GetReportByType(Parser, CollectionIndex, ReportType); + Report = HidParser_GetReportInCollection(CollectionContext, ReportType); if (!Report) { // @@ -898,7 +501,7 @@ // // get report item // - ReportItem = Report->Items[Index]; + ReportItem = &Report->Items[Index]; // // check usage page @@ -1034,7 +637,6 @@ HIDPARSER_STATUS HidParser_TranslateUsage( IN PHID_PARSER Parser, - IN ULONG CollectionIndex, IN USAGE Usage, IN HIDP_KEYBOARD_DIRECTION KeyAction, IN OUT PHIDP_KEYBOARD_MODIFIER_STATE ModifierState, @@ -1066,15 +668,3 @@ // return HIDPARSER_STATUS_SUCCESS; } - -ULONG -HidParser_GetCollectionNumberFromParserContext( - IN PHID_PARSER Parser) -{ - PHID_PARSER_CONTEXT Context = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - // - // get parser context - // - return Context->CollectionIndex; -} Added: branches/usb-bringup-trunk/lib/drivers/hidparser/context.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/lib/drivers/h… ============================================================================== --- branches/usb-bringup-trunk/lib/drivers/hidparser/context.c (added) +++ branches/usb-bringup-trunk/lib/drivers/hidparser/context.c [iso-8859-1] Tue Jan 31 16:19:33 2012 @@ -1,0 +1,344 @@ +/* + * PROJECT: ReactOS HID Parser Library + * LICENSE: GPL - See COPYING in the top level directory + * FILE: lib/drivers/hidparser/context.c + * PURPOSE: HID Parser + * PROGRAMMERS: + * Michael Martin (michael.martin(a)reactos.org) + * Johannes Anderwald (johannes.anderwald(a)reactos.org) + */ + + +#include "parser.h" + +typedef struct +{ + ULONG Size; + union + { + UCHAR RawData[1]; + }; +}HID_COLLECTION_CONTEXT, *PHID_COLLECTION_CONTEXT; + +ULONG +HidParser_CalculateCollectionSize( + IN PHID_COLLECTION Collection) +{ + ULONG Size = 0, Index; + + Size = sizeof(HID_COLLECTION); + + // + // add size required for the number of report items + // + for(Index = 0; Index < Collection->ReportCount; Index++) + { + // + // get report size + // + ASSERT(Collection->Reports[Index]->ItemCount); + Size += sizeof(HID_REPORT) + Collection->Reports[Index]->ItemCount * sizeof(HID_REPORT_ITEM); + } + + // + // calculate size for sub collections + // + for(Index = 0; Index < Collection->NodeCount; Index++) + { + Size += HidParser_CalculateCollectionSize(Collection->Nodes[Index]); + } + + // + // append size for the offset + // + Size += (Collection->ReportCount + Collection->NodeCount) * sizeof(ULONG); + + // + // done + // + return Size; +} + +ULONG +HidParser_CalculateContextSize( + IN PHID_COLLECTION Collection) +{ + ULONG Size; + + // + // minimum size is the size of the collection + // + Size = HidParser_CalculateCollectionSize(Collection); + + // + // append collection context size + // + Size += sizeof(HID_COLLECTION_CONTEXT); + return Size; +} + +ULONG +HidParser_StoreCollection( + IN PHID_PARSER Parser, + IN PHID_COLLECTION Collection, + IN PHID_COLLECTION_CONTEXT CollectionContext, + IN ULONG CurrentOffset) +{ + ULONG Index; + ULONG ReportSize; + ULONG InitialOffset; + ULONG CollectionSize; + PHID_COLLECTION TargetCollection; + + // + // backup initial offset + // + InitialOffset = CurrentOffset; + + // + // get target collection + // + TargetCollection = (PHID_COLLECTION)(&CollectionContext->RawData[CurrentOffset]); + + // + // first copy the collection details + // + Parser->Copy(TargetCollection, Collection, sizeof(HID_COLLECTION)); + + // + // calulcate collection size + // + CollectionSize = sizeof(HID_COLLECTION) + sizeof(ULONG) * (Collection->ReportCount + Collection->NodeCount); + + // + // increase offset + // + CurrentOffset += CollectionSize; + + // + // sanity check + // + ASSERT(CurrentOffset < CollectionContext->Size); + + // + // first store the report items + // + for(Index = 0; Index < Collection->ReportCount; Index++) + { + // + // calculate report size + // + ReportSize = sizeof(HID_REPORT) + Collection->Reports[Index]->ItemCount * sizeof(HID_REPORT_ITEM); + + // + // sanity check + // + ASSERT(CurrentOffset + ReportSize < CollectionContext->Size); + + // + // copy report item + // + Parser->Copy(&CollectionContext->RawData[CurrentOffset], Collection->Reports[Index], ReportSize); + + // + // store offset to report item + // + TargetCollection->Offsets[Index] = CurrentOffset; + + // + // move to next offset + // + CurrentOffset += ReportSize; + } + + ASSERT(CurrentOffset <= CollectionContext->Size); + + // + // now store the sub collections + // + for(Index = 0; Index < Collection->NodeCount; Index++) + { + // + // store offset + // + TargetCollection->Offsets[Collection->NodeCount + Index] = CurrentOffset; + + // + // store sub collections + // + CurrentOffset += HidParser_StoreCollection(Parser, Collection->Nodes[Index], CollectionContext, CurrentOffset); + + // + // sanity check + // + ASSERT(CurrentOffset < CollectionContext->Size); + } + + // + // return size of collection + // + return CurrentOffset - InitialOffset; +} + +HIDPARSER_STATUS +HidParser_BuildCollectionContext( + IN PHID_PARSER Parser, + IN PHID_COLLECTION RootCollection, + IN PVOID Context, + IN ULONG ContextSize) +{ + PHID_COLLECTION_CONTEXT CollectionContext; + ULONG CollectionSize; + + // + // init context + // + CollectionContext = (PHID_COLLECTION_CONTEXT)Context; + CollectionContext->Size = ContextSize; + + // + // store collections + // + CollectionSize = HidParser_StoreCollection(Parser, RootCollection, CollectionContext, 0); + + // + // sanity check + // + ASSERT(CollectionSize + sizeof(HID_COLLECTION_CONTEXT) == ContextSize); + + DPRINT1("CollectionContext %p\n", CollectionContext); + DPRINT1("CollectionContext RawData %p\n", CollectionContext->RawData); + DPRINT1("CollectionContext Size %lu\n", CollectionContext->Size); + + // + // done + // + return HIDPARSER_STATUS_SUCCESS; +} + +PHID_REPORT +HidParser_SearchReportInCollection( + IN PHID_COLLECTION_CONTEXT CollectionContext, + IN PHID_COLLECTION Collection, + IN UCHAR ReportType) +{ + ULONG Index; + PHID_REPORT Report; + PHID_COLLECTION SubCollection; + + // + // search first in local array + // + for(Index = 0; Index < Collection->ReportCount; Index++) + { + // + // get report + // + Report = (PHID_REPORT)(CollectionContext->RawData + Collection->Offsets[Index]); + if (Report->Type == ReportType) + { + // + // found report + // + return Report; + } + } + + // + // now search in sub collections + // + for(Index = 0; Index < Collection->NodeCount; Index++) + { + // + // get collection + // + SubCollection = (PHID_COLLECTION)(CollectionContext->RawData + Collection->Offsets[Collection->NodeCount + Index]); + + // + // recursively search collection + // + Report = HidParser_SearchReportInCollection(CollectionContext, SubCollection, ReportType); + if (Report) + { + // + // found report + // + return Report; + } + } + + // + // not found + // + return NULL; +} + +PHID_REPORT +HidParser_GetReportInCollection( + IN PVOID Context, + IN UCHAR ReportType) +{ + PHID_COLLECTION_CONTEXT CollectionContext = (PHID_COLLECTION_CONTEXT)Context; + + // + // done + // + return HidParser_SearchReportInCollection(CollectionContext, (PHID_COLLECTION)&CollectionContext->RawData, ReportType); +} + +PHID_COLLECTION +HidParser_GetCollectionFromContext( + IN PVOID Context) +{ + PHID_COLLECTION_CONTEXT CollectionContext = (PHID_COLLECTION_CONTEXT)Context; + + // + // return root collection + // + return (PHID_COLLECTION)CollectionContext->RawData; +} + +ULONG +HidParser_GetCollectionCount( + IN PHID_COLLECTION_CONTEXT CollectionContext, + IN PHID_COLLECTION Collection) +{ + ULONG Index; + ULONG Count = Collection->NodeCount; + PHID_COLLECTION SubCollection; + + for(Index = 0; Index < Collection->NodeCount; Index++) + { + // + // get offset to sub collection + // + SubCollection = (PHID_COLLECTION)(CollectionContext->RawData + Collection->Offsets[Collection->NodeCount + Index]); + + // + // count collection for sub nodes + // + Count += HidParser_GetCollectionCount(CollectionContext, SubCollection); + } + + // + // done + // + return Count; +} + +ULONG +HidParser_GetTotalCollectionCount( + IN PVOID Context) +{ + PHID_COLLECTION_CONTEXT CollectionContext; + + // + // get parser context + // + CollectionContext = (PHID_COLLECTION_CONTEXT)Context; + + // + // count collections + // + return HidParser_GetCollectionCount(CollectionContext, (PHID_COLLECTION)CollectionContext->RawData); +} Propchange: branches/usb-bringup-trunk/lib/drivers/hidparser/context.c ------------------------------------------------------------------------------ svn:eol-style = native Modified: branches/usb-bringup-trunk/lib/drivers/hidparser/hidparser.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/lib/drivers/h… ============================================================================== --- branches/usb-bringup-trunk/lib/drivers/hidparser/hidparser.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/lib/drivers/hidparser/hidparser.c [iso-8859-1] Tue Jan 31 16:19:33 2012 @@ -51,11 +51,12 @@ HIDPARSER_STATUS ParserStatus; ULONG CollectionCount; ULONG Index; + PVOID ParserContext; // // first parse the report descriptor // - ParserStatus = HidParser_ParseReportDescriptor(Parser, ReportDesc, DescLength); + ParserStatus = HidParser_ParseReportDescriptor(Parser, ReportDesc, DescLength, &ParserContext); if (ParserStatus != HIDPARSER_STATUS_SUCCESS) { // @@ -68,7 +69,7 @@ // // get collection count // - CollectionCount = HidParser_NumberOfTopCollections(Parser); + CollectionCount = HidParser_NumberOfTopCollections(ParserContext); if (CollectionCount == 0) { // @@ -111,48 +112,43 @@ for(Index = 0; Index < CollectionCount; Index++) { // - // init report description - // - DeviceDescription->ReportIDs[Index].CollectionNumber = Index + 1; - DeviceDescription->ReportIDs[Index].ReportID = Index; //FIXME - DeviceDescription->ReportIDs[Index].InputLength = HidParser_GetReportLength(Parser, Index, HID_REPORT_TYPE_INPUT); - DeviceDescription->ReportIDs[Index].OutputLength = HidParser_GetReportLength(Parser, Index, HID_REPORT_TYPE_OUTPUT); - DeviceDescription->ReportIDs[Index].FeatureLength = HidParser_GetReportLength(Parser, Index, HID_REPORT_TYPE_FEATURE); - - // - // init collection description - // - DeviceDescription->CollectionDesc[Index].CollectionNumber = Index + 1; - - // - // get collection usage page - // - ParserStatus = HidParser_GetCollectionUsagePage(Parser, Index, &DeviceDescription->CollectionDesc[Index].Usage, &DeviceDescription->CollectionDesc[Index].UsagePage); - - // - // windows seems to prepend the report id, regardless if it is required - // - DeviceDescription->CollectionDesc[Index].InputLength = (DeviceDescription->ReportIDs[Index].InputLength > 0 ? DeviceDescription->ReportIDs[Index].InputLength + 1 : 0); - DeviceDescription->CollectionDesc[Index].OutputLength = (DeviceDescription->ReportIDs[Index].OutputLength > 0 ? DeviceDescription->ReportIDs[Index].OutputLength + 1 : 0); - DeviceDescription->CollectionDesc[Index].FeatureLength = (DeviceDescription->ReportIDs[Index].FeatureLength > 0 ? DeviceDescription->ReportIDs[Index].FeatureLength + 1 : 0); - - // // set preparsed data length // - DeviceDescription->CollectionDesc[Index].PreparsedDataLength = HidParser_GetContextSize(Parser, Index); - DeviceDescription->CollectionDesc[Index].PreparsedData = Parser->Alloc(DeviceDescription->CollectionDesc[Index].PreparsedDataLength); - if (!DeviceDescription->CollectionDesc[Index].PreparsedData) + DeviceDescription->CollectionDesc[Index].PreparsedDataLength = HidParser_GetContextSize(Parser, ParserContext, Index); + ParserStatus = HidParser_BuildContext(Parser, ParserContext, Index, DeviceDescription->CollectionDesc[Index].PreparsedDataLength, (PVOID*)&DeviceDescription->CollectionDesc[Index].PreparsedData); + if (ParserStatus != HIDPARSER_STATUS_SUCCESS) { // // no memory // - return STATUS_INSUFFICIENT_RESOURCES; + return TranslateHidParserStatus(ParserStatus); } // - // copy context - // - Parser->Copy(DeviceDescription->CollectionDesc[Index].PreparsedData, Parser->ParserContext, DeviceDescription->CollectionDesc[Index].PreparsedDataLength); + // init report description + // + DeviceDescription->ReportIDs[Index].CollectionNumber = Index + 1; + DeviceDescription->ReportIDs[Index].ReportID = Index; //FIXME + DeviceDescription->ReportIDs[Index].InputLength = HidParser_GetReportLength((PVOID)DeviceDescription->CollectionDesc[Index].PreparsedData, HID_REPORT_TYPE_INPUT); + DeviceDescription->ReportIDs[Index].OutputLength = HidParser_GetReportLength((PVOID)DeviceDescription->CollectionDesc[Index].PreparsedData, HID_REPORT_TYPE_OUTPUT); + DeviceDescription->ReportIDs[Index].FeatureLength = HidParser_GetReportLength((PVOID)DeviceDescription->CollectionDesc[Index].PreparsedData, HID_REPORT_TYPE_FEATURE); + + // + // init collection description + // + DeviceDescription->CollectionDesc[Index].CollectionNumber = Index + 1; + + // + // get collection usage page + // + ParserStatus = HidParser_GetCollectionUsagePage((PVOID)DeviceDescription->CollectionDesc[Index].PreparsedData, &DeviceDescription->CollectionDesc[Index].Usage, &DeviceDescription->CollectionDesc[Index].UsagePage); + + // + // windows seems to prepend the report id, regardless if it is required + // + DeviceDescription->CollectionDesc[Index].InputLength = (DeviceDescription->ReportIDs[Index].InputLength > 0 ? DeviceDescription->ReportIDs[Index].InputLength + 1 : 0); + DeviceDescription->CollectionDesc[Index].OutputLength = (DeviceDescription->ReportIDs[Index].OutputLength > 0 ? DeviceDescription->ReportIDs[Index].OutputLength + 1 : 0); + DeviceDescription->CollectionDesc[Index].FeatureLength = (DeviceDescription->ReportIDs[Index].FeatureLength > 0 ? DeviceDescription->ReportIDs[Index].FeatureLength + 1 : 0); } // @@ -181,9 +177,9 @@ for(Index = 0; Index < DeviceDescription->CollectionDescLength; Index++) { // - // free parser context - // - HidParser_FreeContext(Parser, (PUCHAR)DeviceDescription->CollectionDesc[Index].PreparsedData, DeviceDescription->CollectionDesc[Index].PreparsedDataLength); + // free collection context + // + Parser->Free(DeviceDescription->CollectionDesc[Index].PreparsedData); } // @@ -202,15 +198,9 @@ NTAPI HidParser_GetCaps( IN PHID_PARSER Parser, + IN PVOID CollectionContext, OUT PHIDP_CAPS Capabilities) { - ULONG CollectionNumber; - - // - // get collection number from context - // - CollectionNumber = HidParser_GetCollectionNumberFromParserContext(Parser); - // // zero capabilities // @@ -219,10 +209,10 @@ // // init capabilities // - HidParser_GetCollectionUsagePage(Parser, CollectionNumber, &Capabilities->Usage, &Capabilities->UsagePage); - Capabilities->InputReportByteLength = HidParser_GetReportLength(Parser, CollectionNumber, HID_REPORT_TYPE_INPUT); - Capabilities->OutputReportByteLength = HidParser_GetReportLength(Parser, CollectionNumber, HID_REPORT_TYPE_OUTPUT); - Capabilities->FeatureReportByteLength = HidParser_GetReportLength(Parser, CollectionNumber, HID_REPORT_TYPE_FEATURE); + HidParser_GetCollectionUsagePage(CollectionContext, &Capabilities->Usage, &Capabilities->UsagePage); + Capabilities->InputReportByteLength = HidParser_GetReportLength(CollectionContext, HID_REPORT_TYPE_INPUT); + Capabilities->OutputReportByteLength = HidParser_GetReportLength(CollectionContext, HID_REPORT_TYPE_OUTPUT); + Capabilities->FeatureReportByteLength = HidParser_GetReportLength(CollectionContext, HID_REPORT_TYPE_FEATURE); // // always pre-prend report id @@ -234,29 +224,29 @@ // // get number of link collection nodes // - Capabilities->NumberLinkCollectionNodes = HidParser_GetTotalCollectionCount(Parser, CollectionNumber); + Capabilities->NumberLinkCollectionNodes = HidParser_GetTotalCollectionCount(CollectionContext); // // get data indices // - Capabilities->NumberInputDataIndices = HidParser_GetReportItemTypeCountFromReportType(Parser, CollectionNumber, HID_REPORT_TYPE_INPUT, TRUE); - Capabilities->NumberOutputDataIndices = HidParser_GetReportItemTypeCountFromReportType(Parser, CollectionNumber, HID_REPORT_TYPE_OUTPUT, TRUE); - Capabilities->NumberFeatureDataIndices = HidParser_GetReportItemTypeCountFromReportType(Parser, CollectionNumber, HID_REPORT_TYPE_FEATURE, TRUE); + Capabilities->NumberInputDataIndices = HidParser_GetReportItemTypeCountFromReportType(CollectionContext, HID_REPORT_TYPE_INPUT, TRUE); + Capabilities->NumberOutputDataIndices = HidParser_GetReportItemTypeCountFromReportType(CollectionContext, HID_REPORT_TYPE_OUTPUT, TRUE); + Capabilities->NumberFeatureDataIndices = HidParser_GetReportItemTypeCountFromReportType(CollectionContext, HID_REPORT_TYPE_FEATURE, TRUE); // // get value caps // - Capabilities->NumberInputValueCaps = HidParser_GetReportItemTypeCountFromReportType(Parser, CollectionNumber, HID_REPORT_TYPE_INPUT, FALSE); - Capabilities->NumberOutputValueCaps = HidParser_GetReportItemTypeCountFromReportType(Parser, CollectionNumber, HID_REPORT_TYPE_OUTPUT, FALSE); - Capabilities->NumberFeatureValueCaps = HidParser_GetReportItemTypeCountFromReportType(Parser, CollectionNumber, HID_REPORT_TYPE_FEATURE, FALSE); + Capabilities->NumberInputValueCaps = HidParser_GetReportItemTypeCountFromReportType(CollectionContext, HID_REPORT_TYPE_INPUT, FALSE); + Capabilities->NumberOutputValueCaps = HidParser_GetReportItemTypeCountFromReportType(CollectionContext, HID_REPORT_TYPE_OUTPUT, FALSE); + Capabilities->NumberFeatureValueCaps = HidParser_GetReportItemTypeCountFromReportType(CollectionContext, HID_REPORT_TYPE_FEATURE, FALSE); // // get button caps // - Capabilities->NumberInputButtonCaps = HidParser_GetReportItemCountFromReportType(Parser, CollectionNumber, HID_REPORT_TYPE_INPUT); - Capabilities->NumberOutputButtonCaps = HidParser_GetReportItemCountFromReportType(Parser, CollectionNumber, HID_REPORT_TYPE_OUTPUT); - Capabilities->NumberFeatureButtonCaps = HidParser_GetReportItemCountFromReportType(Parser, CollectionNumber, HID_REPORT_TYPE_FEATURE); + Capabilities->NumberInputButtonCaps = HidParser_GetReportItemCountFromReportType(CollectionContext, HID_REPORT_TYPE_INPUT); + Capabilities->NumberOutputButtonCaps = HidParser_GetReportItemCountFromReportType(CollectionContext, HID_REPORT_TYPE_OUTPUT); + Capabilities->NumberFeatureButtonCaps = HidParser_GetReportItemCountFromReportType(CollectionContext, HID_REPORT_TYPE_FEATURE); // // done @@ -269,17 +259,10 @@ NTAPI HidParser_MaxUsageListLength( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage OPTIONAL) { - ULONG CollectionNumber; - - // - // get collection number from context - // - CollectionNumber = HidParser_GetCollectionNumberFromParserContext(Parser); - - // // FIXME test what should be returned when usage page is not defined // @@ -301,21 +284,21 @@ // // input report // - return HidParser_GetMaxUsageListLengthWithReportAndPage(Parser, CollectionNumber, HID_REPORT_TYPE_INPUT, UsagePage); + return HidParser_GetMaxUsageListLengthWithReportAndPage(CollectionContext, HID_REPORT_TYPE_INPUT, UsagePage); } else if (ReportType == HidP_Output) { // // input report // - return HidParser_GetMaxUsageListLengthWithReportAndPage(Parser, CollectionNumber, HID_REPORT_TYPE_OUTPUT, UsagePage); + return HidParser_GetMaxUsageListLengthWithReportAndPage(CollectionContext, HID_REPORT_TYPE_OUTPUT, UsagePage); } else if (ReportType == HidP_Feature) { // // input report // - return HidParser_GetMaxUsageListLengthWithReportAndPage(Parser, CollectionNumber, HID_REPORT_TYPE_FEATURE, UsagePage); + return HidParser_GetMaxUsageListLengthWithReportAndPage(CollectionContext, HID_REPORT_TYPE_FEATURE, UsagePage); } else { @@ -333,11 +316,12 @@ NTAPI HidParser_GetButtonCaps( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN PHIDP_BUTTON_CAPS ButtonCaps, IN PUSHORT ButtonCapsLength) { - return HidParser_GetSpecificButtonCaps(Parser, ReportType, HID_USAGE_PAGE_UNDEFINED, HIDP_LINK_COLLECTION_UNSPECIFIED, HID_USAGE_PAGE_UNDEFINED, ButtonCaps, (PULONG)ButtonCapsLength); + return HidParser_GetSpecificButtonCaps(Parser, CollectionContext, ReportType, HID_USAGE_PAGE_UNDEFINED, HIDP_LINK_COLLECTION_UNSPECIFIED, HID_USAGE_PAGE_UNDEFINED, ButtonCaps, (PULONG)ButtonCapsLength); } HIDAPI @@ -345,6 +329,7 @@ NTAPI HidParser_GetSpecificValueCaps( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -353,14 +338,6 @@ IN OUT PULONG ValueCapsLength) { HIDPARSER_STATUS ParserStatus; - ULONG CollectionNumber; - - // - // get collection number from context - // - CollectionNumber = HidParser_GetCollectionNumberFromParserContext(Parser); - - // // FIXME: implement searching in specific collection @@ -372,21 +349,21 @@ // // input report // - ParserStatus = HidParser_GetSpecificValueCapsWithReport(Parser, CollectionNumber, HID_REPORT_TYPE_INPUT, UsagePage, Usage, ValueCaps, ValueCapsLength); + ParserStatus = HidParser_GetSpecificValueCapsWithReport(Parser, CollectionContext, HID_REPORT_TYPE_INPUT, UsagePage, Usage, ValueCaps, ValueCapsLength); } else if (ReportType == HidP_Output) { // // input report // - ParserStatus = HidParser_GetSpecificValueCapsWithReport(Parser, CollectionNumber, HID_REPORT_TYPE_OUTPUT, UsagePage, Usage, ValueCaps, ValueCapsLength); + ParserStatus = HidParser_GetSpecificValueCapsWithReport(Parser, CollectionContext, HID_REPORT_TYPE_OUTPUT, UsagePage, Usage, ValueCaps, ValueCapsLength); } else if (ReportType == HidP_Feature) { // // input report // - ParserStatus = HidParser_GetSpecificValueCapsWithReport(Parser, CollectionNumber, HID_REPORT_TYPE_FEATURE, UsagePage, Usage, ValueCaps, ValueCapsLength); + ParserStatus = HidParser_GetSpecificValueCapsWithReport(Parser, CollectionContext, HID_REPORT_TYPE_FEATURE, UsagePage, Usage, ValueCaps, ValueCapsLength); } else { @@ -545,6 +522,7 @@ NTAPI HidParser_GetUsages( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection OPTIONAL, @@ -554,12 +532,6 @@ IN ULONG ReportLength) { HIDPARSER_STATUS ParserStatus; - ULONG CollectionNumber; - - // - // get collection number from context - // - CollectionNumber = HidParser_GetCollectionNumberFromParserContext(Parser); // // FIXME: implement searching in specific collection @@ -571,21 +543,21 @@ // // input report // - ParserStatus = HidParser_GetUsagesWithReport(Parser, CollectionNumber, HID_REPORT_TYPE_INPUT, UsagePage, UsageList, UsageLength, Report, ReportLength); + ParserStatus = HidParser_GetUsagesWithReport(Parser, CollectionContext, HID_REPORT_TYPE_INPUT, UsagePage, UsageList, UsageLength, Report, ReportLength); } else if (ReportType == HidP_Output) { // // input report // - ParserStatus = HidParser_GetUsagesWithReport(Parser, CollectionNumber, HID_REPORT_TYPE_OUTPUT, UsagePage, UsageList, UsageLength, Report, ReportLength); + ParserStatus = HidParser_GetUsagesWithReport(Parser, CollectionContext, HID_REPORT_TYPE_OUTPUT, UsagePage, UsageList, UsageLength, Report, ReportLength); } else if (ReportType == HidP_Feature) { // // input report // - ParserStatus = HidParser_GetUsagesWithReport(Parser, CollectionNumber, HID_REPORT_TYPE_FEATURE, UsagePage, UsageList, UsageLength, Report, ReportLength); + ParserStatus = HidParser_GetUsagesWithReport(Parser, CollectionContext, HID_REPORT_TYPE_FEATURE, UsagePage, UsageList, UsageLength, Report, ReportLength); } else { @@ -614,6 +586,7 @@ NTAPI HidParser_GetScaledUsageValue( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection OPTIONAL, @@ -623,12 +596,6 @@ IN ULONG ReportLength) { HIDPARSER_STATUS ParserStatus; - ULONG CollectionNumber; - - // - // get collection number from context - // - CollectionNumber = HidParser_GetCollectionNumberFromParserContext(Parser); // // FIXME: implement searching in specific collection @@ -640,21 +607,21 @@ // // input report // - ParserStatus = HidParser_GetScaledUsageValueWithReport(Parser, CollectionNumber, HID_REPORT_TYPE_INPUT, UsagePage, Usage, UsageValue, Report, ReportLength); + ParserStatus = HidParser_GetScaledUsageValueWithReport(Parser, CollectionContext, HID_REPORT_TYPE_INPUT, UsagePage, Usage, UsageValue, Report, ReportLength); } else if (ReportType == HidP_Output) { // // input report // - ParserStatus = HidParser_GetScaledUsageValueWithReport(Parser, CollectionNumber, HID_REPORT_TYPE_OUTPUT, UsagePage, Usage, UsageValue, Report, ReportLength); + ParserStatus = HidParser_GetScaledUsageValueWithReport(Parser, CollectionContext, HID_REPORT_TYPE_OUTPUT, UsagePage, Usage, UsageValue, Report, ReportLength); } else if (ReportType == HidP_Feature) { // // input report // - ParserStatus = HidParser_GetScaledUsageValueWithReport(Parser, CollectionNumber, HID_REPORT_TYPE_FEATURE, UsagePage, Usage, UsageValue, Report, ReportLength); + ParserStatus = HidParser_GetScaledUsageValueWithReport(Parser, CollectionContext, HID_REPORT_TYPE_FEATURE, UsagePage, Usage, UsageValue, Report, ReportLength); } else { @@ -692,12 +659,6 @@ { ULONG Index; HIDPARSER_STATUS Status = HIDPARSER_STATUS_SUCCESS; - ULONG CollectionNumber; - - // - // get collection number from context - // - CollectionNumber = HidParser_GetCollectionNumberFromParserContext(Parser); for(Index = 0; Index < UsageListLength; Index++) { @@ -709,7 +670,7 @@ // // process usage // - Status = HidParser_TranslateUsage(Parser, CollectionNumber, ChangedUsageList[Index].Usage, KeyAction, ModifierState, InsertCodesProcedure, InsertCodesContext); + Status = HidParser_TranslateUsage(Parser, ChangedUsageList[Index].Usage, KeyAction, ModifierState, InsertCodesProcedure, InsertCodesContext); } else if (ChangedUsageList[Index].UsagePage == HID_USAGE_PAGE_CONSUMER) { @@ -760,6 +721,7 @@ NTAPI HidParser_GetUsagesEx( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USHORT LinkCollection, OUT PUSAGE_AND_PAGE ButtonList, @@ -767,7 +729,7 @@ IN PCHAR Report, IN ULONG ReportLength) { - return HidParser_GetUsages(Parser, ReportType, HID_USAGE_PAGE_UNDEFINED, LinkCollection, (PUSAGE)ButtonList, UsageLength, Report, ReportLength); + return HidParser_GetUsages(Parser, CollectionContext, ReportType, HID_USAGE_PAGE_UNDEFINED, LinkCollection, (PUSAGE)ButtonList, UsageLength, Report, ReportLength); } HIDAPI @@ -904,6 +866,7 @@ NTAPI HidParser_GetSpecificButtonCaps( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -922,6 +885,7 @@ NTAPI HidParser_GetData( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, OUT PHIDP_DATA DataList, IN OUT PULONG DataLength, @@ -938,6 +902,7 @@ NTAPI HidParser_GetExtendedAttributes( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USHORT DataIndex, OUT PHIDP_EXTENDED_ATTRIBUTES Attributes, @@ -953,6 +918,7 @@ NTAPI HidParser_GetLinkCollectionNodes( IN PHID_PARSER Parser, + IN PVOID CollectionContext, OUT PHIDP_LINK_COLLECTION_NODE LinkCollectionNodes, IN OUT PULONG LinkCollectionNodesLength) { @@ -966,6 +932,7 @@ NTAPI HidParser_GetUsageValue( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -983,6 +950,7 @@ NTAPI HidParser_SysPowerEvent( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN PCHAR HidPacket, IN USHORT HidPacketLength, OUT PULONG OutputBuffer) @@ -996,6 +964,7 @@ NTAPI HidParser_SysPowerCaps ( IN PHID_PARSER Parser, + IN PVOID CollectionContext, OUT PULONG OutputBuffer) { UNIMPLEMENTED @@ -1008,6 +977,7 @@ NTAPI HidParser_GetUsageValueArray( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection OPTIONAL, @@ -1027,6 +997,7 @@ NTAPI HidParser_UnsetUsages( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -1061,6 +1032,7 @@ NTAPI HidParser_SetUsages( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -1079,6 +1051,7 @@ NTAPI HidParser_SetUsageValueArray( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection OPTIONAL, @@ -1098,6 +1071,7 @@ NTAPI HidParser_SetUsageValue( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -1116,6 +1090,7 @@ NTAPI HidParser_SetScaledUsageValue( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection OPTIONAL, @@ -1134,6 +1109,7 @@ NTAPI HidParser_SetData( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN PHIDP_DATA DataList, IN OUT PULONG DataLength, @@ -1150,11 +1126,12 @@ NTAPI HidParser_MaxDataListLength( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType) { UNIMPLEMENTED ASSERT(FALSE); - return STATUS_NOT_IMPLEMENTED; + return 0; } HIDAPI @@ -1162,6 +1139,7 @@ NTAPI HidParser_InitializeReportForID( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN UCHAR ReportID, IN OUT PCHAR Report, @@ -1179,6 +1157,7 @@ NTAPI HidParser_GetValueCaps( IN PHID_PARSER Parser, + IN PVOID CollectionContext, HIDP_REPORT_TYPE ReportType, PHIDP_VALUE_CAPS ValueCaps, PULONG ValueCapsLength) Modified: branches/usb-bringup-trunk/lib/drivers/hidparser/hidparser.h URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/lib/drivers/h… ============================================================================== --- branches/usb-bringup-trunk/lib/drivers/hidparser/hidparser.h [iso-8859-1] (original) +++ branches/usb-bringup-trunk/lib/drivers/hidparser/hidparser.h [iso-8859-1] Tue Jan 31 16:19:33 2012 @@ -79,22 +79,7 @@ // debug function // PHIDPARSER_DEBUG_FUNCTION Debug; - - // - // parser context - // - void * ParserContext; - }HID_PARSER, *PHID_PARSER; - -HIDPARSER_STATUS -HidParser_AllocateParser( - IN PHIDPARSER_ALLOC_FUNCTION AllocFunction, - IN PHIDPARSER_FREE_FUNCTION FreeFunction, - IN PHIDPARSER_ZERO_FUNCTION ZeroFunction, - IN PHIDPARSER_COPY_FUNCTION CopyFunction, - IN PHIDPARSER_DEBUG_FUNCTION DebugFunction, - OUT PHID_PARSER *OutParser); VOID HidParser_InitParser( @@ -103,7 +88,6 @@ IN PHIDPARSER_ZERO_FUNCTION ZeroFunction, IN PHIDPARSER_COPY_FUNCTION CopyFunction, IN PHIDPARSER_DEBUG_FUNCTION DebugFunction, - IN PVOID ParserContext, OUT PHID_PARSER Parser); NTSTATUS @@ -126,6 +110,7 @@ NTAPI HidParser_GetCaps( IN PHID_PARSER Parser, + IN PVOID CollectionContext, OUT PHIDP_CAPS Capabilities); HIDAPI @@ -133,6 +118,7 @@ NTAPI HidParser_GetSpecificValueCaps( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -146,6 +132,7 @@ NTAPI HidParser_GetButtonCaps( IN PHID_PARSER Parser, + IN PVOID CollectionContext, HIDP_REPORT_TYPE ReportType, PHIDP_BUTTON_CAPS ButtonCaps, PUSHORT ButtonCapsLength); @@ -155,6 +142,7 @@ NTAPI HidParser_GetSpecificButtonCaps( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -167,6 +155,7 @@ NTAPI HidParser_GetScaledUsageValue( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection OPTIONAL, @@ -181,6 +170,7 @@ NTAPI HidParser_GetData( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, OUT PHIDP_DATA DataList, IN OUT PULONG DataLength, @@ -192,6 +182,7 @@ NTAPI HidParser_GetExtendedAttributes( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USHORT DataIndex, OUT PHIDP_EXTENDED_ATTRIBUTES Attributes, @@ -202,6 +193,7 @@ NTAPI HidParser_GetLinkCollectionNodes( IN PHID_PARSER Parser, + IN PVOID CollectionContext, OUT PHIDP_LINK_COLLECTION_NODE LinkCollectionNodes, IN OUT PULONG LinkCollectionNodesLength); @@ -210,6 +202,7 @@ NTAPI HidParser_GetUsageValue( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -234,6 +227,7 @@ NTAPI HidParser_MaxUsageListLength( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage OPTIONAL); @@ -242,6 +236,7 @@ NTAPI HidParser_GetUsages( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection OPTIONAL, @@ -255,6 +250,7 @@ NTAPI HidParser_GetUsagesEx( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USHORT LinkCollection, OUT PUSAGE_AND_PAGE ButtonList, @@ -267,6 +263,7 @@ NTAPI HidParser_SysPowerEvent ( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN PCHAR HidPacket, IN USHORT HidPacketLength, OUT PULONG OutputBuffer); @@ -275,6 +272,7 @@ NTAPI HidParser_SysPowerCaps ( IN PHID_PARSER Parser, + IN PVOID CollectionContext, OUT PULONG OutputBuffer); HIDAPI @@ -282,6 +280,7 @@ NTAPI HidParser_GetUsageValueArray( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection OPTIONAL, @@ -307,6 +306,7 @@ NTAPI HidParser_UnsetUsages( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -343,6 +343,7 @@ NTAPI HidParser_SetUsages( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -356,6 +357,7 @@ NTAPI HidParser_SetUsageValueArray( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection OPTIONAL, @@ -370,6 +372,7 @@ NTAPI HidParser_SetUsageValue( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection, @@ -383,6 +386,7 @@ NTAPI HidParser_SetScaledUsageValue( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN USAGE UsagePage, IN USHORT LinkCollection OPTIONAL, @@ -396,6 +400,7 @@ NTAPI HidParser_SetData( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN PHIDP_DATA DataList, IN OUT PULONG DataLength, @@ -407,6 +412,7 @@ NTAPI HidParser_MaxDataListLength( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType); HIDAPI @@ -414,6 +420,7 @@ NTAPI HidParser_InitializeReportForID( IN PHID_PARSER Parser, + IN PVOID CollectionContext, IN HIDP_REPORT_TYPE ReportType, IN UCHAR ReportID, IN OUT PCHAR Report, @@ -422,7 +429,6 @@ HIDPARSER_STATUS HidParser_TranslateUsage( IN PHID_PARSER Parser, - IN ULONG CollectionNumber, IN USAGE Usage, IN HIDP_KEYBOARD_DIRECTION KeyAction, IN OUT PHIDP_KEYBOARD_MODIFIER_STATE ModifierState, @@ -434,6 +440,7 @@ NTAPI HidParser_GetValueCaps( PHID_PARSER Parser, + IN PVOID CollectionContext, HIDP_REPORT_TYPE ReportType, PHIDP_VALUE_CAPS ValueCaps, PULONG ValueCapsLength); Modified: branches/usb-bringup-trunk/lib/drivers/hidparser/parser.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/lib/drivers/h… ============================================================================== --- branches/usb-bringup-trunk/lib/drivers/hidparser/parser.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/lib/drivers/hidparser/parser.c [iso-8859-1] Tue Jan 31 16:19:33 2012 @@ -123,40 +123,6 @@ return HIDPARSER_STATUS_SUCCESS; } - -VOID -HidParser_ResetParser( - OUT PHID_PARSER Parser) -{ - ULONG Index; - PHID_PARSER_CONTEXT ParserContext; - - // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - - if (ParserContext->RootCollection) - { - // - // delete root collection - // - HidParser_FreeCollection(Parser, ParserContext->RootCollection); - } - - // - // reinit parser - // - ParserContext->RootCollection = NULL; - ParserContext->UseReportIDs = FALSE; - - // - // zero item states - // - Parser->Zero(&ParserContext->GlobalItemState, sizeof(GLOBAL_ITEM_STATE)); - Parser->Zero(&ParserContext->LocalItemState, sizeof(LOCAL_ITEM_STATE)); -} - HIDPARSER_STATUS HidParser_AddCollection( IN PHID_PARSER Parser, @@ -260,18 +226,11 @@ HIDPARSER_STATUS HidParser_FindReport( IN PHID_PARSER Parser, + IN PHID_PARSER_CONTEXT ParserContext, IN UCHAR ReportType, IN UCHAR ReportID, OUT PHID_REPORT *OutReport) { - PHID_PARSER_CONTEXT ParserContext; - - // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - ASSERT(ParserContext); - // // search in current top level collection // @@ -315,17 +274,11 @@ HIDPARSER_STATUS HidParser_AddReportToCollection( IN PHID_PARSER Parser, + IN PHID_PARSER_CONTEXT ParserContext, IN PHID_COLLECTION CurrentCollection, IN PHID_REPORT NewReport) { PHID_REPORT * NewReportArray; - PHID_PARSER_CONTEXT ParserContext; - - // - // get parser context - // - ParserContext = (PHID_PARSER_CONTEXT)Parser->ParserContext; - ASSERT(ParserContext); // // allocate new report array @@ -368,6 +321,7 @@ HIDPARSER_STATUS HidParser_GetReport( IN PHID_PARSER Parser, + IN PHID_PARSER_CONTEXT ParserContext, IN PHID_COLLECTION Collection, IN UCHAR ReportType, IN UCHAR ReportID, @@ -379,7 +333,7 @@ // // try finding existing report // - Status = HidParser_FindReport(Parser, ReportType, ReportID, OutReport); + Status = HidParser_FindReport(Parser, ParserContext, ReportType, ReportID, OutReport); if (Status == HIDPARSER_STATUS_SUCCESS || CreateIfNotExists == FALSE) { // @@ -403,7 +357,7 @@ // // add report // - Status = HidParser_AddReportToCollection(Parser, Collection, *OutReport); + Status = HidParser_AddReportToCollection(Parser, ParserContext, Collection, *OutReport); if (Status != HIDPARSER_STATUS_SUCCESS) { // @@ -417,86 +371,37 @@ // return Status; } - - -HIDPARSER_STATUS -HidParser_ReserveCollectionItems( - IN PHID_PARSER Parser, - IN PHID_COLLECTION Collection, - IN ULONG ReportCount) -{ - PHID_REPORT_ITEM * NewReportArray; - - if (Collection->ItemCount + ReportCount <= Collection->ItemCountAllocated) - { - // - // enough space for the next items - // - return HIDPARSER_STATUS_SUCCESS; - } - - // - // allocate report array - // - NewReportArray = (PHID_REPORT_ITEM*)Parser->Alloc(sizeof(PHID_REPORT) * (Collection->ItemCountAllocated + ReportCount)); - if (!NewReportArray) - { - // - // no memory - // - return HIDPARSER_STATUS_INSUFFICIENT_RESOURCES; - } - - // - // are there any items - // - if (Collection->ItemCount) - { - // - // copy old items - // - Parser->Copy(NewReportArray, Collection->Items, sizeof(PHID_REPORT_ITEM) * Collection->ItemCount); - - // - // free old item - // - Parser->Free(Collection->Items); - } - - // - // replace array - // - Collection->Items = NewReportArray; - Collection->ItemCountAllocated += ReportCount; - - // - // completed sucessfully - // - return HIDPARSER_STATUS_SUCCESS; -} - HIDPARSER_STATUS HidParser_ReserveReportItems( IN PHID_PARSER Parser, IN PHID_REPORT Report, - IN ULONG ReportCount) -{ - PHID_REPORT_ITEM * NewReportArray; + IN ULONG ReportCount, + OUT PHID_REPORT *OutReport) +{ + PHID_REPORT NewReport; + ULONG OldSize, Size; if (Report->ItemCount + ReportCount <= Report->ItemAllocated) { // - // enough space for the next items - // + // space is already allocated + // + *OutReport = Report; return HIDPARSER_STATUS_SUCCESS; } // - // allocate report array - // - NewReportArray = (PHID_REPORT_ITEM*)Parser->Alloc(sizeof(PHID_REPORT_ITEM) * (Report->ItemAllocated + ReportCount)); - if (!NewReportArray) + //calculate new size + // + OldSize = sizeof(HID_REPORT) + (Report->ItemCount) * sizeof(HID_REPORT_ITEM); + Size = ReportCount * sizeof(HID_REPORT_ITEM); + + // + // allocate memory + // + NewReport = (PHID_REPORT)Parser->Alloc(Size + OldSize); + if (!NewReport) { // // no memory @@ -504,92 +409,26 @@ return HIDPARSER_STATUS_INSUFFICIENT_RESOURCES; } - // - // are there any items - // - if (Report->ItemCount) - { - // - // copy old items - // - Parser->Copy(NewReportArray, Report->Items, sizeof(PHID_REPORT_ITEM) * Report->ItemCount); - - // - // free old item - // - Parser->Free(Report->Items); - } - - // - // replace array - // - Report->Items = NewReportArray; - Report->ItemAllocated += ReportCount; + + // + // copy old report + // + Parser->Copy(NewReport, Report, OldSize); + + // + // increase array size + // + NewReport->ItemAllocated += ReportCount; + + // + // store result + // + *OutReport = NewReport; // // completed sucessfully // return HIDPARSER_STATUS_SUCCESS; -} - -HIDPARSER_STATUS -HidParser_AllocateReportItem( - IN PHID_PARSER Parser, - OUT PHID_REPORT_ITEM * OutReportItem) -{ - PHID_REPORT_ITEM ReportItem; - - // - // allocate report item - // - ReportItem = (PHID_REPORT_ITEM)Parser->Alloc(sizeof(HID_REPORT_ITEM)); - if (!ReportItem) - { - // - // no memory - // - return HIDPARSER_STATUS_INSUFFICIENT_RESOURCES; - } - - // - // store result - // - *OutReportItem = ReportItem; - return HIDPARSER_STATUS_SUCCESS; -} - -VOID -HidParser_AddReportItemToReport( - IN OUT PHID_REPORT Report, - IN PHID_REPORT_ITEM ReportItem) -{ - // - // there should be space in item array - // - ASSERT(Report->ItemCount + 1 <= Report->ItemAllocated); - - // - // store item - // - Report->Items[Report->ItemCount] = ReportItem; - Report->ItemCount++; -} - -VOID -HidParser_AddReportItemToCollection( - IN OUT PHID_COLLECTION Collection, - IN PHID_REPORT_ITEM ReportItem) -{ - // - // there should be space in item array - // - ASSERT(Collection->ItemCount + 1 <= Collection->ItemCountAllocated); - - // - // store item - // - Collection->Items[Collection->ItemCount] = ReportItem; - Collection->ItemCount++; } VOID @@ -745,23 +584,87 @@ return HIDPARSER_STATUS_SUCCESS; } +BOOLEAN +HidParser_UpdateCurrentCollectionReport( + IN PHID_COLLECTION Collection, + IN PHID_REPORT Report, + IN PHID_REPORT NewReport) +{ + ULONG Index; + BOOLEAN Found = FALSE, TempFound; + + // + // search in local list + // + for(Index = 0; Index < Collection->ReportCount; Index++) + { + if (Collection->Reports[Index] == Report) + { + // + // update report + // + Collection->Reports[Index] = NewReport; + Found = TRUE; + } + } + + // + // search in sub collections + // + for(Index = 0; Index < Collection->NodeCount; Index++) + { + // + // was it found + // + TempFound = HidParser_UpdateCurrentCollectionReport(Collection->Nodes[Index], Report, NewReport); + if (TempFound) + { + // + // the same report should not be found in different collections + // + ASSERT(Found == FALSE); + Found = TRUE; + } + } + + // + // done + // + return Found; +} + +BOOLEAN +HidParser_UpdateCollectionReport( + IN PHID_PARSER_CONTEXT ParserContext, + IN PHID_REPORT Report, + IN PHID_REPORT NewReport) +{ + // + // update in current collection + // + return HidParser_UpdateCurrentCollectionReport(ParserContext->RootCollection->Nodes[ParserContext->RootCollection->NodeCount-1], Report, NewReport); +} + + HIDPARSER_STATUS HidParser_AddMainItem( IN PHID_PARSER Parser, + IN PHID_PARSER_CONTEXT ParserContext, IN PHID_REPORT Report, IN PGLOBAL_ITEM_STATE GlobalItemState, IN PLOCAL_ITEM_STATE LocalItemState, IN PMAIN_ITEM_DATA ItemData, IN PHID_COLLECTION Collection) { - PHID_REPORT_ITEM ReportItem; HIDPARSER_STATUS Status; ULONG Index; + PHID_REPORT NewReport; + BOOLEAN Found; // // first grow report item array // - Status = HidParser_ReserveReportItems(Parser, Report, GlobalItemState->ReportCount); + Status = HidParser_ReserveReportItems(Parser, Report, GlobalItemState->ReportCount, &NewReport); if (Status != HIDPARSER_STATUS_SUCCESS) { // @@ -770,48 +673,35 @@ return Status; } - // - // grow collection item array - // - Status = HidParser_ReserveCollectionItems(Parser, Collection, GlobalItemState->ReportCount); - if (Status != HIDPARSER_STATUS_SUCCESS) - { - // - // failed to allocate memory - // - return Status; - } - - + if (NewReport != Report) + { + // + // update current top level collection + // + Found = HidParser_UpdateCollectionReport(ParserContext, Report, NewReport); + ASSERT(Found); + } + + // + // sanity check + // + ASSERT(NewReport->ItemCount + GlobalItemState->ReportCount <= NewReport->ItemAllocated); for(Index = 0; Index < GlobalItemState->ReportCount; Index++) { - // - // create report item - // - Status = HidParser_AllocateReportItem(Parser, &ReportItem); + Status = HidParser_InitReportItem(NewReport, &NewReport->Items[NewReport->ItemCount], GlobalItemState, LocalItemState, ItemData, Index); if (Status != HIDPARSER_STATUS_SUCCESS) { // - // failed to allocate memory + // failed to init report item // return Status; } - Status = HidParser_InitReportItem(Report, ReportItem, GlobalItemState, LocalItemState, ItemData, Index); - if (Status != HIDPARSER_STATUS_SUCCESS) - { - // - // failed to init report item - // - return Status; - } - - // - // add report item - // - HidParser_AddReportItemToReport(Report, ReportItem); - HidParser_AddReportItemToCollection(Collection, ReportItem); + // + // increment report item count + // + NewReport->ItemCount++; } // @@ -819,12 +709,37 @@ // return HIDPARSER_STATUS_SUCCESS; } + +HIDPARSER_STATUS +AllocateParserContext( + IN PHID_PARSER Parser, + OUT PHID_PARSER_CONTEXT *OutParserContext) +{ + PHID_PARSER_CONTEXT ParserContext; + + ParserContext = Parser->Alloc(sizeof(HID_PARSER_CONTEXT)); + if (!ParserContext) + { + // + // failed + // + return HIDPARSER_STATUS_INSUFFICIENT_RESOURCES; + } + + // + // store result + // + *OutParserContext = ParserContext; + return HIDPARSER_STATUS_SUCCESS; +} + HIDPARSER_STATUS HidParser_ParseReportDescriptor( IN PHID_PARSER Parser, IN PUCHAR ReportDescriptor, - IN ULONG ReportLength) + IN ULONG ReportLength, + OUT PVOID *OutParser) { PGLOBAL_ITEM_STATE LinkedGlobalItemState, NextLinkedGlobalItemState; ULONG Index; @@ -843,15 +758,12 @@ PHID_PARSER_CONTEXT ParserContext; // - // reset parser - // - HidParser_ResetParser(Parser); - - // - // get parser context - // - ParserContext =(PHID_PARSER_CONTEXT)Parser->ParserContext; - ASSERT(ParserContext); + // allocate parser + // + Status = AllocateParserContext(Parser, &ParserContext); + if (Status != HIDPARSER_STATUS_SUCCESS) + return Status; + // // allocate usage stack @@ -1039,7 +951,7 @@ // // get report // - Status = HidParser_GetReport(Parser, CurrentCollection, ReportType, ParserContext->GlobalItemState.ReportId, TRUE, &Report); + Status = HidParser_GetReport(Parser, ParserContext, CurrentCollection, ReportType, ParserContext->GlobalItemState.ReportId, TRUE, &Report); ASSERT(Status == HIDPARSER_STATUS_SUCCESS); // fill in a sensible default if the index isn't set @@ -1059,7 +971,7 @@ // // add states & data to the report // - Status = HidParser_AddMainItem(Parser, Report, &ParserContext->GlobalItemState, &ParserContext->LocalItemState, MainItemData, CurrentCollection); + Status = HidParser_AddMainItem(Parser, ParserContext, Report, &ParserContext->GlobalItemState, &ParserContext->LocalItemState, MainItemData, CurrentCollection); ASSERT(Status == HIDPARSER_STATUS_SUCCESS); } @@ -1349,7 +1261,139 @@ ParserContext->LocalItemState.UsageStack = NULL; // + // store result + // + *OutParser = ParserContext; + + // // done // return HIDPARSER_STATUS_SUCCESS; } + +PHID_COLLECTION +HidParser_GetCollection( + IN PHID_PARSER Parser, + PHID_PARSER_CONTEXT ParserContext, + IN ULONG CollectionNumber) +{ + // + // sanity checks + // + ASSERT(ParserContext); + ASSERT(ParserContext->RootCollection); + ASSERT(ParserContext->RootCollection->NodeCount); + + // + // is collection index out of bounds + // + if (CollectionNumber < ParserContext->RootCollection->NodeCount) + { + // + // valid collection + // + return ParserContext->RootCollection->Nodes[CollectionNumber]; + } + + // + // no such collection + // + Parser->Debug("HIDPARSE] No such collection %lu\n", CollectionNumber); + return NULL; +} + + +ULONG +HidParser_NumberOfTopCollections( + IN PVOID ParserCtx) +{ + PHID_PARSER_CONTEXT ParserContext; + + // + // get parser context + // + ParserContext = (PHID_PARSER_CONTEXT)ParserCtx; + + // + // sanity checks + // + ASSERT(ParserContext); + ASSERT(ParserContext->RootCollection); + ASSERT(ParserContext->RootCollection->NodeCount); + + // + // number of top collections + // + return ParserContext->RootCollection->NodeCount; +} + +HIDPARSER_STATUS +HidParser_BuildContext( + IN PHID_PARSER Parser, + IN PVOID ParserContext, + IN ULONG CollectionIndex, + IN ULONG ContextSize, + OUT PVOID *CollectionContext) +{ + PHID_COLLECTION Collection; + PVOID Context; + HIDPARSER_STATUS Status; + + // + // lets get the collection + // + Collection = HidParser_GetCollection(Parser, (PHID_PARSER_CONTEXT)ParserContext, CollectionIndex); + ASSERT(Collection); + + // + // lets allocate the context + // + Context = Parser->Alloc(ContextSize); + if (Context == NULL) + { + // + // no memory + // + return HIDPARSER_STATUS_INSUFFICIENT_RESOURCES; + } + + // + // lets build the context + // + Status = HidParser_BuildCollectionContext(Parser, Collection, Context, ContextSize); + if (Status == HIDPARSER_STATUS_SUCCESS) + { + // + // store context + // + *CollectionContext = Context; + } + + // + // done + // + return Status; +} + + +ULONG +HidParser_GetContextSize( + IN PHID_PARSER Parser, + IN PVOID ParserContext, + IN ULONG CollectionIndex) +{ + PHID_COLLECTION Collection; + ULONG Size; + + // + // lets get the collection + // + Collection = HidParser_GetCollection(Parser, (PHID_PARSER_CONTEXT)ParserContext, CollectionIndex); + + // + // calculate size + // + Size = HidParser_CalculateContextSize(Collection); + return Size; +} + Modified: branches/usb-bringup-trunk/lib/drivers/hidparser/parser.h URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/lib/drivers/h… ============================================================================== --- branches/usb-bringup-trunk/lib/drivers/hidparser/parser.h [iso-8859-1] (original) +++ branches/usb-bringup-trunk/lib/drivers/hidparser/parser.h [iso-8859-1] Tue Jan 31 16:19:33 2012 @@ -199,22 +199,18 @@ typedef struct __HID_COLLECTION__ { - struct __HID_COLLECTION__ * Root; - UCHAR Type; ULONG Usage; UCHAR StringID; UCHAR PhysicalID; + ULONG ReportCount; ULONG NodeCount; + struct __HID_COLLECTION__ ** Nodes; - - ULONG ItemCount; - ULONG ItemCountAllocated; - - PHID_REPORT_ITEM * Items; - - ULONG ReportCount; + struct __HID_COLLECTION__ * Root; struct _HID_REPORT ** Reports; + + ULONG Offsets[1]; }HID_COLLECTION, *PHID_COLLECTION; @@ -223,11 +219,9 @@ UCHAR Type; UCHAR ReportID; ULONG ReportSize; - ULONG ItemCount; ULONG ItemAllocated; - PHID_REPORT_ITEM* Items; - + HID_REPORT_ITEM Items[1]; }HID_REPORT, *PHID_REPORT; typedef struct @@ -258,16 +252,6 @@ ULONG CollectionIndex; }HID_PARSER_CONTEXT, *PHID_PARSER_CONTEXT; - -HIDPARSER_STATUS -HidParser_ParseReportDescriptor( - PHID_PARSER Parser, - PUCHAR Report, - ULONG ReportSize); - -ULONG -HidParser_NumberOfTopCollections( - IN PHID_PARSER Parser); #define HID_REPORT_TYPE_INPUT 0x01 #define HID_REPORT_TYPE_OUTPUT 0x02 @@ -275,16 +259,14 @@ HIDPARSER_STATUS HidParser_GetCollectionUsagePage( - IN PHID_PARSER Parser, - IN ULONG CollectionNumber, + IN PVOID CollectionContext, OUT PUSHORT Usage, OUT PUSHORT UsagePage); ULONG HidParser_GetReportLength( - IN PHID_PARSER Parser, - IN ULONG CollectionIndex, - IN ULONG ReportType); + IN PVOID CollectionContext, + IN UCHAR ReportType); UCHAR HidParser_IsReportIDUsed( @@ -292,44 +274,26 @@ ULONG HidParser_GetReportItemCountFromReportType( - IN PHID_PARSER Parser, - IN ULONG CollectionNumber, - IN ULONG ReportType); + IN PVOID CollectionContext, + IN UCHAR ReportType); ULONG HidParser_GetReportItemTypeCountFromReportType( - IN PHID_PARSER Parser, - IN ULONG CollectionNumber, - IN ULONG ReportType, + IN PVOID CollectionContext, + IN UCHAR ReportType, IN ULONG bData); ULONG -HidParser_GetContextSize( - IN PHID_PARSER Parser, - IN ULONG CollectionNumber); - -VOID -HidParser_FreeContext( - IN PHID_PARSER Parser, - IN PUCHAR Context, - IN ULONG ContextLength); - -ULONG -HidParser_GetTotalCollectionCount( - IN PHID_PARSER Parser); - -ULONG HidParser_GetMaxUsageListLengthWithReportAndPage( - IN PHID_PARSER Parser, - IN ULONG CollectionNumber, - IN ULONG ReportType, + IN PVOID CollectionContext, + IN UCHAR ReportType, IN USAGE UsagePage OPTIONAL); HIDPARSER_STATUS HidParser_GetSpecificValueCapsWithReport( IN PHID_PARSER Parser, - IN ULONG CollectionNumber, - IN ULONG ReportType, + IN PVOID CollectionContext, + IN UCHAR ReportType, IN USHORT UsagePage, IN USHORT Usage, OUT PHIDP_VALUE_CAPS ValueCaps, @@ -339,8 +303,8 @@ HIDPARSER_STATUS HidParser_GetUsagesWithReport( IN PHID_PARSER Parser, - IN ULONG CollectionNumber, - IN ULONG ReportType, + IN PVOID CollectionContext, + IN UCHAR ReportType, IN USAGE UsagePage, OUT USAGE *UsageList, IN OUT PULONG UsageLength, @@ -350,15 +314,64 @@ HIDPARSER_STATUS HidParser_GetScaledUsageValueWithReport( IN PHID_PARSER Parser, - IN ULONG CollectionNumber, - IN ULONG ReportType, + IN PVOID CollectionContext, + IN UCHAR ReportType, IN USAGE UsagePage, IN USAGE Usage, OUT PLONG UsageValue, IN PCHAR ReportDescriptor, IN ULONG ReportDescriptorLength); -ULONG -HidParser_GetCollectionNumberFromParserContext( - IN PHID_PARSER Parser); - +/* parser.c */ + +HIDPARSER_STATUS +HidParser_BuildContext( + IN PHID_PARSER Parser, + IN PVOID ParserContext, + IN ULONG CollectionIndex, + IN ULONG ContextSize, + OUT PVOID *CollectionContext); + +ULONG +HidParser_CalculateContextSize( + IN PHID_COLLECTION Collection); + +HIDPARSER_STATUS +HidParser_ParseReportDescriptor( + PHID_PARSER Parser, + PUCHAR Report, + ULONG ReportSize, + OUT PVOID *ParserContext); + +ULONG +HidParser_NumberOfTopCollections( + IN PVOID ParserContext); + +ULONG +HidParser_GetContextSize( + IN PHID_PARSER Parser, + IN PVOID ParserContext, + IN ULONG CollectionNumber); + + +/* context.c */ + +PHID_COLLECTION +HidParser_GetCollectionFromContext( + IN PVOID Context); + +ULONG +HidParser_GetTotalCollectionCount( + IN PVOID CollectionContext); + +HIDPARSER_STATUS +HidParser_BuildCollectionContext( + IN PHID_PARSER Parser, + IN PHID_COLLECTION RootCollection, + IN PVOID Context, + IN ULONG ContextSize); + +PHID_REPORT +HidParser_GetReportInCollection( + IN PVOID Context, + IN UCHAR ReportType);

1 0

[cgutman] 55352: [USBSTOR] - Add forgotten event initialization from last usbstor commit
by cgutman＠svn.reactos.org 31 Jan '12

31 Jan '12

Author: cgutman Date: Tue Jan 31 15:59:59 2012 New Revision: 55352 URL: http://svn.reactos.org/svn/reactos?rev=55352&view=rev Log: [USBSTOR] - Add forgotten event initialization from last usbstor commit Modified: branches/usb-bringup-trunk/drivers/usb/usbstor/queue.c Modified: branches/usb-bringup-trunk/drivers/usb/usbstor/queue.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/usb/u… ============================================================================== --- branches/usb-bringup-trunk/drivers/usb/usbstor/queue.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/usb/usbstor/queue.c [iso-8859-1] Tue Jan 31 15:59:59 2012 @@ -29,6 +29,11 @@ // initialize irp list head // InitializeListHead(&FDODeviceExtension->IrpListHead); + + // + // initialize event + // + KeInitializeEvent(&FDODeviceExtension->NoPendingRequests, NotificationEvent, TRUE); } VOID

1 0

[cgutman] 55351: [USETUP] - Don't quit if setup doesn't find a non-PnP HDD (until PnP stuff works better)
by cgutman＠svn.reactos.org 31 Jan '12

31 Jan '12

Author: cgutman Date: Tue Jan 31 15:45:43 2012 New Revision: 55351 URL: http://svn.reactos.org/svn/reactos?rev=55351&view=rev Log: [USETUP] - Don't quit if setup doesn't find a non-PnP HDD (until PnP stuff works better) Modified: branches/usb-bringup-trunk/base/setup/usetup/interface/usetup.c Modified: branches/usb-bringup-trunk/base/setup/usetup/interface/usetup.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/base/setup/us… ============================================================================== --- branches/usb-bringup-trunk/base/setup/usetup/interface/usetup.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/base/setup/usetup/interface/usetup.c [iso-8859-1] Tue Jan 31 15:45:43 2012 @@ -728,19 +728,19 @@ static PAGE_NUMBER SetupStartPage(PINPUT_RECORD Ir) { - SYSTEM_DEVICE_INFORMATION Sdi; + //SYSTEM_DEVICE_INFORMATION Sdi; NTSTATUS Status; WCHAR FileNameBuffer[MAX_PATH]; INFCONTEXT Context; PWCHAR Value; UINT ErrorLine; - ULONG ReturnSize; + //ULONG ReturnSize; PGENERIC_LIST_ENTRY ListEntry; INT IntValue; CONSOLE_SetStatusText(MUIGetString(STRING_PLEASEWAIT)); - +#if 0 /* Check whether a harddisk is available */ Status = NtQuerySystemInformation(SystemDeviceInformation, &Sdi, @@ -759,6 +759,7 @@ MUIDisplayError(ERROR_NO_HDD, Ir, POPUP_WAIT_ENTER); return QUIT_PAGE; } +#endif /* Get the source path and source root path */ Status = GetSourcePaths(&SourcePath,

1 0

[cgutman] 55350: [USBSTOR] - Fix handling of SRB_FUNCTION_FLUSH and SRB_FUNCTION_SHUTDOWN to prevent data loss
by cgutman＠svn.reactos.org 31 Jan '12

31 Jan '12

Author: cgutman Date: Tue Jan 31 15:33:32 2012 New Revision: 55350 URL: http://svn.reactos.org/svn/reactos?rev=55350&view=rev Log: [USBSTOR] - Fix handling of SRB_FUNCTION_FLUSH and SRB_FUNCTION_SHUTDOWN to prevent data loss Modified: branches/usb-bringup-trunk/drivers/usb/usbstor/disk.c branches/usb-bringup-trunk/drivers/usb/usbstor/queue.c branches/usb-bringup-trunk/drivers/usb/usbstor/usbstor.h Modified: branches/usb-bringup-trunk/drivers/usb/usbstor/disk.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/usb/u… ============================================================================== --- branches/usb-bringup-trunk/drivers/usb/usbstor/disk.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/usb/usbstor/disk.c [iso-8859-1] Tue Jan 31 15:33:32 2012 @@ -175,9 +175,9 @@ DPRINT1("SRB_FUNCTION_FLUSH / SRB_FUNCTION_FLUSH_QUEUE / SRB_FUNCTION_SHUTDOWN\n"); // - // flush all requests - // - USBSTOR_QueueFlushIrps(PDODeviceExtension->LowerDeviceObject); + // wait for pending requests to finish + // + USBSTOR_QueueWaitForPendingRequests(PDODeviceExtension->LowerDeviceObject); // // set status success Modified: branches/usb-bringup-trunk/drivers/usb/usbstor/queue.c URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/usb/u… ============================================================================== --- branches/usb-bringup-trunk/drivers/usb/usbstor/queue.c [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/usb/usbstor/queue.c [iso-8859-1] Tue Jan 31 15:33:32 2012 @@ -176,12 +176,18 @@ // add irp to queue // InsertTailList(&FDODeviceExtension->IrpListHead, &Irp->Tail.Overlay.ListEntry); - } - - // - // increment pending count - // - FDODeviceExtension->IrpPendingCount++; + } + + // + // increment pending count + // + FDODeviceExtension->IrpPendingCount++; + + + // + // clear the no requests pending event + // + KeClearEvent(&FDODeviceExtension->NoPendingRequests); // // check if queue is freezed @@ -296,105 +302,24 @@ } VOID -USBSTOR_QueueFlushIrps( +USBSTOR_QueueWaitForPendingRequests( IN PDEVICE_OBJECT DeviceObject) { - KIRQL OldLevel; - PFDO_DEVICE_EXTENSION FDODeviceExtension; - PLIST_ENTRY Entry; - PIRP Irp; - PIO_STACK_LOCATION IoStack; - PSCSI_REQUEST_BLOCK Request; - KIRQL OldIrql; - - // - // get FDO device extension - // - FDODeviceExtension = (PFDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; - - // - // sanity check - // - ASSERT(FDODeviceExtension->Common.IsFDO); - - // - // acquire lock - // - KeAcquireSpinLock(&FDODeviceExtension->IrpListLock, &OldLevel); - - // - // complete all irps with status cancelled - // - while(!IsListEmpty(&FDODeviceExtension->IrpListHead)) - { - // - // remove irp - // - Entry = RemoveHeadList(&FDODeviceExtension->IrpListHead); - - // - // get start of irp structure - // - Irp = (PIRP)CONTAINING_RECORD(Entry, IRP, Tail.Overlay.ListEntry); - - // - // remove the cancellation routine - // - IoAcquireCancelSpinLock(&OldIrql); - (void)IoSetCancelRoutine(Irp, NULL); - IoReleaseCancelSpinLock(OldIrql); - - // - // get current stack location - // - IoStack = IoGetCurrentIrpStackLocation(Irp); - - // - // get request block - // - Request = (PSCSI_REQUEST_BLOCK)IoStack->Parameters.Others.Argument1; - - // - // sanity check - // - ASSERT(Request); - - // - // set srb status to flushed - // - Request->SrbStatus = SRB_STATUS_REQUEST_FLUSHED; - - // - // set unsuccessful status - // - Irp->IoStatus.Status = STATUS_UNSUCCESSFUL; - - // - // release lock - // - KeReleaseSpinLock(&FDODeviceExtension->IrpListLock, OldLevel); - - // - // complete request - // - USBSTOR_QueueTerminateRequest(DeviceObject, Irp); - IoCompleteRequest(Irp, IO_NO_INCREMENT); - - // - // start next one - // - USBSTOR_QueueNextRequest(DeviceObject); - - // - // acquire lock - // - KeAcquireSpinLock(&FDODeviceExtension->IrpListLock, &OldLevel); - } - - // - // release lock - // - KeReleaseSpinLock(&FDODeviceExtension->IrpListLock, OldLevel); + PFDO_DEVICE_EXTENSION FDODeviceExtension; + + // + // get FDO device extension + // + FDODeviceExtension = (PFDO_DEVICE_EXTENSION)DeviceObject->DeviceExtension; + + // + // perform the wait + // + KeWaitForSingleObject(&FDODeviceExtension->NoPendingRequests, + Executive, + KernelMode, + FALSE, + NULL); } VOID @@ -436,6 +361,15 @@ // indicate processing is completed // FDODeviceExtension->ActiveSrb = NULL; + } + + // + // Set the event if nothing else is pending + // + if (FDODeviceExtension->IrpPendingCount == 0 && + FDODeviceExtension->ActiveSrb == NULL) + { + KeSetEvent(&FDODeviceExtension->NoPendingRequests, IO_NO_INCREMENT, FALSE); } // Modified: branches/usb-bringup-trunk/drivers/usb/usbstor/usbstor.h URL: http://svn.reactos.org/svn/reactos/branches/usb-bringup-trunk/drivers/usb/u… ============================================================================== --- branches/usb-bringup-trunk/drivers/usb/usbstor/usbstor.h [iso-8859-1] (original) +++ branches/usb-bringup-trunk/drivers/usb/usbstor/usbstor.h [iso-8859-1] Tue Jan 31 15:33:32 2012 @@ -67,6 +67,7 @@ BOOLEAN ResetInProgress; // if hard reset is in progress ULONG IrpPendingCount; // count of irp pending PSCSI_REQUEST_BLOCK ActiveSrb; // stores the current active SRB + KEVENT NoPendingRequests; // set if no pending or in progress requests }FDO_DEVICE_EXTENSION, *PFDO_DEVICE_EXTENSION; typedef struct @@ -405,7 +406,7 @@ PIRP Irp); VOID -USBSTOR_QueueFlushIrps( +USBSTOR_QueueWaitForPendingRequests( IN PDEVICE_OBJECT DeviceObject); VOID

1 0

[gadamopoulos] 55349: [user32_apitest] - Add some more tests
by gadamopoulos＠svn.reactos.org 31 Jan '12

31 Jan '12

Author: gadamopoulos Date: Tue Jan 31 10:49:44 2012 New Revision: 55349 URL: http://svn.reactos.org/svn/reactos?rev=55349&view=rev Log: [user32_apitest] - Add some more tests Modified: trunk/rostests/apitests/user32/SystemParametersInfo.c Modified: trunk/rostests/apitests/user32/SystemParametersInfo.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/user32/SystemPar… ============================================================================== --- trunk/rostests/apitests/user32/SystemParametersInfo.c [iso-8859-1] (original) +++ trunk/rostests/apitests/user32/SystemParametersInfo.c [iso-8859-1] Tue Jan 31 10:49:44 2012 @@ -150,6 +150,27 @@ ok(ret, "SystemParametersInfo failed\n"); ok(temp == 1, "SPI_GETMOUSESPEED did not get value set by SPI_SETMOUSESPEED (%d instead of 1)\n", (int)temp); + temp = 20; + ret = SystemParametersInfo(SPI_SETMOUSESPEED, 0, (PVOID)temp, SPIF_SENDCHANGE | SPIF_UPDATEINIFILE); + ok(ret, "SystemParametersInfo failed\n"); + ret = SystemParametersInfo(SPI_GETMOUSESPEED, 0, &temp, 0); + ok(ret, "SystemParametersInfo failed\n"); + ok(temp == 20, "SPI_GETMOUSESPEED did not get value set by SPI_SETMOUSESPEED (%d instead of 20)\n", (int)temp); + + temp = 21; + ret = SystemParametersInfo(SPI_SETMOUSESPEED, 0, (PVOID)temp, SPIF_SENDCHANGE | SPIF_UPDATEINIFILE); + ok(!ret, "expected SystemParametersInfo to fail\n"); + ret = SystemParametersInfo(SPI_GETMOUSESPEED, 0, &temp, 0); + ok(ret, "SystemParametersInfo failed\n"); + ok(temp == 20, "SPI_GETMOUSESPEED got unexpected value (%d instead of 20)\n", (int)temp); + + temp = 0; + ret = SystemParametersInfo(SPI_SETMOUSESPEED, 0, (PVOID)temp, SPIF_SENDCHANGE | SPIF_UPDATEINIFILE); + ok(!ret, "expected SystemParametersInfo to fail\n"); + ret = SystemParametersInfo(SPI_GETMOUSESPEED, 0, &temp, 0); + ok(ret, "SystemParametersInfo failed\n"); + ok(temp == 20, "SPI_GETMOUSESPEED got unexpected value (%d instead of 20)\n", (int)temp); + ret = SystemParametersInfo(SPI_SETMOUSESPEED, 0, (PVOID)ulMouseSpeed, SPIF_SENDCHANGE | SPIF_UPDATEINIFILE); ok(ret, "SystemParametersInfo failed\n"); }

1 0

[janderwald] 55348: - Remove old version of usbehci and usbhub
by janderwald＠svn.reactos.org 31 Jan '12

31 Jan '12

Author: janderwald Date: Tue Jan 31 10:11:39 2012 New Revision: 55348 URL: http://svn.reactos.org/svn/reactos?rev=55348&view=rev Log: - Remove old version of usbehci and usbhub Removed: branches/usb-bringup-trunk/drivers/usb/usbehci/ branches/usb-bringup-trunk/drivers/usb/usbhub/

1 0

[gadamopoulos] 55347: [user32_apitest] - Add a couple of tests for SPI_SETMOUSESPEED and SPI_GETMOUSESPEED - inspired by bug 905
by gadamopoulos＠svn.reactos.org 31 Jan '12

31 Jan '12

Author: gadamopoulos Date: Tue Jan 31 09:58:05 2012 New Revision: 55347 URL: http://svn.reactos.org/svn/reactos?rev=55347&view=rev Log: [user32_apitest] - Add a couple of tests for SPI_SETMOUSESPEED and SPI_GETMOUSESPEED - inspired by bug 905 Modified: trunk/rostests/apitests/user32/SystemParametersInfo.c Modified: trunk/rostests/apitests/user32/SystemParametersInfo.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/apitests/user32/SystemPar… ============================================================================== --- trunk/rostests/apitests/user32/SystemParametersInfo.c [iso-8859-1] (original) +++ trunk/rostests/apitests/user32/SystemParametersInfo.c [iso-8859-1] Tue Jan 31 09:58:05 2012 @@ -134,6 +134,26 @@ COMPARE_CACHE(CaptionHeight_chain); } +static void Test_MouseSpeed() +{ + ULONG ulMouseSpeed, temp; + BOOL ret; + + ret = SystemParametersInfo(SPI_GETMOUSESPEED, 0, &ulMouseSpeed, 0); + ok(ret, "SystemParametersInfo failed\n"); + ok(ulMouseSpeed >= 1 && ulMouseSpeed <=20, "Wrong mouse speed (%d)\n", (int)ulMouseSpeed); + + temp = 1; + ret = SystemParametersInfo(SPI_SETMOUSESPEED, 0, (PVOID)temp, SPIF_SENDCHANGE | SPIF_UPDATEINIFILE); + ok(ret, "SystemParametersInfo failed\n"); + ret = SystemParametersInfo(SPI_GETMOUSESPEED, 0, &temp, 0); + ok(ret, "SystemParametersInfo failed\n"); + ok(temp == 1, "SPI_GETMOUSESPEED did not get value set by SPI_SETMOUSESPEED (%d instead of 1)\n", (int)temp); + + ret = SystemParametersInfo(SPI_SETMOUSESPEED, 0, (PVOID)ulMouseSpeed, SPIF_SENDCHANGE | SPIF_UPDATEINIFILE); + ok(ret, "SystemParametersInfo failed\n"); +} + START_TEST(SystemParametersInfo) { RegisterSimpleClass(SysParamsTestProc, L"sysparamstest"); @@ -144,6 +164,7 @@ 200, 200, 300, 300, NULL, NULL, 0, NULL); Test_NonClientMetrics(); + Test_MouseSpeed(); DestroyWindow(hWnd1); DestroyWindow(hWnd2);

1 0

[ion] 55346: [SMSS2]: Implement helper functions for reading and writing to the BSD (What lets Windows know if this was a good boot/shutdown or a bad one, and to display the recovery settings or no...
by ion＠svn.reactos.org 31 Jan '12

31 Jan '12

Author: ion Date: Tue Jan 31 02:38:42 2012 New Revision: 55346 URL: http://svn.reactos.org/svn/reactos?rev=55346&view=rev Log: [SMSS2]: Implement helper functions for reading and writing to the BSD (What lets Windows know if this was a good boot/shutdown or a bad one, and to display the recovery settings or not (FreeLDR should do this...)). Implement the functions needed to launch native applications, and add support for launching autochk and any other bootexecute applications. Set the correct SessionID with the kernel by calling ProcessSessionInformation. Pending File Rename Operations are the last big thing left, plus actual pagefile creation. [SMSS2]: Fix a bug in SmpParseCommandLine. Modified: trunk/reactos/base/system/smss2/smsessn.c trunk/reactos/base/system/smss2/smss.c trunk/reactos/base/system/smss2/smss.h trunk/reactos/base/system/smss2/smsubsys.c trunk/reactos/base/system/smss2/smutil.c Modified: trunk/reactos/base/system/smss2/smsessn.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss2/smsessn.… ============================================================================== --- trunk/reactos/base/system/smss2/smsessn.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss2/smsessn.c [iso-8859-1] Tue Jan 31 02:38:42 2012 @@ -22,3 +22,25 @@ HANDLE SmpSessionsObjectDirectory; /* FUNCTIONS ******************************************************************/ + +NTSTATUS +NTAPI +SmpSetProcessMuSessionId(IN HANDLE ProcessHandle, + IN ULONG SessionId) +{ + NTSTATUS Status; + + /* Tell the kernel about our session ID */ + Status = NtSetInformationProcess(ProcessHandle, + ProcessSessionInformation, + &SessionId, + sizeof(SessionId)); + if (!NT_SUCCESS(Status)) + { + DPRINT1("SMSS: SetProcessMuSessionId, Process=%x, Status=%x\n", + ProcessHandle, Status); + } + + /* Return */ + return Status; +} Modified: trunk/reactos/base/system/smss2/smss.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss2/smss.c?r… ============================================================================== --- trunk/reactos/base/system/smss2/smss.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss2/smss.c [iso-8859-1] Tue Jan 31 02:38:42 2012 @@ -28,19 +28,271 @@ UNICODE_STRING SmpSystemRoot; ULONG AttachedSessionId = -1; -BOOLEAN SmpDebug; +BOOLEAN SmpDebug, SmpEnableDots; /* FUNCTIONS ******************************************************************/ + +/* GCC's incompetence strikes again */ +VOID +sprintf_nt(IN PCHAR Buffer, + IN PCHAR Format, + IN ...) +{ + va_list ap; + va_start(ap, Format); + sprintf(Buffer, Format, ap); + va_end(ap); +} + +NTSTATUS +NTAPI +SmpExecuteImage(IN PUNICODE_STRING FileName, + IN PUNICODE_STRING Directory, + IN PUNICODE_STRING CommandLine, + IN ULONG MuSessionId, + IN ULONG Flags, + IN PRTL_USER_PROCESS_INFORMATION ProcessInformation) +{ + PRTL_USER_PROCESS_INFORMATION ProcessInfo; + NTSTATUS Status; + RTL_USER_PROCESS_INFORMATION LocalProcessInfo; + PRTL_USER_PROCESS_PARAMETERS ProcessParameters; + + /* Use the input process information if we have it, otherwise use local */ + ProcessInfo = ProcessInformation; + if (!ProcessInfo) ProcessInfo = &LocalProcessInfo; + + /* Create parameters for the target process */ + Status = RtlCreateProcessParameters(&ProcessParameters, + FileName, + SmpDefaultLibPath.Length ? + &SmpDefaultLibPath : NULL, + Directory, + CommandLine, + SmpDefaultEnvironment, + NULL, + NULL, + NULL, + 0); + if (!NT_SUCCESS(Status)) + { + /* This is a pretty bad failure. ASSERT on checked builds and exit */ + ASSERTMSG(NT_SUCCESS(Status), "RtlCreateProcessParameters"); + DPRINT1("SMSS: RtlCreateProcessParameters failed for %wZ - Status == %lx\n", + FileName, Status); + return Status; + } + + /* Set the size field as required */ + ProcessInfo->Size = sizeof(RTL_USER_PROCESS_INFORMATION); + + /* Check if the debug flag was requested */ + if (Flags & SMP_DEBUG_FLAG) + { + /* Write it in the process parameters */ + ProcessParameters->DebugFlags = 1; + } + else + { + /* Otherwise inherit the flag that was passed to SMSS itself */ + ProcessParameters->DebugFlags = SmpDebug; + } + + /* Subsystems get the first 1MB of memory reserved for DOS/IVT purposes */ + if (Flags & SMP_SUBSYSTEM_FLAG) + { + ProcessParameters->Flags |= RTL_USER_PROCESS_PARAMETERS_RESERVE_1MB; + } + + /* And always force NX for anything that SMSS launches */ + ProcessParameters->Flags |= RTL_USER_PROCESS_PARAMETERS_NX; + + /* Now create the process */ + Status = RtlCreateUserProcess(FileName, + OBJ_CASE_INSENSITIVE, + ProcessParameters, + NULL, + NULL, + NULL, + FALSE, + NULL, + NULL, + ProcessInfo); + RtlDestroyProcessParameters(ProcessParameters); + if (!NT_SUCCESS(Status)) + { + /* If we couldn't create it, fail back to the caller */ + DPRINT1("SMSS: Failed load of %wZ - Status == %lx\n", + FileName, Status); + return Status; + } + + /* Associate a session with this process */ + Status = SmpSetProcessMuSessionId(ProcessInfo->ProcessHandle, MuSessionId); + + /* If the application is deferred (suspended), there's nothing to do */ + if (Flags & SMP_DEFERRED_FLAG) return Status; + + /* Otherwise, get ready to start it, but make sure it's a native app */ + if (ProcessInfo->ImageInformation.SubSystemType == IMAGE_SUBSYSTEM_NATIVE) + { + /* Resume it */ + NtResumeThread(ProcessInfo->ThreadHandle, NULL); + if (!(Flags & SMP_ASYNC_FLAG)) + { + /* Block on it unless Async was requested */ + NtWaitForSingleObject(ProcessInfo->ThreadHandle, FALSE, NULL); + } + + /* It's up and running now, close our handles */ + NtClose(ProcessInfo->ThreadHandle); + NtClose(ProcessInfo->ProcessHandle); + } + else + { + /* This image is invalid, so kill it, close our handles, and fail */ + Status = STATUS_INVALID_IMAGE_FORMAT; + NtTerminateProcess(ProcessInfo->ProcessHandle, Status); + NtWaitForSingleObject(ProcessInfo->ThreadHandle, 0, 0); + NtClose(ProcessInfo->ThreadHandle); + NtClose(ProcessInfo->ProcessHandle); + DPRINT1("SMSS: Not an NT image - %wZ\n", FileName); + } + + /* Return the outcome of the process create */ + return Status; +} + +NTSTATUS +NTAPI +SmpInvokeAutoChk(IN PUNICODE_STRING FileName, + IN PUNICODE_STRING Directory, + IN PUNICODE_STRING Arguments, + IN ULONG Flags) +{ + ANSI_STRING DestinationString; + CHAR SourceString[256]; + UNICODE_STRING Destination; + WCHAR Buffer[1024]; + BOOLEAN BootState, BootOkay, ShutdownOkay; + + /* Check if autochk should show dots (if the user booted with /SOS) */ + if (SmpQueryRegistrySosOption()) SmpEnableDots = FALSE; + + /* Make sure autochk was actually found */ + if (Flags & SMP_INVALID_PATH) + { + /* It wasn't, so create an error message to print on the screen */ + sprintf_nt(SourceString, + "%wZ program not found - skipping AUTOCHECK\n", + FileName); + RtlInitAnsiString(&DestinationString, SourceString); + if (RtlAnsiStringToUnicodeString(&Destination, + &DestinationString, + TRUE)) + { + /* And show it */ + NtDisplayString(&Destination); + RtlFreeUnicodeString(&Destination); + } + } + else + { + /* Autochk is there, so record the BSD state */ + BootState = SmpSaveAndClearBootStatusData(&BootOkay, &ShutdownOkay); + + /* Build the path to autochk and place its arguments */ + RtlInitEmptyUnicodeString(&Destination, Buffer, sizeof(Buffer)); + RtlAppendUnicodeStringToString(&Destination, FileName); + RtlAppendUnicodeToString(&Destination, L" "); + RtlAppendUnicodeStringToString(&Destination, Arguments); + + /* Execute it */ + SmpExecuteImage(FileName, + Directory, + &Destination, + 0, + Flags & ~SMP_AUTOCHK_FLAG, + NULL); + + /* Restore the BSD state */ + if (BootState) SmpRestoreBootStatusData(BootOkay, ShutdownOkay); + } + + /* We're all done! */ + return STATUS_SUCCESS; +} NTSTATUS NTAPI SmpExecuteCommand(IN PUNICODE_STRING CommandLine, IN ULONG MuSessionId, - OUT PULONG ProcessId, + OUT PHANDLE ProcessId, IN ULONG Flags) { - DPRINT1("Should run: %wZ for session %d\n", CommandLine, MuSessionId); - return STATUS_SUCCESS; + NTSTATUS Status; + UNICODE_STRING Arguments, Directory, FileName; + + /* There's no longer a debugging subsystem */ + if (Flags & SMP_DEBUG_FLAG) return STATUS_SUCCESS; + + /* Parse the command line to see what execution flags are requested */ + Status = SmpParseCommandLine(CommandLine, + &Flags, + &FileName, + &Directory, + &Arguments); + if (!NT_SUCCESS(Status)) + { + /* Fail if we couldn't do that */ + DPRINT1("SMSS: SmpParseCommand( %wZ ) failed - Status == %lx\n", + CommandLine, Status); + return Status; + } + + /* Check if autochk is requested */ + if (Flags & SMP_AUTOCHK_FLAG) + { + /* Run it */ + Status = SmpInvokeAutoChk(&FileName, &Directory, &Arguments, Flags); + } + else if (Flags & SMP_SUBSYSTEM_FLAG) + { + Status = SmpLoadSubSystem(&FileName, + &Directory, + CommandLine, + MuSessionId, + ProcessId); + } + else if (Flags & SMP_INVALID_PATH) + { + /* An invalid image was specified, fail */ + DPRINT1("SMSS: Image file (%wZ) not found\n", &FileName); + Status = STATUS_OBJECT_NAME_NOT_FOUND; + } + else + { + /* An actual image name was present -- execute it */ + Status = SmpExecuteImage(&FileName, + &Directory, + CommandLine, + MuSessionId, + Flags, + NULL); + } + + /* Free all the token parameters */ + if (FileName.Buffer) RtlFreeHeap(RtlGetProcessHeap(), 0, FileName.Buffer); + if (Directory.Buffer) RtlFreeHeap(RtlGetProcessHeap(), 0, Directory.Buffer); + if (Arguments.Buffer) RtlFreeHeap(RtlGetProcessHeap(), 0, Arguments.Buffer); + + /* Return to the caller */ + if (!NT_SUCCESS(Status)) + { + DPRINT1("SMSS: Command '%wZ' failed - Status == %x\n", + CommandLine, Status); + } + return Status; } NTSTATUS Modified: trunk/reactos/base/system/smss2/smss.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss2/smss.h?r… ============================================================================== --- trunk/reactos/base/system/smss2/smss.h [iso-8859-1] (original) +++ trunk/reactos/base/system/smss2/smss.h [iso-8859-1] Tue Jan 31 02:38:42 2012 @@ -33,7 +33,9 @@ #define SMP_DEBUG_FLAG 0x01 #define SMP_ASYNC_FLAG 0x02 #define SMP_AUTOCHK_FLAG 0x04 +#define SMP_SUBSYSTEM_FLAG 0x08 #define SMP_INVALID_PATH 0x10 +#define SMP_DEFERRED_FLAG 0x20 /* EXTERNALS ******************************************************************/ @@ -102,7 +104,7 @@ SmpExecuteCommand( IN PUNICODE_STRING CommandLine, IN ULONG MuSessionId, - OUT PULONG ProcessId, + OUT PHANDLE ProcessId, IN ULONG Flags ); @@ -141,3 +143,42 @@ OUT PUNICODE_STRING Directory, OUT PUNICODE_STRING Arguments ); + +NTSTATUS +NTAPI +SmpLoadSubSystem( + IN PUNICODE_STRING FileName, + IN PUNICODE_STRING Directory, + IN PUNICODE_STRING CommandLine, + IN ULONG MuSessionId, + OUT PHANDLE ProcessId +); + +NTSTATUS +NTAPI +SmpSetProcessMuSessionId( + IN HANDLE ProcessHandle, + IN ULONG SessionId +); + +BOOLEAN +NTAPI +SmpQueryRegistrySosOption( + VOID +); + +BOOLEAN +NTAPI +SmpSaveAndClearBootStatusData( + OUT PBOOLEAN BootOkay, + OUT PBOOLEAN ShutdownOkay +); + +VOID +NTAPI +SmpRestoreBootStatusData( + IN BOOLEAN BootOkay, + IN BOOLEAN ShutdownOkay +); + + Modified: trunk/reactos/base/system/smss2/smsubsys.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss2/smsubsys… ============================================================================== --- trunk/reactos/base/system/smss2/smsubsys.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss2/smsubsys.c [iso-8859-1] Tue Jan 31 02:38:42 2012 @@ -24,6 +24,18 @@ NTSTATUS NTAPI +SmpLoadSubSystem(IN PUNICODE_STRING FileName, + IN PUNICODE_STRING Directory, + IN PUNICODE_STRING CommandLine, + IN ULONG MuSessionId, + OUT PHANDLE ProcessId) +{ + DPRINT1("Should start subsystem %wZ for Session: %lx\n", FileName, MuSessionId); + return STATUS_SUCCESS; +} + +NTSTATUS +NTAPI SmpLoadSubSystemsForMuSession(IN PULONG MuSessionId, OUT PHANDLE ProcessId, IN PUNICODE_STRING InitialCommand) Modified: trunk/reactos/base/system/smss2/smutil.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/smss2/smutil.c… ============================================================================== --- trunk/reactos/base/system/smss2/smutil.c [iso-8859-1] (original) +++ trunk/reactos/base/system/smss2/smutil.c [iso-8859-1] Tue Jan 31 02:38:42 2012 @@ -13,6 +13,11 @@ #include "debug.h" /* GLOBALS ********************************************************************/ + +// +// Taken from an ASSERT +// +#define VALUE_BUFFER_SIZE (sizeof(KEY_VALUE_PARTIAL_INFORMATION) + 512) typedef struct _SMP_PRIVILEGE_STATE { @@ -399,8 +404,7 @@ else { /* There is a directory, and a filename -- separate those two */ - FilePart -= sizeof(UNICODE_NULL); - *FilePart = UNICODE_NULL; + *--FilePart = UNICODE_NULL; RtlCreateUnicodeString(Directory, PathBuffer); } } @@ -408,3 +412,139 @@ /* We are done -- move on to the second pass to get the arguments */ return SmpParseToken(&CmdLineCopy, TRUE, Arguments); } + +BOOLEAN +NTAPI +SmpQueryRegistrySosOption(VOID) +{ + NTSTATUS Status; + UNICODE_STRING KeyName, ValueName; + OBJECT_ATTRIBUTES ObjectAttributes; + HANDLE KeyHandle; + WCHAR ValueBuffer[VALUE_BUFFER_SIZE]; + PKEY_VALUE_PARTIAL_INFORMATION PartialInfo = (PVOID)ValueBuffer; + ULONG Length; + + /* Open the key */ + RtlInitUnicodeString(&KeyName, + L"\\Registry\\Machine\\System\\CurrentControlSet\\Control"); + InitializeObjectAttributes(&ObjectAttributes, + &KeyName, + OBJ_CASE_INSENSITIVE, + NULL, + NULL); + Status = NtOpenKey(&KeyHandle, KEY_READ, &ObjectAttributes); + if (!NT_SUCCESS(Status)) + { + DPRINT1("SMSS: can't open control key: 0x%x\n", Status); + return FALSE; + } + + /* Query the value */ + RtlInitUnicodeString(&ValueName, L"SystemStartOptions"); + Status = NtQueryValueKey(KeyHandle, + &ValueName, + KeyValuePartialInformation, + PartialInfo, + sizeof(ValueBuffer), + &Length); + ASSERT(Length < VALUE_BUFFER_SIZE); + NtClose(KeyHandle); + if (!NT_SUCCESS(Status)) + { + DPRINT1("SMSS: can't query value key: 0x%x\n", Status); + return FALSE; + } + + /* Check if it's set to SOS or sos */ + if (!(wcsstr((PWCHAR)PartialInfo->Data, L"SOS")) || + (wcsstr((PWCHAR)PartialInfo->Data, L"sos"))) + { + /* It's not set, return FALSE */ + return FALSE; + } + + /* It's set, return TRUE */ + return TRUE; +} + +BOOLEAN +NTAPI +SmpSaveAndClearBootStatusData(OUT PBOOLEAN BootOkay, + OUT PBOOLEAN ShutdownOkay) +{ + NTSTATUS Status; + BOOLEAN Value = TRUE; + PVOID BootStatusDataHandle; + + /* Assume failure */ + *BootOkay = FALSE; + *ShutdownOkay = FALSE; + + /* Lock the BSD and fail if we couldn't */ + Status = RtlLockBootStatusData(&BootStatusDataHandle); + if (!NT_SUCCESS(Status)) return FALSE; + + /* Read the old settings */ + RtlGetSetBootStatusData(BootStatusDataHandle, + TRUE, + RtlBsdItemBootGood, + BootOkay, + sizeof(BOOLEAN), + NULL); + RtlGetSetBootStatusData(BootStatusDataHandle, + TRUE, + RtlBsdItemBootShutdown, + ShutdownOkay, + sizeof(BOOLEAN), + NULL); + + /* Set new ones indicating we got at least this far */ + RtlGetSetBootStatusData(BootStatusDataHandle, + FALSE, + RtlBsdItemBootGood, + &Value, + sizeof(Value), + NULL); + RtlGetSetBootStatusData(BootStatusDataHandle, + FALSE, + RtlBsdItemBootShutdown, + &Value, + sizeof(Value), + NULL); + + /* Unlock the BSD and return */ + RtlUnlockBootStatusData(BootStatusDataHandle); + return TRUE; +} + +VOID +NTAPI +SmpRestoreBootStatusData(IN BOOLEAN BootOkay, + IN BOOLEAN ShutdownOkay) +{ + NTSTATUS Status; + PVOID BootState; + + /* Lock the BSD */ + Status = RtlLockBootStatusData(&BootState); + if (NT_SUCCESS(Status)) + { + /* Write the bootokay and bootshudown values */ + RtlGetSetBootStatusData(BootState, + FALSE, + RtlBsdItemBootGood, + &BootOkay, + sizeof(BootOkay), + NULL); + RtlGetSetBootStatusData(BootState, + FALSE, + RtlBsdItemBootShutdown, + &ShutdownOkay, + sizeof(ShutdownOkay), + NULL); + + /* Unlock the BSD and return */ + RtlUnlockBootStatusData(BootState); + } +}

1 0

[ion] 55345: [NDK/RTL]: Fix RtlGetSetBootStatusData prototype and add the BSD Items that can be set with it.
by ion＠svn.reactos.org 31 Jan '12

31 Jan '12

Author: ion Date: Tue Jan 31 02:33:02 2012 New Revision: 55345 URL: http://svn.reactos.org/svn/reactos?rev=55345&view=rev Log: [NDK/RTL]: Fix RtlGetSetBootStatusData prototype and add the BSD Items that can be set with it. Modified: trunk/reactos/include/ndk/rtlfuncs.h trunk/reactos/include/ndk/rtltypes.h trunk/reactos/lib/rtl/bootdata.c Modified: trunk/reactos/include/ndk/rtlfuncs.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ndk/rtlfuncs.h?rev… ============================================================================== --- trunk/reactos/include/ndk/rtlfuncs.h [iso-8859-1] (original) +++ trunk/reactos/include/ndk/rtlfuncs.h [iso-8859-1] Tue Jan 31 02:33:02 2012 @@ -3877,12 +3877,12 @@ NTSTATUS NTAPI RtlGetSetBootStatusData( - HANDLE FileHandle, - BOOLEAN WriteMode, - DWORD DataClass, - PVOID Buffer, - ULONG BufferSize, - DWORD DataClass2 + IN HANDLE FileHandle, + IN BOOLEAN WriteMode, + IN RTL_BSD_ITEM_TYPE DataClass, + IN PVOID Buffer, + IN ULONG BufferSize, + OUT PULONG ReturnLength OPTIONAL ); NTSYSAPI Modified: trunk/reactos/include/ndk/rtltypes.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/ndk/rtltypes.h?rev… ============================================================================== --- trunk/reactos/include/ndk/rtltypes.h [iso-8859-1] (original) +++ trunk/reactos/include/ndk/rtltypes.h [iso-8859-1] Tue Jan 31 02:33:02 2012 @@ -366,6 +366,20 @@ #endif /* NTOS_MODE_USER */ #ifdef NTOS_MODE_USER + +// +// Boot Status Data Field Types +// +typedef enum _RTL_BSD_ITEM_TYPE +{ + RtlBsdItemVersionNumber, + RtlBsdItemProductType, + RtlBsdItemAabEnabled, + RtlBsdItemAabTimeout, + RtlBsdItemBootGood, + RtlBsdItemBootShutdown, + RtlBsdItemMax +} RTL_BSD_ITEM_TYPE, *PRTL_BSD_ITEM_TYPE; // // Table and Compare result types Modified: trunk/reactos/lib/rtl/bootdata.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/bootdata.c?rev=553… ============================================================================== --- trunk/reactos/lib/rtl/bootdata.c [iso-8859-1] (original) +++ trunk/reactos/lib/rtl/bootdata.c [iso-8859-1] Tue Jan 31 02:33:02 2012 @@ -686,12 +686,12 @@ */ NTSTATUS NTAPI -RtlGetSetBootStatusData(HANDLE Filehandle, - BOOLEAN WriteMode, - DWORD DataClass, - PVOID Buffer, - ULONG BufferSize, - DWORD DataClass2) +RtlGetSetBootStatusData(IN HANDLE Filehandle, + IN BOOLEAN WriteMode, + IN RTL_BSD_ITEM_TYPE DataClass, + IN PVOID Buffer, + IN ULONG BufferSize, + OUT PULONG ReturnLength) { UNIMPLEMENTED; return STATUS_NOT_IMPLEMENTED;

1 0

[zguo] 55344: Really update the release version this time.
by zguo＠svn.reactos.org 31 Jan '12

31 Jan '12

Author: zguo Date: Tue Jan 31 01:41:32 2012 New Revision: 55344 URL: http://svn.reactos.org/svn/reactos?rev=55344&view=rev Log: Really update the release version this time. Modified: branches/ros-branch-0_3_14/include/reactos/version.h Modified: branches/ros-branch-0_3_14/include/reactos/version.h URL: http://svn.reactos.org/svn/reactos/branches/ros-branch-0_3_14/include/react… ============================================================================== --- branches/ros-branch-0_3_14/include/reactos/version.h [iso-8859-1] (original) +++ branches/ros-branch-0_3_14/include/reactos/version.h [iso-8859-1] Tue Jan 31 01:41:32 2012 @@ -18,8 +18,8 @@ #define __VERSION_H #define KERNEL_VERSION_MAJOR 0 -#define KERNEL_VERSION_MINOR 4 -#define KERNEL_VERSION_PATCH_LEVEL 0 +#define KERNEL_VERSION_MINOR 3 +#define KERNEL_VERSION_PATCH_LEVEL 14 #define COPYRIGHT_YEAR "2012"

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs