* [dpdk-test-report] |WARNING| pw79510 [PATCH v5] windows/netuio: add Windows NetUIO kernel driver
[not found] <1601592908-11262-1-git-send-email-navasile@linux.microsoft.com>
@ 2020-10-01 22:57 ` checkpatch
2020-10-02 0:55 ` [dpdk-test-report] |SUCCESS| pw79510 " 0-day Robot
1 sibling, 0 replies; 3+ messages in thread
From: checkpatch @ 2020-10-01 22:57 UTC (permalink / raw)
To: test-report; +Cc: Narcisa Ana Maria Vasile
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/79510
_coding style issues_
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#513: FILE: windows/netuio/netuio_dev.c:12:
+#pragma alloc_text (PAGE, netuio_create_device)
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#514: FILE: windows/netuio/netuio_dev.c:13:
+#pragma alloc_text (PAGE, netuio_map_hw_resources)
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#515: FILE: windows/netuio/netuio_dev.c:14:
+#pragma alloc_text (PAGE, netuio_free_hw_resources)
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#521: FILE: windows/netuio/netuio_dev.c:20:
+ NTSTATUS status = STATUS_UNSUCCESSFUL;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#523: FILE: windows/netuio/netuio_dev.c:22:
+ PNETUIO_CONTEXT_DATA ctx;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#524: FILE: windows/netuio/netuio_dev.c:23:
+ ctx = netuio_get_context_data(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#526: FILE: windows/netuio/netuio_dev.c:25:
+ ctx->wdf_device = device; // Store for later use$
ERROR:C99_COMMENTS: do not use C99 // comments
#526: FILE: windows/netuio/netuio_dev.c:25:
+ ctx->wdf_device = device; // Store for later use
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#528: FILE: windows/netuio/netuio_dev.c:27:
+ // Obtain the BUS_INTERFACE_STANDARD interface from the Bus Driver$
ERROR:C99_COMMENTS: do not use C99 // comments
#528: FILE: windows/netuio/netuio_dev.c:27:
+ // Obtain the BUS_INTERFACE_STANDARD interface from the Bus Driver
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#529: FILE: windows/netuio/netuio_dev.c:28:
+ status = WdfFdoQueryForInterface(device,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#530: FILE: windows/netuio/netuio_dev.c:29:
+ &GUID_BUS_INTERFACE_STANDARD,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#530: FILE: windows/netuio/netuio_dev.c:29:
+ &GUID_BUS_INTERFACE_STANDARD,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#531: FILE: windows/netuio/netuio_dev.c:30:
+ (PINTERFACE)&ctx->bus_interface,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#531: FILE: windows/netuio/netuio_dev.c:30:
+ (PINTERFACE)&ctx->bus_interface,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#532: FILE: windows/netuio/netuio_dev.c:31:
+ sizeof(BUS_INTERFACE_STANDARD),$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#532: FILE: windows/netuio/netuio_dev.c:31:
+ sizeof(BUS_INTERFACE_STANDARD),$
ERROR:CODE_INDENT: code indent should use tabs where possible
#533: FILE: windows/netuio/netuio_dev.c:32:
+ 1,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#533: FILE: windows/netuio/netuio_dev.c:32:
+ 1,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#534: FILE: windows/netuio/netuio_dev.c:33:
+ NULL);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#534: FILE: windows/netuio/netuio_dev.c:33:
+ NULL);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#535: FILE: windows/netuio/netuio_dev.c:34:
+ if (!NT_SUCCESS(status))$
ERROR:CODE_INDENT: code indent should use tabs where possible
#536: FILE: windows/netuio/netuio_dev.c:35:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#536: FILE: windows/netuio/netuio_dev.c:35:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#538: FILE: windows/netuio/netuio_dev.c:37:
+ // Retrieve the B:D:F details of our device$
ERROR:C99_COMMENTS: do not use C99 // comments
#538: FILE: windows/netuio/netuio_dev.c:37:
+ // Retrieve the B:D:F details of our device
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#539: FILE: windows/netuio/netuio_dev.c:38:
+ PDEVICE_OBJECT pdo = NULL;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#540: FILE: windows/netuio/netuio_dev.c:39:
+ pdo = WdfDeviceWdmGetPhysicalDevice(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#541: FILE: windows/netuio/netuio_dev.c:40:
+ if (pdo) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#542: FILE: windows/netuio/netuio_dev.c:41:
+ ULONG prop = 0, length = 0;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#542: FILE: windows/netuio/netuio_dev.c:41:
+ ULONG prop = 0, length = 0;$
WARNING:LONG_LINE: line over 100 characters
#543: FILE: windows/netuio/netuio_dev.c:42:
+ status = IoGetDeviceProperty(pdo, DevicePropertyBusNumber, sizeof(ULONG), (PVOID)&ctx->addr.bus_num, &length);
ERROR:CODE_INDENT: code indent should use tabs where possible
#543: FILE: windows/netuio/netuio_dev.c:42:
+ status = IoGetDeviceProperty(pdo, DevicePropertyBusNumber, sizeof(ULONG), (PVOID)&ctx->addr.bus_num, &length);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#543: FILE: windows/netuio/netuio_dev.c:42:
+ status = IoGetDeviceProperty(pdo, DevicePropertyBusNumber, sizeof(ULONG), (PVOID)&ctx->addr.bus_num, &length);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#544: FILE: windows/netuio/netuio_dev.c:43:
+ if (!NT_SUCCESS(status)) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#544: FILE: windows/netuio/netuio_dev.c:43:
+ if (!NT_SUCCESS(status)) {$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#544: FILE: windows/netuio/netuio_dev.c:43:
+ if (!NT_SUCCESS(status)) {
+ return status;
WARNING:BRACES: braces {} are not necessary for single statement blocks
#544: FILE: windows/netuio/netuio_dev.c:43:
+ if (!NT_SUCCESS(status)) {
+ return status;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#545: FILE: windows/netuio/netuio_dev.c:44:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#545: FILE: windows/netuio/netuio_dev.c:44:
+ return status;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#546: FILE: windows/netuio/netuio_dev.c:45:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#546: FILE: windows/netuio/netuio_dev.c:45:
+ }$
WARNING:LONG_LINE: line over 100 characters
#548: FILE: windows/netuio/netuio_dev.c:47:
+ status = IoGetDeviceProperty(pdo, DevicePropertyAddress, sizeof(ULONG), (PVOID)&prop, &length);
ERROR:CODE_INDENT: code indent should use tabs where possible
#548: FILE: windows/netuio/netuio_dev.c:47:
+ status = IoGetDeviceProperty(pdo, DevicePropertyAddress, sizeof(ULONG), (PVOID)&prop, &length);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#548: FILE: windows/netuio/netuio_dev.c:47:
+ status = IoGetDeviceProperty(pdo, DevicePropertyAddress, sizeof(ULONG), (PVOID)&prop, &length);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#549: FILE: windows/netuio/netuio_dev.c:48:
+ if (!NT_SUCCESS(status)) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#549: FILE: windows/netuio/netuio_dev.c:48:
+ if (!NT_SUCCESS(status)) {$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#549: FILE: windows/netuio/netuio_dev.c:48:
+ if (!NT_SUCCESS(status)) {
+ return status;
WARNING:BRACES: braces {} are not necessary for single statement blocks
#549: FILE: windows/netuio/netuio_dev.c:48:
+ if (!NT_SUCCESS(status)) {
+ return status;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#550: FILE: windows/netuio/netuio_dev.c:49:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#550: FILE: windows/netuio/netuio_dev.c:49:
+ return status;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#551: FILE: windows/netuio/netuio_dev.c:50:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#551: FILE: windows/netuio/netuio_dev.c:50:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#553: FILE: windows/netuio/netuio_dev.c:52:
+ ctx->addr.func_num = prop & 0x0000FFFF;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#553: FILE: windows/netuio/netuio_dev.c:52:
+ ctx->addr.func_num = prop & 0x0000FFFF;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#554: FILE: windows/netuio/netuio_dev.c:53:
+ ctx->addr.dev_num = ((prop >> 16) & 0x0000FFFF);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#554: FILE: windows/netuio/netuio_dev.c:53:
+ ctx->addr.dev_num = ((prop >> 16) & 0x0000FFFF);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#555: FILE: windows/netuio/netuio_dev.c:54:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#557: FILE: windows/netuio/netuio_dev.c:56:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#563: FILE: windows/netuio/netuio_dev.c:62:
+ DECLARE_UNICODE_STRING_SIZE(symbolic_link, NETUIO_MAX_SYMLINK_LEN);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#565: FILE: windows/netuio/netuio_dev.c:64:
+ PNETUIO_CONTEXT_DATA ctx;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#566: FILE: windows/netuio/netuio_dev.c:65:
+ ctx = netuio_get_context_data(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#568: FILE: windows/netuio/netuio_dev.c:67:
+ // Build symbolic link name as <netuio_symbolic_link>_BDF (bus/device/func)$
ERROR:C99_COMMENTS: do not use C99 // comments
#568: FILE: windows/netuio/netuio_dev.c:67:
+ // Build symbolic link name as <netuio_symbolic_link>_BDF (bus/device/func)
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#569: FILE: windows/netuio/netuio_dev.c:68:
+ RtlUnicodeStringPrintf(&symbolic_link,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#570: FILE: windows/netuio/netuio_dev.c:69:
+ L"%s_%04x%02x%02x",$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#570: FILE: windows/netuio/netuio_dev.c:69:
+ L"%s_%04x%02x%02x",$
ERROR:CODE_INDENT: code indent should use tabs where possible
#571: FILE: windows/netuio/netuio_dev.c:70:
+ NETUIO_DEVICE_SYMBOLIC_LINK_UNICODE,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#571: FILE: windows/netuio/netuio_dev.c:70:
+ NETUIO_DEVICE_SYMBOLIC_LINK_UNICODE,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#572: FILE: windows/netuio/netuio_dev.c:71:
+ ctx->addr.bus_num,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#572: FILE: windows/netuio/netuio_dev.c:71:
+ ctx->addr.bus_num,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#573: FILE: windows/netuio/netuio_dev.c:72:
+ ctx->addr.dev_num,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#573: FILE: windows/netuio/netuio_dev.c:72:
+ ctx->addr.dev_num,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#574: FILE: windows/netuio/netuio_dev.c:73:
+ ctx->addr.func_num);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#574: FILE: windows/netuio/netuio_dev.c:73:
+ ctx->addr.func_num);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#576: FILE: windows/netuio/netuio_dev.c:75:
+ return WdfDeviceCreateSymbolicLink(device, &symbolic_link);$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#580: FILE: windows/netuio/netuio_dev.c:79:
+/*
+Routine Description:
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#590: FILE: windows/netuio/netuio_dev.c:89:
+ NTSTATUS status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#591: FILE: windows/netuio/netuio_dev.c:90:
+ WDFDEVICE device = NULL;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#593: FILE: windows/netuio/netuio_dev.c:92:
+ WDF_OBJECT_ATTRIBUTES deviceAttributes;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#594: FILE: windows/netuio/netuio_dev.c:93:
+ WDF_PNPPOWER_EVENT_CALLBACKS pnpPowerCallbacks;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#595: FILE: windows/netuio/netuio_dev.c:94:
+ WDF_FILEOBJECT_CONFIG fileConfig;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#596: FILE: windows/netuio/netuio_dev.c:95:
+ WDF_OBJECT_ATTRIBUTES fileAttributes;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#598: FILE: windows/netuio/netuio_dev.c:97:
+ PAGED_CODE();$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#600: FILE: windows/netuio/netuio_dev.c:99:
+ // Register PnP power management callbacks$
ERROR:C99_COMMENTS: do not use C99 // comments
#600: FILE: windows/netuio/netuio_dev.c:99:
+ // Register PnP power management callbacks
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#601: FILE: windows/netuio/netuio_dev.c:100:
+ WDF_PNPPOWER_EVENT_CALLBACKS_INIT(&pnpPowerCallbacks);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#602: FILE: windows/netuio/netuio_dev.c:101:
+ pnpPowerCallbacks.EvtDevicePrepareHardware = netuio_evt_prepare_hw;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#603: FILE: windows/netuio/netuio_dev.c:102:
+ pnpPowerCallbacks.EvtDeviceReleaseHardware = netuio_evt_release_hw;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#604: FILE: windows/netuio/netuio_dev.c:103:
+ WdfDeviceInitSetPnpPowerEventCallbacks(DeviceInit, &pnpPowerCallbacks);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#606: FILE: windows/netuio/netuio_dev.c:105:
+ // Register callbacks for when a HANDLE is opened or closed.$
ERROR:C99_COMMENTS: do not use C99 // comments
#606: FILE: windows/netuio/netuio_dev.c:105:
+ // Register callbacks for when a HANDLE is opened or closed.
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#607: FILE: windows/netuio/netuio_dev.c:106:
+ WDF_OBJECT_ATTRIBUTES_INIT_CONTEXT_TYPE(&fileAttributes, NETUIO_FILE_CONTEXT_DATA);$
WARNING:LONG_LINE: line over 100 characters
#608: FILE: windows/netuio/netuio_dev.c:107:
+ WDF_FILEOBJECT_CONFIG_INIT(&fileConfig, WDF_NO_EVENT_CALLBACK, WDF_NO_EVENT_CALLBACK, netuio_evt_file_cleanup);
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#608: FILE: windows/netuio/netuio_dev.c:107:
+ WDF_FILEOBJECT_CONFIG_INIT(&fileConfig, WDF_NO_EVENT_CALLBACK, WDF_NO_EVENT_CALLBACK, netuio_evt_file_cleanup);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#609: FILE: windows/netuio/netuio_dev.c:108:
+ WdfDeviceInitSetFileObjectConfig(DeviceInit, &fileConfig, &fileAttributes);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#611: FILE: windows/netuio/netuio_dev.c:110:
+ WDF_OBJECT_ATTRIBUTES_INIT_CONTEXT_TYPE(&deviceAttributes, NETUIO_CONTEXT_DATA);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#612: FILE: windows/netuio/netuio_dev.c:111:
+ WdfDeviceInitSetIoInCallerContextCallback(DeviceInit, netuio_evt_IO_in_caller_context);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#614: FILE: windows/netuio/netuio_dev.c:113:
+ status = WdfDeviceCreate(&DeviceInit, &deviceAttributes, &device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#616: FILE: windows/netuio/netuio_dev.c:115:
+ if (NT_SUCCESS(status)) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#617: FILE: windows/netuio/netuio_dev.c:116:
+ // Create a device interface so that applications can find and talk to us.$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#617: FILE: windows/netuio/netuio_dev.c:116:
+ // Create a device interface so that applications can find and talk to us.$
ERROR:C99_COMMENTS: do not use C99 // comments
#617: FILE: windows/netuio/netuio_dev.c:116:
+ // Create a device interface so that applications can find and talk to us.
ERROR:CODE_INDENT: code indent should use tabs where possible
#618: FILE: windows/netuio/netuio_dev.c:117:
+ status = WdfDeviceCreateDeviceInterface(device, &GUID_DEVINTERFACE_netUIO, NULL);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#618: FILE: windows/netuio/netuio_dev.c:117:
+ status = WdfDeviceCreateDeviceInterface(device, &GUID_DEVINTERFACE_netUIO, NULL);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#619: FILE: windows/netuio/netuio_dev.c:118:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#621: FILE: windows/netuio/netuio_dev.c:120:
+ if (NT_SUCCESS(status)) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#622: FILE: windows/netuio/netuio_dev.c:121:
+ // Retrieve and store PCI information$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#622: FILE: windows/netuio/netuio_dev.c:121:
+ // Retrieve and store PCI information$
ERROR:C99_COMMENTS: do not use C99 // comments
#622: FILE: windows/netuio/netuio_dev.c:121:
+ // Retrieve and store PCI information
ERROR:CODE_INDENT: code indent should use tabs where possible
#623: FILE: windows/netuio/netuio_dev.c:122:
+ status = get_pci_device_info(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#623: FILE: windows/netuio/netuio_dev.c:122:
+ status = get_pci_device_info(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#624: FILE: windows/netuio/netuio_dev.c:123:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#626: FILE: windows/netuio/netuio_dev.c:125:
+ if (NT_SUCCESS(status)) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#627: FILE: windows/netuio/netuio_dev.c:126:
+ // Create a symbolic link name for user-space access$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#627: FILE: windows/netuio/netuio_dev.c:126:
+ // Create a symbolic link name for user-space access$
ERROR:C99_COMMENTS: do not use C99 // comments
#627: FILE: windows/netuio/netuio_dev.c:126:
+ // Create a symbolic link name for user-space access
ERROR:CODE_INDENT: code indent should use tabs where possible
#628: FILE: windows/netuio/netuio_dev.c:127:
+ status = create_device_specific_symbolic_link(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#628: FILE: windows/netuio/netuio_dev.c:127:
+ status = create_device_specific_symbolic_link(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#629: FILE: windows/netuio/netuio_dev.c:128:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#631: FILE: windows/netuio/netuio_dev.c:130:
+ if (NT_SUCCESS(status)) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#632: FILE: windows/netuio/netuio_dev.c:131:
+ // Initialize the I/O Package and any Queues$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#632: FILE: windows/netuio/netuio_dev.c:131:
+ // Initialize the I/O Package and any Queues$
ERROR:C99_COMMENTS: do not use C99 // comments
#632: FILE: windows/netuio/netuio_dev.c:131:
+ // Initialize the I/O Package and any Queues
ERROR:CODE_INDENT: code indent should use tabs where possible
#633: FILE: windows/netuio/netuio_dev.c:132:
+ status = netuio_queue_initialize(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#633: FILE: windows/netuio/netuio_dev.c:132:
+ status = netuio_queue_initialize(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#634: FILE: windows/netuio/netuio_dev.c:133:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#636: FILE: windows/netuio/netuio_dev.c:135:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#643: FILE: windows/netuio/netuio_dev.c:142:
+ UNREFERENCED_PARAMETER(Resources);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#645: FILE: windows/netuio/netuio_dev.c:144:
+ NTSTATUS status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#647: FILE: windows/netuio/netuio_dev.c:146:
+ PNETUIO_CONTEXT_DATA ctx;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#648: FILE: windows/netuio/netuio_dev.c:147:
+ ctx = netuio_get_context_data(Device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#650: FILE: windows/netuio/netuio_dev.c:149:
+ PCI_COMMON_HEADER pci_config = {0};$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#651: FILE: windows/netuio/netuio_dev.c:150:
+ ULONG bytes_returned;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#653: FILE: windows/netuio/netuio_dev.c:152:
+ // Read PCI configuration space$
ERROR:C99_COMMENTS: do not use C99 // comments
#653: FILE: windows/netuio/netuio_dev.c:152:
+ // Read PCI configuration space
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#654: FILE: windows/netuio/netuio_dev.c:153:
+ bytes_returned = ctx->bus_interface.GetBusData($
ERROR:CODE_INDENT: code indent should use tabs where possible
#655: FILE: windows/netuio/netuio_dev.c:154:
+ ctx->bus_interface.Context,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#655: FILE: windows/netuio/netuio_dev.c:154:
+ ctx->bus_interface.Context,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#656: FILE: windows/netuio/netuio_dev.c:155:
+ PCI_WHICHSPACE_CONFIG,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#656: FILE: windows/netuio/netuio_dev.c:155:
+ PCI_WHICHSPACE_CONFIG,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#657: FILE: windows/netuio/netuio_dev.c:156:
+ &pci_config,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#657: FILE: windows/netuio/netuio_dev.c:156:
+ &pci_config,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#658: FILE: windows/netuio/netuio_dev.c:157:
+ 0,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#658: FILE: windows/netuio/netuio_dev.c:157:
+ 0,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#659: FILE: windows/netuio/netuio_dev.c:158:
+ sizeof(pci_config));$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#659: FILE: windows/netuio/netuio_dev.c:158:
+ sizeof(pci_config));$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#661: FILE: windows/netuio/netuio_dev.c:160:
+ if (bytes_returned != sizeof(pci_config)) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#662: FILE: windows/netuio/netuio_dev.c:161:
+ status = STATUS_NOT_SUPPORTED;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#662: FILE: windows/netuio/netuio_dev.c:161:
+ status = STATUS_NOT_SUPPORTED;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#663: FILE: windows/netuio/netuio_dev.c:162:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#663: FILE: windows/netuio/netuio_dev.c:162:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#664: FILE: windows/netuio/netuio_dev.c:163:
+ }$
WARNING:TYPO_SPELLING: 'implictly' may be misspelled - perhaps 'implicitly'?
#666: FILE: windows/netuio/netuio_dev.c:165:
+ // Device type is implictly enforced by .inf
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#666: FILE: windows/netuio/netuio_dev.c:165:
+ // Device type is implictly enforced by .inf$
ERROR:C99_COMMENTS: do not use C99 // comments
#666: FILE: windows/netuio/netuio_dev.c:165:
+ // Device type is implictly enforced by .inf
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#667: FILE: windows/netuio/netuio_dev.c:166:
+ ASSERT(PCI_CONFIGURATION_TYPE(&pci_config) == PCI_DEVICE_TYPE);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#669: FILE: windows/netuio/netuio_dev.c:168:
+ RtlZeroMemory(ctx->bar, sizeof(ctx->bar));$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#670: FILE: windows/netuio/netuio_dev.c:169:
+ RtlZeroMemory(ctx->dpdk_hw, sizeof(ctx->dpdk_hw));$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#672: FILE: windows/netuio/netuio_dev.c:171:
+ PCM_PARTIAL_RESOURCE_DESCRIPTOR descriptor;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#673: FILE: windows/netuio/netuio_dev.c:172:
+ ULONG next_descriptor = 0;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#674: FILE: windows/netuio/netuio_dev.c:173:
+ ULONG curr_bar = 0;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#675: FILE: windows/netuio/netuio_dev.c:174:
+ ULONG prev_bar = 0;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#698: FILE: windows/netuio/netuio_dev.c:197:
+ for (INT bar_index = 0; bar_index < PCI_MAX_BAR; bar_index++) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#699: FILE: windows/netuio/netuio_dev.c:198:
+ prev_bar = curr_bar;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#699: FILE: windows/netuio/netuio_dev.c:198:
+ prev_bar = curr_bar;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#700: FILE: windows/netuio/netuio_dev.c:199:
+ curr_bar = pci_config.u.type0.BaseAddresses[bar_index];$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#700: FILE: windows/netuio/netuio_dev.c:199:
+ curr_bar = pci_config.u.type0.BaseAddresses[bar_index];$
ERROR:CODE_INDENT: code indent should use tabs where possible
#701: FILE: windows/netuio/netuio_dev.c:200:
+ if (curr_bar == 0 || (prev_bar & PCI_TYPE_64BIT)) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#701: FILE: windows/netuio/netuio_dev.c:200:
+ if (curr_bar == 0 || (prev_bar & PCI_TYPE_64BIT)) {$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#701: FILE: windows/netuio/netuio_dev.c:200:
+ if (curr_bar == 0 || (prev_bar & PCI_TYPE_64BIT)) {
+ continue;
WARNING:BRACES: braces {} are not necessary for single statement blocks
#701: FILE: windows/netuio/netuio_dev.c:200:
+ if (curr_bar == 0 || (prev_bar & PCI_TYPE_64BIT)) {
+ continue;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#702: FILE: windows/netuio/netuio_dev.c:201:
+ continue;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#702: FILE: windows/netuio/netuio_dev.c:201:
+ continue;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#703: FILE: windows/netuio/netuio_dev.c:202:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#703: FILE: windows/netuio/netuio_dev.c:202:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#705: FILE: windows/netuio/netuio_dev.c:204:
+ // Find next CmResourceTypeMemory$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#705: FILE: windows/netuio/netuio_dev.c:204:
+ // Find next CmResourceTypeMemory$
ERROR:C99_COMMENTS: do not use C99 // comments
#705: FILE: windows/netuio/netuio_dev.c:204:
+ // Find next CmResourceTypeMemory
ERROR:CODE_INDENT: code indent should use tabs where possible
#706: FILE: windows/netuio/netuio_dev.c:205:
+ do {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#706: FILE: windows/netuio/netuio_dev.c:205:
+ do {$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#706: FILE: windows/netuio/netuio_dev.c:205:
+ do {
+ descriptor = WdfCmResourceListGetDescriptor(ResourcesTranslated, next_descriptor);
ERROR:CODE_INDENT: code indent should use tabs where possible
#707: FILE: windows/netuio/netuio_dev.c:206:
+ descriptor = WdfCmResourceListGetDescriptor(ResourcesTranslated, next_descriptor);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#707: FILE: windows/netuio/netuio_dev.c:206:
+ descriptor = WdfCmResourceListGetDescriptor(ResourcesTranslated, next_descriptor);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#708: FILE: windows/netuio/netuio_dev.c:207:
+ next_descriptor++;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#708: FILE: windows/netuio/netuio_dev.c:207:
+ next_descriptor++;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#710: FILE: windows/netuio/netuio_dev.c:209:
+ if (descriptor == NULL) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#710: FILE: windows/netuio/netuio_dev.c:209:
+ if (descriptor == NULL) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#711: FILE: windows/netuio/netuio_dev.c:210:
+ status = STATUS_DEVICE_CONFIGURATION_ERROR;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#711: FILE: windows/netuio/netuio_dev.c:210:
+ status = STATUS_DEVICE_CONFIGURATION_ERROR;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#712: FILE: windows/netuio/netuio_dev.c:211:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#712: FILE: windows/netuio/netuio_dev.c:211:
+ goto end;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#713: FILE: windows/netuio/netuio_dev.c:212:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#713: FILE: windows/netuio/netuio_dev.c:212:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#714: FILE: windows/netuio/netuio_dev.c:213:
+ } while ((descriptor->Type != CmResourceTypeMemory) ||$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#714: FILE: windows/netuio/netuio_dev.c:213:
+ } while ((descriptor->Type != CmResourceTypeMemory) ||$
ERROR:CODE_INDENT: code indent should use tabs where possible
#715: FILE: windows/netuio/netuio_dev.c:214:
+ !(descriptor->Flags & CM_RESOURCE_MEMORY_BAR));$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#715: FILE: windows/netuio/netuio_dev.c:214:
+ !(descriptor->Flags & CM_RESOURCE_MEMORY_BAR));$
ERROR:CODE_INDENT: code indent should use tabs where possible
#717: FILE: windows/netuio/netuio_dev.c:216:
+ // Retrieve and map the BARs$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#717: FILE: windows/netuio/netuio_dev.c:216:
+ // Retrieve and map the BARs$
ERROR:C99_COMMENTS: do not use C99 // comments
#717: FILE: windows/netuio/netuio_dev.c:216:
+ // Retrieve and map the BARs
ERROR:CODE_INDENT: code indent should use tabs where possible
#718: FILE: windows/netuio/netuio_dev.c:217:
+ ctx->bar[bar_index].base_addr.QuadPart = descriptor->u.Memory.Start.QuadPart;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#718: FILE: windows/netuio/netuio_dev.c:217:
+ ctx->bar[bar_index].base_addr.QuadPart = descriptor->u.Memory.Start.QuadPart;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#719: FILE: windows/netuio/netuio_dev.c:218:
+ ctx->bar[bar_index].size = descriptor->u.Memory.Length;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#719: FILE: windows/netuio/netuio_dev.c:218:
+ ctx->bar[bar_index].size = descriptor->u.Memory.Length;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#720: FILE: windows/netuio/netuio_dev.c:219:
+ ctx->bar[bar_index].virt_addr = MmMapIoSpace(descriptor->u.Memory.Start,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#720: FILE: windows/netuio/netuio_dev.c:219:
+ ctx->bar[bar_index].virt_addr = MmMapIoSpace(descriptor->u.Memory.Start,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#721: FILE: windows/netuio/netuio_dev.c:220:
+ descriptor->u.Memory.Length,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#721: FILE: windows/netuio/netuio_dev.c:220:
+ descriptor->u.Memory.Length,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#722: FILE: windows/netuio/netuio_dev.c:221:
+ MmNonCached);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#722: FILE: windows/netuio/netuio_dev.c:221:
+ MmNonCached);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#723: FILE: windows/netuio/netuio_dev.c:222:
+ if (ctx->bar[bar_index].virt_addr == NULL) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#723: FILE: windows/netuio/netuio_dev.c:222:
+ if (ctx->bar[bar_index].virt_addr == NULL) {$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#723: FILE: windows/netuio/netuio_dev.c:222:
+ if (ctx->bar[bar_index].virt_addr == NULL) {
+ status = STATUS_INSUFFICIENT_RESOURCES;
ERROR:CODE_INDENT: code indent should use tabs where possible
#724: FILE: windows/netuio/netuio_dev.c:223:
+ status = STATUS_INSUFFICIENT_RESOURCES;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#724: FILE: windows/netuio/netuio_dev.c:223:
+ status = STATUS_INSUFFICIENT_RESOURCES;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#725: FILE: windows/netuio/netuio_dev.c:224:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#725: FILE: windows/netuio/netuio_dev.c:224:
+ goto end;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#726: FILE: windows/netuio/netuio_dev.c:225:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#726: FILE: windows/netuio/netuio_dev.c:225:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#728: FILE: windows/netuio/netuio_dev.c:227:
+ // Allocate an MDL for the device BAR, so we can map it to the user's process context later.$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#728: FILE: windows/netuio/netuio_dev.c:227:
+ // Allocate an MDL for the device BAR, so we can map it to the user's process context later.$
ERROR:C99_COMMENTS: do not use C99 // comments
#728: FILE: windows/netuio/netuio_dev.c:227:
+ // Allocate an MDL for the device BAR, so we can map it to the user's process context later.
ERROR:CODE_INDENT: code indent should use tabs where possible
#729: FILE: windows/netuio/netuio_dev.c:228:
+ ctx->dpdk_hw[bar_index].mdl = IoAllocateMdl(ctx->bar[bar_index].virt_addr,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#729: FILE: windows/netuio/netuio_dev.c:228:
+ ctx->dpdk_hw[bar_index].mdl = IoAllocateMdl(ctx->bar[bar_index].virt_addr,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#730: FILE: windows/netuio/netuio_dev.c:229:
+ (ULONG)ctx->bar[bar_index].size,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#730: FILE: windows/netuio/netuio_dev.c:229:
+ (ULONG)ctx->bar[bar_index].size,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#731: FILE: windows/netuio/netuio_dev.c:230:
+ FALSE,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#731: FILE: windows/netuio/netuio_dev.c:230:
+ FALSE,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#732: FILE: windows/netuio/netuio_dev.c:231:
+ FALSE,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#732: FILE: windows/netuio/netuio_dev.c:231:
+ FALSE,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#733: FILE: windows/netuio/netuio_dev.c:232:
+ NULL);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#733: FILE: windows/netuio/netuio_dev.c:232:
+ NULL);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#734: FILE: windows/netuio/netuio_dev.c:233:
+ if (!ctx->dpdk_hw[bar_index].mdl) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#734: FILE: windows/netuio/netuio_dev.c:233:
+ if (!ctx->dpdk_hw[bar_index].mdl) {$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#734: FILE: windows/netuio/netuio_dev.c:233:
+ if (!ctx->dpdk_hw[bar_index].mdl) {
+ status = STATUS_INSUFFICIENT_RESOURCES;
ERROR:CODE_INDENT: code indent should use tabs where possible
#735: FILE: windows/netuio/netuio_dev.c:234:
+ status = STATUS_INSUFFICIENT_RESOURCES;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#735: FILE: windows/netuio/netuio_dev.c:234:
+ status = STATUS_INSUFFICIENT_RESOURCES;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#736: FILE: windows/netuio/netuio_dev.c:235:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#736: FILE: windows/netuio/netuio_dev.c:235:
+ goto end;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#737: FILE: windows/netuio/netuio_dev.c:236:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#737: FILE: windows/netuio/netuio_dev.c:236:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#739: FILE: windows/netuio/netuio_dev.c:238:
+ ctx->dpdk_hw[bar_index].mem.size = ctx->bar[bar_index].size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#739: FILE: windows/netuio/netuio_dev.c:238:
+ ctx->dpdk_hw[bar_index].mem.size = ctx->bar[bar_index].size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#740: FILE: windows/netuio/netuio_dev.c:239:
+ } // for bar_index$
ERROR:C99_COMMENTS: do not use C99 // comments
#740: FILE: windows/netuio/netuio_dev.c:239:
+ } // for bar_index
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#742: FILE: windows/netuio/netuio_dev.c:241:
+ status = STATUS_SUCCESS;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#745: FILE: windows/netuio/netuio_dev.c:244:
+ if (status != STATUS_SUCCESS) {$
WARNING:BRACES: braces {} are not necessary for single statement blocks
#745: FILE: windows/netuio/netuio_dev.c:244:
+ if (status != STATUS_SUCCESS) {
+ netuio_free_hw_resources(Device);
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#746: FILE: windows/netuio/netuio_dev.c:245:
+ netuio_free_hw_resources(Device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#746: FILE: windows/netuio/netuio_dev.c:245:
+ netuio_free_hw_resources(Device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#747: FILE: windows/netuio/netuio_dev.c:246:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#749: FILE: windows/netuio/netuio_dev.c:248:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#756: FILE: windows/netuio/netuio_dev.c:255:
+ PNETUIO_CONTEXT_DATA ctx;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#757: FILE: windows/netuio/netuio_dev.c:256:
+ ctx = netuio_get_context_data(Device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#759: FILE: windows/netuio/netuio_dev.c:258:
+ for (UINT8 bar_index = 0; bar_index < PCI_MAX_BAR; bar_index++) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#761: FILE: windows/netuio/netuio_dev.c:260:
+ // Free the allocated MDLs$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#761: FILE: windows/netuio/netuio_dev.c:260:
+ // Free the allocated MDLs$
ERROR:C99_COMMENTS: do not use C99 // comments
#761: FILE: windows/netuio/netuio_dev.c:260:
+ // Free the allocated MDLs
ERROR:CODE_INDENT: code indent should use tabs where possible
#762: FILE: windows/netuio/netuio_dev.c:261:
+ if (ctx->dpdk_hw[bar_index].mdl) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#762: FILE: windows/netuio/netuio_dev.c:261:
+ if (ctx->dpdk_hw[bar_index].mdl) {$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#762: FILE: windows/netuio/netuio_dev.c:261:
+ if (ctx->dpdk_hw[bar_index].mdl) {
+ IoFreeMdl(ctx->dpdk_hw[bar_index].mdl);
WARNING:BRACES: braces {} are not necessary for single statement blocks
#762: FILE: windows/netuio/netuio_dev.c:261:
+ if (ctx->dpdk_hw[bar_index].mdl) {
+ IoFreeMdl(ctx->dpdk_hw[bar_index].mdl);
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#763: FILE: windows/netuio/netuio_dev.c:262:
+ IoFreeMdl(ctx->dpdk_hw[bar_index].mdl);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#763: FILE: windows/netuio/netuio_dev.c:262:
+ IoFreeMdl(ctx->dpdk_hw[bar_index].mdl);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#764: FILE: windows/netuio/netuio_dev.c:263:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#764: FILE: windows/netuio/netuio_dev.c:263:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#766: FILE: windows/netuio/netuio_dev.c:265:
+ // Unmap all the BAR regions previously mapped$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#766: FILE: windows/netuio/netuio_dev.c:265:
+ // Unmap all the BAR regions previously mapped$
ERROR:C99_COMMENTS: do not use C99 // comments
#766: FILE: windows/netuio/netuio_dev.c:265:
+ // Unmap all the BAR regions previously mapped
ERROR:CODE_INDENT: code indent should use tabs where possible
#767: FILE: windows/netuio/netuio_dev.c:266:
+ if (ctx->bar[bar_index].virt_addr) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#767: FILE: windows/netuio/netuio_dev.c:266:
+ if (ctx->bar[bar_index].virt_addr) {$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#767: FILE: windows/netuio/netuio_dev.c:266:
+ if (ctx->bar[bar_index].virt_addr) {
+ MmUnmapIoSpace(ctx->bar[bar_index].virt_addr, ctx->bar[bar_index].size);
WARNING:BRACES: braces {} are not necessary for single statement blocks
#767: FILE: windows/netuio/netuio_dev.c:266:
+ if (ctx->bar[bar_index].virt_addr) {
+ MmUnmapIoSpace(ctx->bar[bar_index].virt_addr, ctx->bar[bar_index].size);
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#768: FILE: windows/netuio/netuio_dev.c:267:
+ MmUnmapIoSpace(ctx->bar[bar_index].virt_addr, ctx->bar[bar_index].size);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#768: FILE: windows/netuio/netuio_dev.c:267:
+ MmUnmapIoSpace(ctx->bar[bar_index].virt_addr, ctx->bar[bar_index].size);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#769: FILE: windows/netuio/netuio_dev.c:268:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#769: FILE: windows/netuio/netuio_dev.c:268:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#770: FILE: windows/netuio/netuio_dev.c:269:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#772: FILE: windows/netuio/netuio_dev.c:271:
+ RtlZeroMemory(ctx->dpdk_hw, sizeof(ctx->dpdk_hw));$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#773: FILE: windows/netuio/netuio_dev.c:272:
+ RtlZeroMemory(ctx->bar, sizeof(ctx->bar));$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#791: FILE: windows/netuio/netuio_dev.h:11:
+ PHYSICAL_ADDRESS base_addr;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#792: FILE: windows/netuio/netuio_dev.h:12:
+ PVOID virt_addr;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#793: FILE: windows/netuio/netuio_dev.h:13:
+ UINT64 size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#797: FILE: windows/netuio/netuio_dev.h:17:
+ PMDL mdl; /**< MDL describing the memory region */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#798: FILE: windows/netuio/netuio_dev.h:18:
+ struct mem_region mem; /**< Memory region details */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#802: FILE: windows/netuio/netuio_dev.h:22:
+ ULONG bus_num;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#803: FILE: windows/netuio/netuio_dev.h:23:
+ USHORT dev_num;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#804: FILE: windows/netuio/netuio_dev.h:24:
+ USHORT func_num;$
ERROR:OPEN_BRACE: open brace '{' following struct go on the same line
#811: FILE: windows/netuio/netuio_dev.h:31:
+typedef struct _NETUIO_CONTEXT_DATA
+{
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#812: FILE: windows/netuio/netuio_dev.h:32:
+ WDFDEVICE wdf_device; /**< WDF device handle to the FDO */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#813: FILE: windows/netuio/netuio_dev.h:33:
+ BUS_INTERFACE_STANDARD bus_interface; /**< Bus interface for config space access */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#814: FILE: windows/netuio/netuio_dev.h:34:
+ struct pci_bar bar[PCI_MAX_BAR]; /**< device BARs */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#815: FILE: windows/netuio/netuio_dev.h:35:
+ struct dev_addr addr; /**< B:D:F details of device */$
WARNING:LONG_LINE_COMMENT: line over 100 characters
#816: FILE: windows/netuio/netuio_dev.h:36:
+ struct mem_map_region dpdk_hw[PCI_MAX_BAR]; /**< mapped region for the device's register space */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#816: FILE: windows/netuio/netuio_dev.h:36:
+ struct mem_map_region dpdk_hw[PCI_MAX_BAR]; /**< mapped region for the device's register space */$
ERROR:OPEN_BRACE: open brace '{' following struct go on the same line
#828: FILE: windows/netuio/netuio_dev.h:48:
+typedef struct
+{
WARNING:LONG_LINE_COMMENT: line over 100 characters
#829: FILE: windows/netuio/netuio_dev.h:49:
+ BOOLEAN bMapped; /**< value is set to TRUE if the User-mode mapping was done for this file object. */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#829: FILE: windows/netuio/netuio_dev.h:49:
+ BOOLEAN bMapped; /**< value is set to TRUE if the User-mode mapping was done for this file object. */$
ERROR:SPACING: space prohibited after that '*' (ctx:WxW)
#830: FILE: windows/netuio/netuio_dev.h:50:
+} NETUIO_FILE_CONTEXT_DATA, * PNETUIO_FILE_CONTEXT_DATA;
^
WARNING:TYPO_SPELLING: 'Inout' may be misspelled - perhaps 'Input'?
#842: FILE: windows/netuio/netuio_dev.h:62:
+NTSTATUS netuio_create_device(_Inout_ PWDFDEVICE_INIT DeviceInit);
WARNING:LONG_LINE: line over 100 characters
#843: FILE: windows/netuio/netuio_dev.h:63:
+NTSTATUS netuio_map_hw_resources(_In_ WDFDEVICE Device, _In_ WDFCMRESLIST Resources, _In_ WDFCMRESLIST ResourcesTranslated);
ERROR:C99_COMMENTS: do not use C99 // comments
#846: FILE: windows/netuio/netuio_dev.h:66:
+#endif // NETUIO_DEV_H
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#860: FILE: windows/netuio/netuio_drv.c:8:
+#pragma alloc_text (INIT, DriverEntry)
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#861: FILE: windows/netuio/netuio_drv.c:9:
+#pragma alloc_text (PAGE, netuio_evt_device_add)
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#865: FILE: windows/netuio/netuio_drv.c:13:
+/*
+Routine Description:
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#877: FILE: windows/netuio/netuio_drv.c:25:
+ WDF_DRIVER_CONFIG config;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#878: FILE: windows/netuio/netuio_drv.c:26:
+ NTSTATUS status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#879: FILE: windows/netuio/netuio_drv.c:27:
+ WDF_OBJECT_ATTRIBUTES attributes;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#881: FILE: windows/netuio/netuio_drv.c:29:
+ WDF_OBJECT_ATTRIBUTES_INIT(&attributes);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#882: FILE: windows/netuio/netuio_drv.c:30:
+ WDF_DRIVER_CONFIG_INIT(&config, netuio_evt_device_add);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#884: FILE: windows/netuio/netuio_drv.c:32:
+ status = WdfDriverCreate(DriverObject, RegistryPath,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#885: FILE: windows/netuio/netuio_drv.c:33:
+ &attributes, &config,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#885: FILE: windows/netuio/netuio_drv.c:33:
+ &attributes, &config,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#886: FILE: windows/netuio/netuio_drv.c:34:
+ WDF_NO_HANDLE);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#886: FILE: windows/netuio/netuio_drv.c:34:
+ WDF_NO_HANDLE);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#888: FILE: windows/netuio/netuio_drv.c:36:
+ if (!NT_SUCCESS(status)) {$
WARNING:BRACES: braces {} are not necessary for single statement blocks
#888: FILE: windows/netuio/netuio_drv.c:36:
+ if (!NT_SUCCESS(status)) {
+ return status;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#889: FILE: windows/netuio/netuio_drv.c:37:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#889: FILE: windows/netuio/netuio_drv.c:37:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#890: FILE: windows/netuio/netuio_drv.c:38:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#892: FILE: windows/netuio/netuio_drv.c:40:
+ return status;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#896: FILE: windows/netuio/netuio_drv.c:44:
+/*
+Routine Description:
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#908: FILE: windows/netuio/netuio_drv.c:56:
+ UNREFERENCED_PARAMETER(Driver);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#909: FILE: windows/netuio/netuio_drv.c:57:
+ return netuio_create_device(DeviceInit);$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#913: FILE: windows/netuio/netuio_drv.c:61:
+/*
+Routine Description :
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#919: FILE: windows/netuio/netuio_drv.c:67:
+-*/
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#924: FILE: windows/netuio/netuio_drv.c:72:
+ NTSTATUS status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#926: FILE: windows/netuio/netuio_drv.c:74:
+ status = netuio_map_hw_resources(Device, Resources, ResourcesTranslated);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#927: FILE: windows/netuio/netuio_drv.c:75:
+ if (!NT_SUCCESS(status)) {$
WARNING:BRACES: braces {} are not necessary for single statement blocks
#927: FILE: windows/netuio/netuio_drv.c:75:
+ if (!NT_SUCCESS(status)) {
+ return status;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#928: FILE: windows/netuio/netuio_drv.c:76:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#928: FILE: windows/netuio/netuio_drv.c:76:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#929: FILE: windows/netuio/netuio_drv.c:77:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#931: FILE: windows/netuio/netuio_drv.c:79:
+ PNETUIO_CONTEXT_DATA ctx = netuio_get_context_data(Device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#932: FILE: windows/netuio/netuio_drv.c:80:
+ DbgPrintEx(DPFLTR_IHVNETWORK_ID, DPFLTR_NETUIO_INFO_LEVEL, "netUIO Driver loaded...on device (B:D:F) %04d:%02d:%02d
",$
ERROR:CODE_INDENT: code indent should use tabs where possible
#933: FILE: windows/netuio/netuio_drv.c:81:
+ ctx->addr.bus_num, ctx->addr.dev_num, ctx->addr.func_num);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#933: FILE: windows/netuio/netuio_drv.c:81:
+ ctx->addr.bus_num, ctx->addr.dev_num, ctx->addr.func_num);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#935: FILE: windows/netuio/netuio_drv.c:83:
+ return status;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#939: FILE: windows/netuio/netuio_drv.c:87:
+/*
+Routine Description :
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#944: FILE: windows/netuio/netuio_drv.c:92:
+-*/
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#949: FILE: windows/netuio/netuio_drv.c:97:
+ UNREFERENCED_PARAMETER(ResourcesTranslated);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#951: FILE: windows/netuio/netuio_drv.c:99:
+ netuio_free_hw_resources(Device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#953: FILE: windows/netuio/netuio_drv.c:101:
+ return STATUS_SUCCESS;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#957: FILE: windows/netuio/netuio_drv.c:105:
+/*
+Routine Description :
WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#964: FILE: windows/netuio/netuio_drv.c:112:
+-*/
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#969: FILE: windows/netuio/netuio_drv.c:117:
+ PNETUIO_FILE_CONTEXT_DATA file_ctx;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#971: FILE: windows/netuio/netuio_drv.c:119:
+ file_ctx = netuio_get_file_object_context_data(FileObject);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#973: FILE: windows/netuio/netuio_drv.c:121:
+ if (file_ctx->bMapped == TRUE)$
ERROR:OPEN_BRACE: that open brace { should be on the previous line
#973: FILE: windows/netuio/netuio_drv.c:121:
+ if (file_ctx->bMapped == TRUE)
+ {
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#974: FILE: windows/netuio/netuio_drv.c:122:
+ {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#975: FILE: windows/netuio/netuio_drv.c:123:
+ WDFDEVICE device;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#975: FILE: windows/netuio/netuio_drv.c:123:
+ WDFDEVICE device;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#976: FILE: windows/netuio/netuio_drv.c:124:
+ PNETUIO_CONTEXT_DATA ctx;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#976: FILE: windows/netuio/netuio_drv.c:124:
+ PNETUIO_CONTEXT_DATA ctx;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#978: FILE: windows/netuio/netuio_drv.c:126:
+ device = WdfFileObjectGetDevice(FileObject);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#978: FILE: windows/netuio/netuio_drv.c:126:
+ device = WdfFileObjectGetDevice(FileObject);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#979: FILE: windows/netuio/netuio_drv.c:127:
+ ctx = netuio_get_context_data(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#979: FILE: windows/netuio/netuio_drv.c:127:
+ ctx = netuio_get_context_data(device);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#980: FILE: windows/netuio/netuio_drv.c:128:
+ netuio_unmap_address_from_user_process(ctx);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#980: FILE: windows/netuio/netuio_drv.c:128:
+ netuio_unmap_address_from_user_process(ctx);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#981: FILE: windows/netuio/netuio_drv.c:129:
+ file_ctx->bMapped = FALSE;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#981: FILE: windows/netuio/netuio_drv.c:129:
+ file_ctx->bMapped = FALSE;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#982: FILE: windows/netuio/netuio_drv.c:130:
+ }$
ERROR:C99_COMMENTS: do not use C99 // comments
#1019: FILE: windows/netuio/netuio_drv.h:30:
+#endif // NETUIO_DRV_H
WARNING:LONG_LINE: line over 100 characters
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
ERROR:C99_COMMENTS: do not use C99 // comments
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
ERROR:SPACING: space required after that ',' (ctx:VxV)
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
^
ERROR:SPACING: space required after that ',' (ctx:VxV)
#1046: FILE: windows/netuio/netuio_interface.h:21:
+DEFINE_GUID (GUID_DEVINTERFACE_netUIO, 0x08336f60,0x0679,0x4c6c,0x85,0xd2,0xae,0x7c,0xed,0x65,0xff,0xf7); // {08336f60-0679-4c6c-85d2-ae7ced65fff7}
^
WARNING:LONG_LINE: line over 100 characters
#1059: FILE: windows/netuio/netuio_interface.h:34:
+#define IOCTL_NETUIO_MAP_HW_INTO_USERSPACE CTL_CODE(FILE_DEVICE_NETWORK, 51, METHOD_BUFFERED, FILE_READ_ACCESS | FILE_WRITE_ACCESS)
WARNING:LONG_LINE: line over 100 characters
#1076: FILE: windows/netuio/netuio_interface.h:51:
+#define IOCTL_NETUIO_PCI_CONFIG_IO CTL_CODE(FILE_DEVICE_NETWORK, 52, METHOD_BUFFERED, FILE_READ_ACCESS | FILE_WRITE_ACCESS)
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1079: FILE: windows/netuio/netuio_interface.h:54:
+ UINT64 size; /**< memory region size */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1080: FILE: windows/netuio/netuio_interface.h:55:
+ LARGE_INTEGER phys_addr; /**< physical address of the memory region */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1081: FILE: windows/netuio/netuio_interface.h:56:
+ PVOID virt_addr; /**< virtual address of the memory region */$
WARNING:LONG_LINE_COMMENT: line over 100 characters
#1082: FILE: windows/netuio/netuio_interface.h:57:
+ PVOID user_mapped_virt_addr; /**< virtual address of the region mapped into user process context */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1082: FILE: windows/netuio/netuio_interface.h:57:
+ PVOID user_mapped_virt_addr; /**< virtual address of the region mapped into user process context */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1086: FILE: windows/netuio/netuio_interface.h:61:
+ PCI_IO_READ = 0,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1087: FILE: windows/netuio/netuio_interface.h:62:
+ PCI_IO_WRITE = 1$
ERROR:OPEN_BRACE: open brace '{' following struct go on the same line
#1093: FILE: windows/netuio/netuio_interface.h:68:
+struct device_info
+{
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1094: FILE: windows/netuio/netuio_interface.h:69:
+ struct mem_region hw[PCI_MAX_BAR];$
ERROR:OPEN_BRACE: open brace '{' following struct go on the same line
#1098: FILE: windows/netuio/netuio_interface.h:73:
+struct dpdk_pci_config_io
+{
WARNING:LONG_LINE_COMMENT: line over 100 characters
#1099: FILE: windows/netuio/netuio_interface.h:74:
+ UINT32 offset; /**< offset into the device config space where the reading/writing starts */
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1099: FILE: windows/netuio/netuio_interface.h:74:
+ UINT32 offset; /**< offset into the device config space where the reading/writing starts */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1100: FILE: windows/netuio/netuio_interface.h:75:
+ UINT8 op; /**< operation type: read or write */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1101: FILE: windows/netuio/netuio_interface.h:76:
+ UINT32 access_size; /**< 1, 2, 4, or 8 bytes */$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1103: FILE: windows/netuio/netuio_interface.h:78:
+ union dpdk_pci_config_io_data {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1104: FILE: windows/netuio/netuio_interface.h:79:
+ UINT8^I^I^Iu8;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1104: FILE: windows/netuio/netuio_interface.h:79:
+ UINT8^I^I^Iu8;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1105: FILE: windows/netuio/netuio_interface.h:80:
+ UINT16^I^I^Iu16;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1105: FILE: windows/netuio/netuio_interface.h:80:
+ UINT16^I^I^Iu16;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1106: FILE: windows/netuio/netuio_interface.h:81:
+ UINT32^I^I^Iu32;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1106: FILE: windows/netuio/netuio_interface.h:81:
+ UINT32^I^I^Iu32;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1107: FILE: windows/netuio/netuio_interface.h:82:
+ UINT64^I^I^Iu64;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1107: FILE: windows/netuio/netuio_interface.h:82:
+ UINT64^I^I^Iu64;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1108: FILE: windows/netuio/netuio_interface.h:83:
+ } data; /**< Data to be written, in case of write operations */$
ERROR:C99_COMMENTS: do not use C99 // comments
#1113: FILE: windows/netuio/netuio_interface.h:88:
+#endif // NETUIO_INTERFACE_H
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1127: FILE: windows/netuio/netuio_queue.c:8:
+#pragma alloc_text (PAGE, netuio_queue_initialize)
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1134: FILE: windows/netuio/netuio_queue.c:15:
+ WDFFILEOBJECT file_object;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1136: FILE: windows/netuio/netuio_queue.c:17:
+ file_object = WdfRequestGetFileObject(Request);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1137: FILE: windows/netuio/netuio_queue.c:18:
+ if (file_object == NULL)$
ERROR:OPEN_BRACE: that open brace { should be on the previous line
#1137: FILE: windows/netuio/netuio_queue.c:18:
+ if (file_object == NULL)
+ {
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1138: FILE: windows/netuio/netuio_queue.c:19:
+ {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1139: FILE: windows/netuio/netuio_queue.c:20:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1139: FILE: windows/netuio/netuio_queue.c:20:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1140: FILE: windows/netuio/netuio_queue.c:21:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1142: FILE: windows/netuio/netuio_queue.c:23:
+ PNETUIO_FILE_CONTEXT_DATA file_ctx;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1144: FILE: windows/netuio/netuio_queue.c:25:
+ file_ctx = netuio_get_file_object_context_data(file_object);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1145: FILE: windows/netuio/netuio_queue.c:26:
+ if (file_ctx->bMapped)$
ERROR:OPEN_BRACE: that open brace { should be on the previous line
#1145: FILE: windows/netuio/netuio_queue.c:26:
+ if (file_ctx->bMapped)
+ {
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1146: FILE: windows/netuio/netuio_queue.c:27:
+ {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1147: FILE: windows/netuio/netuio_queue.c:28:
+ return TRUE;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1147: FILE: windows/netuio/netuio_queue.c:28:
+ return TRUE;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1148: FILE: windows/netuio/netuio_queue.c:29:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1151: FILE: windows/netuio/netuio_queue.c:32:
+ return FALSE;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1158: FILE: windows/netuio/netuio_queue.c:39:
+ WDFFILEOBJECT file_object;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1160: FILE: windows/netuio/netuio_queue.c:41:
+ file_object = WdfRequestGetFileObject(Request);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1161: FILE: windows/netuio/netuio_queue.c:42:
+ if (file_object == NULL)$
ERROR:OPEN_BRACE: that open brace { should be on the previous line
#1161: FILE: windows/netuio/netuio_queue.c:42:
+ if (file_object == NULL)
+ {
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1162: FILE: windows/netuio/netuio_queue.c:43:
+ {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1163: FILE: windows/netuio/netuio_queue.c:44:
+ return;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1163: FILE: windows/netuio/netuio_queue.c:44:
+ return;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1164: FILE: windows/netuio/netuio_queue.c:45:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1166: FILE: windows/netuio/netuio_queue.c:47:
+ PNETUIO_FILE_CONTEXT_DATA file_ctx;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1168: FILE: windows/netuio/netuio_queue.c:49:
+ file_ctx = netuio_get_file_object_context_data(file_object);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1170: FILE: windows/netuio/netuio_queue.c:51:
+ file_ctx->bMapped = TRUE;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1175: FILE: windows/netuio/netuio_queue.c:56:
+ _In_ PVOID outputBuf, _In_ size_t outputBufSize)$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1175: FILE: windows/netuio/netuio_queue.c:56:
+ _In_ PVOID outputBuf, _In_ size_t outputBufSize)$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1177: FILE: windows/netuio/netuio_queue.c:58:
+ ASSERT(outputBufSize == sizeof(struct device_info));$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1179: FILE: windows/netuio/netuio_queue.c:60:
+ struct device_info *dpdk_pvt_info = (struct device_info *)outputBuf;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1180: FILE: windows/netuio/netuio_queue.c:61:
+ RtlZeroMemory(dpdk_pvt_info, outputBufSize);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1182: FILE: windows/netuio/netuio_queue.c:63:
+ for (ULONG idx = 0; idx < PCI_MAX_BAR; idx++) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1183: FILE: windows/netuio/netuio_queue.c:64:
+ dpdk_pvt_info->hw[idx].phys_addr.QuadPart = ctx->bar[idx].base_addr.QuadPart;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1183: FILE: windows/netuio/netuio_queue.c:64:
+ dpdk_pvt_info->hw[idx].phys_addr.QuadPart = ctx->bar[idx].base_addr.QuadPart;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1184: FILE: windows/netuio/netuio_queue.c:65:
+ dpdk_pvt_info->hw[idx].user_mapped_virt_addr = ctx->dpdk_hw[idx].mem.user_mapped_virt_addr;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1184: FILE: windows/netuio/netuio_queue.c:65:
+ dpdk_pvt_info->hw[idx].user_mapped_virt_addr = ctx->dpdk_hw[idx].mem.user_mapped_virt_addr;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1185: FILE: windows/netuio/netuio_queue.c:66:
+ dpdk_pvt_info->hw[idx].size = ctx->bar[idx].size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1185: FILE: windows/netuio/netuio_queue.c:66:
+ dpdk_pvt_info->hw[idx].size = ctx->bar[idx].size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1186: FILE: windows/netuio/netuio_queue.c:67:
+ }$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#1190: FILE: windows/netuio/netuio_queue.c:71:
+/*
+Routine Description:
WARNING:TYPO_SPELLING: 'process'' may be misspelled - perhaps 'process's'?
#1191: FILE: windows/netuio/netuio_queue.c:72:
+ Maps address ranges into the usermode process's address space. The following
ERROR:CODE_INDENT: code indent should use tabs where possible
#1194: FILE: windows/netuio/netuio_queue.c:75:
+ * Any PCI BARs that our device was assigned$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1195: FILE: windows/netuio/netuio_queue.c:76:
+ * The scratch buffer of contiguous pages$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1203: FILE: windows/netuio/netuio_queue.c:84:
+ NTSTATUS status = STATUS_SUCCESS;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1205: FILE: windows/netuio/netuio_queue.c:86:
+ if (netuio_get_usermode_mapping_flag(Request))$
ERROR:OPEN_BRACE: that open brace { should be on the previous line
#1205: FILE: windows/netuio/netuio_queue.c:86:
+ if (netuio_get_usermode_mapping_flag(Request))
+ {
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1206: FILE: windows/netuio/netuio_queue.c:87:
+ {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1207: FILE: windows/netuio/netuio_queue.c:88:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1207: FILE: windows/netuio/netuio_queue.c:88:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1208: FILE: windows/netuio/netuio_queue.c:89:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1210: FILE: windows/netuio/netuio_queue.c:91:
+ // Map any device BAR(s) to the user's process context$
ERROR:C99_COMMENTS: do not use C99 // comments
#1210: FILE: windows/netuio/netuio_queue.c:91:
+ // Map any device BAR(s) to the user's process context
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1211: FILE: windows/netuio/netuio_queue.c:92:
+ for (INT idx = 0; idx < PCI_MAX_BAR; idx++) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1212: FILE: windows/netuio/netuio_queue.c:93:
+ if (ctx->dpdk_hw[idx].mdl == NULL) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1212: FILE: windows/netuio/netuio_queue.c:93:
+ if (ctx->dpdk_hw[idx].mdl == NULL) {$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#1212: FILE: windows/netuio/netuio_queue.c:93:
+ if (ctx->dpdk_hw[idx].mdl == NULL) {
+ continue;
WARNING:BRACES: braces {} are not necessary for single statement blocks
#1212: FILE: windows/netuio/netuio_queue.c:93:
+ if (ctx->dpdk_hw[idx].mdl == NULL) {
+ continue;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#1213: FILE: windows/netuio/netuio_queue.c:94:
+ continue;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1213: FILE: windows/netuio/netuio_queue.c:94:
+ continue;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1214: FILE: windows/netuio/netuio_queue.c:95:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1214: FILE: windows/netuio/netuio_queue.c:95:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1216: FILE: windows/netuio/netuio_queue.c:97:
+ MmBuildMdlForNonPagedPool(ctx->dpdk_hw[idx].mdl);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1216: FILE: windows/netuio/netuio_queue.c:97:
+ MmBuildMdlForNonPagedPool(ctx->dpdk_hw[idx].mdl);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1217: FILE: windows/netuio/netuio_queue.c:98:
+ __try {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1217: FILE: windows/netuio/netuio_queue.c:98:
+ __try {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1218: FILE: windows/netuio/netuio_queue.c:99:
+ ctx->dpdk_hw[idx].mem.user_mapped_virt_addr =$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1218: FILE: windows/netuio/netuio_queue.c:99:
+ ctx->dpdk_hw[idx].mem.user_mapped_virt_addr =$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1219: FILE: windows/netuio/netuio_queue.c:100:
+ MmMapLockedPagesSpecifyCache(ctx->dpdk_hw[idx].mdl, UserMode,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1219: FILE: windows/netuio/netuio_queue.c:100:
+ MmMapLockedPagesSpecifyCache(ctx->dpdk_hw[idx].mdl, UserMode,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1220: FILE: windows/netuio/netuio_queue.c:101:
+ MmCached, NULL, FALSE, NormalPagePriority);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1220: FILE: windows/netuio/netuio_queue.c:101:
+ MmCached, NULL, FALSE, NormalPagePriority);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1222: FILE: windows/netuio/netuio_queue.c:103:
+ if (ctx->dpdk_hw[idx].mem.user_mapped_virt_addr == NULL) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1222: FILE: windows/netuio/netuio_queue.c:103:
+ if (ctx->dpdk_hw[idx].mem.user_mapped_virt_addr == NULL) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1223: FILE: windows/netuio/netuio_queue.c:104:
+ status = STATUS_INSUFFICIENT_RESOURCES;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1223: FILE: windows/netuio/netuio_queue.c:104:
+ status = STATUS_INSUFFICIENT_RESOURCES;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1224: FILE: windows/netuio/netuio_queue.c:105:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1224: FILE: windows/netuio/netuio_queue.c:105:
+ goto end;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1225: FILE: windows/netuio/netuio_queue.c:106:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1225: FILE: windows/netuio/netuio_queue.c:106:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1226: FILE: windows/netuio/netuio_queue.c:107:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1226: FILE: windows/netuio/netuio_queue.c:107:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1227: FILE: windows/netuio/netuio_queue.c:108:
+ __except (EXCEPTION_EXECUTE_HANDLER) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1227: FILE: windows/netuio/netuio_queue.c:108:
+ __except (EXCEPTION_EXECUTE_HANDLER) {$
WARNING:SPACING: space prohibited between function name and open parenthesis '('
#1227: FILE: windows/netuio/netuio_queue.c:108:
+ __except (EXCEPTION_EXECUTE_HANDLER) {
ERROR:CODE_INDENT: code indent should use tabs where possible
#1228: FILE: windows/netuio/netuio_queue.c:109:
+ status = GetExceptionCode();$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1228: FILE: windows/netuio/netuio_queue.c:109:
+ status = GetExceptionCode();$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1229: FILE: windows/netuio/netuio_queue.c:110:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1229: FILE: windows/netuio/netuio_queue.c:110:
+ goto end;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1230: FILE: windows/netuio/netuio_queue.c:111:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1230: FILE: windows/netuio/netuio_queue.c:111:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1231: FILE: windows/netuio/netuio_queue.c:112:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1234: FILE: windows/netuio/netuio_queue.c:115:
+ if (status != STATUS_SUCCESS) {$
WARNING:BRACES: braces {} are not necessary for single statement blocks
#1234: FILE: windows/netuio/netuio_queue.c:115:
+ if (status != STATUS_SUCCESS) {
+ netuio_unmap_address_from_user_process(ctx);
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#1235: FILE: windows/netuio/netuio_queue.c:116:
+ netuio_unmap_address_from_user_process(ctx);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1235: FILE: windows/netuio/netuio_queue.c:116:
+ netuio_unmap_address_from_user_process(ctx);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1236: FILE: windows/netuio/netuio_queue.c:117:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1238: FILE: windows/netuio/netuio_queue.c:119:
+ return status;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#1242: FILE: windows/netuio/netuio_queue.c:123:
+/*
+Routine Description:
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1254: FILE: windows/netuio/netuio_queue.c:135:
+ for (INT idx = 0; idx < PCI_MAX_BAR; idx++) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1255: FILE: windows/netuio/netuio_queue.c:136:
+ if (ctx->dpdk_hw[idx].mem.user_mapped_virt_addr != NULL) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1255: FILE: windows/netuio/netuio_queue.c:136:
+ if (ctx->dpdk_hw[idx].mem.user_mapped_virt_addr != NULL) {$
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#1255: FILE: windows/netuio/netuio_queue.c:136:
+ if (ctx->dpdk_hw[idx].mem.user_mapped_virt_addr != NULL) {
+ MmUnmapLockedPages(ctx->dpdk_hw[idx].mem.user_mapped_virt_addr,
ERROR:CODE_INDENT: code indent should use tabs where possible
#1256: FILE: windows/netuio/netuio_queue.c:137:
+ MmUnmapLockedPages(ctx->dpdk_hw[idx].mem.user_mapped_virt_addr,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1256: FILE: windows/netuio/netuio_queue.c:137:
+ MmUnmapLockedPages(ctx->dpdk_hw[idx].mem.user_mapped_virt_addr,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1257: FILE: windows/netuio/netuio_queue.c:138:
+ ctx->dpdk_hw[idx].mdl);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1257: FILE: windows/netuio/netuio_queue.c:138:
+ ctx->dpdk_hw[idx].mdl);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1259: FILE: windows/netuio/netuio_queue.c:140:
+ ctx->dpdk_hw[idx].mem.user_mapped_virt_addr = NULL;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1259: FILE: windows/netuio/netuio_queue.c:140:
+ ctx->dpdk_hw[idx].mem.user_mapped_virt_addr = NULL;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1260: FILE: windows/netuio/netuio_queue.c:141:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1260: FILE: windows/netuio/netuio_queue.c:141:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1261: FILE: windows/netuio/netuio_queue.c:142:
+ }$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#1265: FILE: windows/netuio/netuio_queue.c:146:
+/*
+Routine Description:
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1277: FILE: windows/netuio/netuio_queue.c:158:
+ WDFQUEUE queue;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1278: FILE: windows/netuio/netuio_queue.c:159:
+ NTSTATUS status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1279: FILE: windows/netuio/netuio_queue.c:160:
+ WDF_IO_QUEUE_CONFIG queueConfig;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1281: FILE: windows/netuio/netuio_queue.c:162:
+ PAGED_CODE();$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1283: FILE: windows/netuio/netuio_queue.c:164:
+ // Configure a default queue so that requests that are not$
ERROR:C99_COMMENTS: do not use C99 // comments
#1283: FILE: windows/netuio/netuio_queue.c:164:
+ // Configure a default queue so that requests that are not
WARNING:TYPO_SPELLING: 'fowarded' may be misspelled - perhaps 'forwarded'?
#1284: FILE: windows/netuio/netuio_queue.c:165:
+ // configure-fowarded using WdfDeviceConfigureRequestDispatching to goto
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1284: FILE: windows/netuio/netuio_queue.c:165:
+ // configure-fowarded using WdfDeviceConfigureRequestDispatching to goto$
ERROR:C99_COMMENTS: do not use C99 // comments
#1284: FILE: windows/netuio/netuio_queue.c:165:
+ // configure-fowarded using WdfDeviceConfigureRequestDispatching to goto
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1285: FILE: windows/netuio/netuio_queue.c:166:
+ // other queues get dispatched here.$
ERROR:C99_COMMENTS: do not use C99 // comments
#1285: FILE: windows/netuio/netuio_queue.c:166:
+ // other queues get dispatched here.
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1286: FILE: windows/netuio/netuio_queue.c:167:
+ WDF_IO_QUEUE_CONFIG_INIT_DEFAULT_QUEUE(&queueConfig, WdfIoQueueDispatchParallel);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1288: FILE: windows/netuio/netuio_queue.c:169:
+ queueConfig.EvtIoDeviceControl = netuio_evt_IO_device_control;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1290: FILE: windows/netuio/netuio_queue.c:171:
+ status = WdfIoQueueCreate(Device,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1291: FILE: windows/netuio/netuio_queue.c:172:
+ &queueConfig,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1291: FILE: windows/netuio/netuio_queue.c:172:
+ &queueConfig,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1292: FILE: windows/netuio/netuio_queue.c:173:
+ WDF_NO_OBJECT_ATTRIBUTES,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1292: FILE: windows/netuio/netuio_queue.c:173:
+ WDF_NO_OBJECT_ATTRIBUTES,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1293: FILE: windows/netuio/netuio_queue.c:174:
+ &queue);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1293: FILE: windows/netuio/netuio_queue.c:174:
+ &queue);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1295: FILE: windows/netuio/netuio_queue.c:176:
+ if( !NT_SUCCESS(status) ) {$
ERROR:SPACING: space prohibited after that open parenthesis '('
#1295: FILE: windows/netuio/netuio_queue.c:176:
+ if( !NT_SUCCESS(status) ) {
ERROR:SPACING: space prohibited before that close parenthesis ')'
#1295: FILE: windows/netuio/netuio_queue.c:176:
+ if( !NT_SUCCESS(status) ) {
ERROR:SPACING: space required before the open parenthesis '('
#1295: FILE: windows/netuio/netuio_queue.c:176:
+ if( !NT_SUCCESS(status) ) {
WARNING:BRACES: braces {} are not necessary for single statement blocks
#1295: FILE: windows/netuio/netuio_queue.c:176:
+ if( !NT_SUCCESS(status) ) {
+ return status;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#1296: FILE: windows/netuio/netuio_queue.c:177:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1296: FILE: windows/netuio/netuio_queue.c:177:
+ return status;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1297: FILE: windows/netuio/netuio_queue.c:178:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1299: FILE: windows/netuio/netuio_queue.c:180:
+ return status;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#1303: FILE: windows/netuio/netuio_queue.c:184:
+/*
+Routine Description:
ERROR:CODE_INDENT: code indent should use tabs where possible
#1313: FILE: windows/netuio/netuio_queue.c:194:
+ WDFREQUEST Request)$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1313: FILE: windows/netuio/netuio_queue.c:194:
+ WDFREQUEST Request)$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1315: FILE: windows/netuio/netuio_queue.c:196:
+ WDF_REQUEST_PARAMETERS params = { 0 };$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1316: FILE: windows/netuio/netuio_queue.c:197:
+ NTSTATUS status = STATUS_SUCCESS;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1317: FILE: windows/netuio/netuio_queue.c:198:
+ PVOID output_buf = NULL;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1318: FILE: windows/netuio/netuio_queue.c:199:
+ size_t output_buf_size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1319: FILE: windows/netuio/netuio_queue.c:200:
+ size_t bytes_returned = 0;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1320: FILE: windows/netuio/netuio_queue.c:201:
+ PNETUIO_CONTEXT_DATA ctx = NULL;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1322: FILE: windows/netuio/netuio_queue.c:203:
+ ctx = netuio_get_context_data(Device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1324: FILE: windows/netuio/netuio_queue.c:205:
+ WDF_REQUEST_PARAMETERS_INIT(¶ms);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1325: FILE: windows/netuio/netuio_queue.c:206:
+ WdfRequestGetParameters(Request, ¶ms);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1327: FILE: windows/netuio/netuio_queue.c:208:
+ if (params.Type != WdfRequestTypeDeviceControl)$
ERROR:OPEN_BRACE: that open brace { should be on the previous line
#1327: FILE: windows/netuio/netuio_queue.c:208:
+ if (params.Type != WdfRequestTypeDeviceControl)
+ {
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1328: FILE: windows/netuio/netuio_queue.c:209:
+ {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1329: FILE: windows/netuio/netuio_queue.c:210:
+ status = STATUS_INVALID_DEVICE_REQUEST;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1329: FILE: windows/netuio/netuio_queue.c:210:
+ status = STATUS_INVALID_DEVICE_REQUEST;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1330: FILE: windows/netuio/netuio_queue.c:211:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1330: FILE: windows/netuio/netuio_queue.c:211:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1331: FILE: windows/netuio/netuio_queue.c:212:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1333: FILE: windows/netuio/netuio_queue.c:214:
+ // We only need to be in the context of the process that initiated the request$
ERROR:C99_COMMENTS: do not use C99 // comments
#1333: FILE: windows/netuio/netuio_queue.c:214:
+ // We only need to be in the context of the process that initiated the request
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1334: FILE: windows/netuio/netuio_queue.c:215:
+ //when we need to map memory to userspace. Otherwise, send the request back to framework.$
ERROR:C99_COMMENTS: do not use C99 // comments
#1334: FILE: windows/netuio/netuio_queue.c:215:
+ //when we need to map memory to userspace. Otherwise, send the request back to framework.
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1335: FILE: windows/netuio/netuio_queue.c:216:
+ if (params.Parameters.DeviceIoControl.IoControlCode != IOCTL_NETUIO_MAP_HW_INTO_USERSPACE)$
ERROR:OPEN_BRACE: that open brace { should be on the previous line
#1335: FILE: windows/netuio/netuio_queue.c:216:
+ if (params.Parameters.DeviceIoControl.IoControlCode != IOCTL_NETUIO_MAP_HW_INTO_USERSPACE)
+ {
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1336: FILE: windows/netuio/netuio_queue.c:217:
+ {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1337: FILE: windows/netuio/netuio_queue.c:218:
+ status = WdfDeviceEnqueueRequest(Device, Request);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1337: FILE: windows/netuio/netuio_queue.c:218:
+ status = WdfDeviceEnqueueRequest(Device, Request);$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1338: FILE: windows/netuio/netuio_queue.c:219:
+ if (!NT_SUCCESS(status))$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1338: FILE: windows/netuio/netuio_queue.c:219:
+ if (!NT_SUCCESS(status))$
ERROR:OPEN_BRACE: that open brace { should be on the previous line
#1338: FILE: windows/netuio/netuio_queue.c:219:
+ if (!NT_SUCCESS(status))
+ {
WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 12)
#1338: FILE: windows/netuio/netuio_queue.c:219:
+ if (!NT_SUCCESS(status))
+ {
ERROR:CODE_INDENT: code indent should use tabs where possible
#1339: FILE: windows/netuio/netuio_queue.c:220:
+ {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1339: FILE: windows/netuio/netuio_queue.c:220:
+ {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1340: FILE: windows/netuio/netuio_queue.c:221:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1340: FILE: windows/netuio/netuio_queue.c:221:
+ goto end;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1341: FILE: windows/netuio/netuio_queue.c:222:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1341: FILE: windows/netuio/netuio_queue.c:222:
+ }$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1342: FILE: windows/netuio/netuio_queue.c:223:
+ return;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1342: FILE: windows/netuio/netuio_queue.c:223:
+ return;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1343: FILE: windows/netuio/netuio_queue.c:224:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1345: FILE: windows/netuio/netuio_queue.c:226:
+ // Return relevant data to the caller$
ERROR:C99_COMMENTS: do not use C99 // comments
#1345: FILE: windows/netuio/netuio_queue.c:226:
+ // Return relevant data to the caller
WARNING:LONG_LINE: line over 100 characters
#1346: FILE: windows/netuio/netuio_queue.c:227:
+ status = WdfRequestRetrieveOutputBuffer(Request, sizeof(struct device_info), &output_buf, &output_buf_size);
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1346: FILE: windows/netuio/netuio_queue.c:227:
+ status = WdfRequestRetrieveOutputBuffer(Request, sizeof(struct device_info), &output_buf, &output_buf_size);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1347: FILE: windows/netuio/netuio_queue.c:228:
+ if (!NT_SUCCESS(status)) {$
WARNING:BRACES: braces {} are not necessary for single statement blocks
#1347: FILE: windows/netuio/netuio_queue.c:228:
+ if (!NT_SUCCESS(status)) {
+ goto end;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#1348: FILE: windows/netuio/netuio_queue.c:229:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1348: FILE: windows/netuio/netuio_queue.c:229:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1349: FILE: windows/netuio/netuio_queue.c:230:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1351: FILE: windows/netuio/netuio_queue.c:232:
+ status = netuio_map_address_into_user_process(ctx, Request);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1352: FILE: windows/netuio/netuio_queue.c:233:
+ if (!NT_SUCCESS(status)) {$
WARNING:BRACES: braces {} are not necessary for single statement blocks
#1352: FILE: windows/netuio/netuio_queue.c:233:
+ if (!NT_SUCCESS(status)) {
+ goto end;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#1353: FILE: windows/netuio/netuio_queue.c:234:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1353: FILE: windows/netuio/netuio_queue.c:234:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1354: FILE: windows/netuio/netuio_queue.c:235:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1356: FILE: windows/netuio/netuio_queue.c:237:
+ netuio_set_usermode_mapping_flag(Request);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1358: FILE: windows/netuio/netuio_queue.c:239:
+ netuio_handle_get_hw_data_request(ctx, output_buf, output_buf_size);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1359: FILE: windows/netuio/netuio_queue.c:240:
+ bytes_returned = output_buf_size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1362: FILE: windows/netuio/netuio_queue.c:243:
+ WdfRequestCompleteWithInformation(Request, status, bytes_returned);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1364: FILE: windows/netuio/netuio_queue.c:245:
+ return;$
WARNING:BLOCK_COMMENT_STYLE: Block comments use * on subsequent lines
#1368: FILE: windows/netuio/netuio_queue.c:249:
+/*
+Routine Description:
ERROR:CODE_INDENT: code indent should use tabs where possible
#1377: FILE: windows/netuio/netuio_queue.c:258:
+ size_t OutputBufferLength, size_t InputBufferLength,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1377: FILE: windows/netuio/netuio_queue.c:258:
+ size_t OutputBufferLength, size_t InputBufferLength,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1378: FILE: windows/netuio/netuio_queue.c:259:
+ ULONG IoControlCode)$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1378: FILE: windows/netuio/netuio_queue.c:259:
+ ULONG IoControlCode)$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1380: FILE: windows/netuio/netuio_queue.c:261:
+ UNREFERENCED_PARAMETER(OutputBufferLength);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1381: FILE: windows/netuio/netuio_queue.c:262:
+ UNREFERENCED_PARAMETER(InputBufferLength);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1383: FILE: windows/netuio/netuio_queue.c:264:
+ NTSTATUS status = STATUS_SUCCESS;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1384: FILE: windows/netuio/netuio_queue.c:265:
+ PVOID input_buf = NULL, output_buf = NULL;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1385: FILE: windows/netuio/netuio_queue.c:266:
+ size_t input_buf_size, output_buf_size;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1386: FILE: windows/netuio/netuio_queue.c:267:
+ size_t bytes_returned = 0;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1388: FILE: windows/netuio/netuio_queue.c:269:
+ WDFDEVICE device = WdfIoQueueGetDevice(Queue);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1390: FILE: windows/netuio/netuio_queue.c:271:
+ PNETUIO_CONTEXT_DATA ctx;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1391: FILE: windows/netuio/netuio_queue.c:272:
+ ctx = netuio_get_context_data(device);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1393: FILE: windows/netuio/netuio_queue.c:274:
+ if (IoControlCode != IOCTL_NETUIO_PCI_CONFIG_IO)$
ERROR:OPEN_BRACE: that open brace { should be on the previous line
#1393: FILE: windows/netuio/netuio_queue.c:274:
+ if (IoControlCode != IOCTL_NETUIO_PCI_CONFIG_IO)
+ {
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1394: FILE: windows/netuio/netuio_queue.c:275:
+ {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1395: FILE: windows/netuio/netuio_queue.c:276:
+ status = STATUS_INVALID_DEVICE_REQUEST;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1395: FILE: windows/netuio/netuio_queue.c:276:
+ status = STATUS_INVALID_DEVICE_REQUEST;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1396: FILE: windows/netuio/netuio_queue.c:277:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1396: FILE: windows/netuio/netuio_queue.c:277:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1397: FILE: windows/netuio/netuio_queue.c:278:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1399: FILE: windows/netuio/netuio_queue.c:280:
+ // First retrieve the input buffer and see if it matches our device$
ERROR:C99_COMMENTS: do not use C99 // comments
#1399: FILE: windows/netuio/netuio_queue.c:280:
+ // First retrieve the input buffer and see if it matches our device
WARNING:LONG_LINE: line over 100 characters
#1400: FILE: windows/netuio/netuio_queue.c:281:
+ status = WdfRequestRetrieveInputBuffer(Request, sizeof(struct dpdk_pci_config_io), &input_buf, &input_buf_size);
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1400: FILE: windows/netuio/netuio_queue.c:281:
+ status = WdfRequestRetrieveInputBuffer(Request, sizeof(struct dpdk_pci_config_io), &input_buf, &input_buf_size);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1401: FILE: windows/netuio/netuio_queue.c:282:
+ if (!NT_SUCCESS(status)) {$
WARNING:BRACES: braces {} are not necessary for single statement blocks
#1401: FILE: windows/netuio/netuio_queue.c:282:
+ if (!NT_SUCCESS(status)) {
+ goto end;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#1402: FILE: windows/netuio/netuio_queue.c:283:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1402: FILE: windows/netuio/netuio_queue.c:283:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1403: FILE: windows/netuio/netuio_queue.c:284:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1405: FILE: windows/netuio/netuio_queue.c:286:
+ struct dpdk_pci_config_io *dpdk_pci_io_input = (struct dpdk_pci_config_io *)input_buf;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1407: FILE: windows/netuio/netuio_queue.c:288:
+ if (dpdk_pci_io_input->access_size != 1 &&$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1408: FILE: windows/netuio/netuio_queue.c:289:
+ dpdk_pci_io_input->access_size != 2 &&$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1408: FILE: windows/netuio/netuio_queue.c:289:
+ dpdk_pci_io_input->access_size != 2 &&$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1409: FILE: windows/netuio/netuio_queue.c:290:
+ dpdk_pci_io_input->access_size != 4 &&$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1409: FILE: windows/netuio/netuio_queue.c:290:
+ dpdk_pci_io_input->access_size != 4 &&$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1410: FILE: windows/netuio/netuio_queue.c:291:
+ dpdk_pci_io_input->access_size != 8) {$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1410: FILE: windows/netuio/netuio_queue.c:291:
+ dpdk_pci_io_input->access_size != 8) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1411: FILE: windows/netuio/netuio_queue.c:292:
+ status = STATUS_INVALID_PARAMETER;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1411: FILE: windows/netuio/netuio_queue.c:292:
+ status = STATUS_INVALID_PARAMETER;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1412: FILE: windows/netuio/netuio_queue.c:293:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1412: FILE: windows/netuio/netuio_queue.c:293:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1413: FILE: windows/netuio/netuio_queue.c:294:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1415: FILE: windows/netuio/netuio_queue.c:296:
+ // Retrieve output buffer$
ERROR:C99_COMMENTS: do not use C99 // comments
#1415: FILE: windows/netuio/netuio_queue.c:296:
+ // Retrieve output buffer
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1416: FILE: windows/netuio/netuio_queue.c:297:
+ status = WdfRequestRetrieveOutputBuffer(Request, sizeof(UINT64), &output_buf, &output_buf_size);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1417: FILE: windows/netuio/netuio_queue.c:298:
+ if (!NT_SUCCESS(status)) {$
WARNING:BRACES: braces {} are not necessary for single statement blocks
#1417: FILE: windows/netuio/netuio_queue.c:298:
+ if (!NT_SUCCESS(status)) {
+ goto end;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#1418: FILE: windows/netuio/netuio_queue.c:299:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1418: FILE: windows/netuio/netuio_queue.c:299:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1419: FILE: windows/netuio/netuio_queue.c:300:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1420: FILE: windows/netuio/netuio_queue.c:301:
+ ASSERT(output_buf_size == OutputBufferLength);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1422: FILE: windows/netuio/netuio_queue.c:303:
+ if (dpdk_pci_io_input->op == PCI_IO_READ) {$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1423: FILE: windows/netuio/netuio_queue.c:304:
+ *(UINT64 *)output_buf = 0;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1423: FILE: windows/netuio/netuio_queue.c:304:
+ *(UINT64 *)output_buf = 0;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1424: FILE: windows/netuio/netuio_queue.c:305:
+ bytes_returned = ctx->bus_interface.GetBusData($
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1424: FILE: windows/netuio/netuio_queue.c:305:
+ bytes_returned = ctx->bus_interface.GetBusData($
ERROR:CODE_INDENT: code indent should use tabs where possible
#1425: FILE: windows/netuio/netuio_queue.c:306:
+ ctx->bus_interface.Context,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1425: FILE: windows/netuio/netuio_queue.c:306:
+ ctx->bus_interface.Context,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1426: FILE: windows/netuio/netuio_queue.c:307:
+ PCI_WHICHSPACE_CONFIG,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1426: FILE: windows/netuio/netuio_queue.c:307:
+ PCI_WHICHSPACE_CONFIG,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1427: FILE: windows/netuio/netuio_queue.c:308:
+ output_buf,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1427: FILE: windows/netuio/netuio_queue.c:308:
+ output_buf,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1428: FILE: windows/netuio/netuio_queue.c:309:
+ dpdk_pci_io_input->offset,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1428: FILE: windows/netuio/netuio_queue.c:309:
+ dpdk_pci_io_input->offset,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1429: FILE: windows/netuio/netuio_queue.c:310:
+ dpdk_pci_io_input->access_size);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1429: FILE: windows/netuio/netuio_queue.c:310:
+ dpdk_pci_io_input->access_size);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1430: FILE: windows/netuio/netuio_queue.c:311:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1431: FILE: windows/netuio/netuio_queue.c:312:
+ else if (dpdk_pci_io_input->op == PCI_IO_WRITE) {$
ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#1431: FILE: windows/netuio/netuio_queue.c:312:
+ }
+ else if (dpdk_pci_io_input->op == PCI_IO_WRITE) {
ERROR:CODE_INDENT: code indent should use tabs where possible
#1432: FILE: windows/netuio/netuio_queue.c:313:
+ // returns bytes written$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1432: FILE: windows/netuio/netuio_queue.c:313:
+ // returns bytes written$
ERROR:C99_COMMENTS: do not use C99 // comments
#1432: FILE: windows/netuio/netuio_queue.c:313:
+ // returns bytes written
ERROR:CODE_INDENT: code indent should use tabs where possible
#1433: FILE: windows/netuio/netuio_queue.c:314:
+ bytes_returned = ctx->bus_interface.SetBusData($
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1433: FILE: windows/netuio/netuio_queue.c:314:
+ bytes_returned = ctx->bus_interface.SetBusData($
ERROR:CODE_INDENT: code indent should use tabs where possible
#1434: FILE: windows/netuio/netuio_queue.c:315:
+ ctx->bus_interface.Context,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1434: FILE: windows/netuio/netuio_queue.c:315:
+ ctx->bus_interface.Context,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1435: FILE: windows/netuio/netuio_queue.c:316:
+ PCI_WHICHSPACE_CONFIG,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1435: FILE: windows/netuio/netuio_queue.c:316:
+ PCI_WHICHSPACE_CONFIG,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1436: FILE: windows/netuio/netuio_queue.c:317:
+ (PVOID)&dpdk_pci_io_input->data,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1436: FILE: windows/netuio/netuio_queue.c:317:
+ (PVOID)&dpdk_pci_io_input->data,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1437: FILE: windows/netuio/netuio_queue.c:318:
+ dpdk_pci_io_input->offset,$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1437: FILE: windows/netuio/netuio_queue.c:318:
+ dpdk_pci_io_input->offset,$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1438: FILE: windows/netuio/netuio_queue.c:319:
+ dpdk_pci_io_input->access_size);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1438: FILE: windows/netuio/netuio_queue.c:319:
+ dpdk_pci_io_input->access_size);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1439: FILE: windows/netuio/netuio_queue.c:320:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1440: FILE: windows/netuio/netuio_queue.c:321:
+ else {$
ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#1440: FILE: windows/netuio/netuio_queue.c:321:
+ }
+ else {
ERROR:CODE_INDENT: code indent should use tabs where possible
#1441: FILE: windows/netuio/netuio_queue.c:322:
+ status = STATUS_INVALID_PARAMETER;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1441: FILE: windows/netuio/netuio_queue.c:322:
+ status = STATUS_INVALID_PARAMETER;$
ERROR:CODE_INDENT: code indent should use tabs where possible
#1442: FILE: windows/netuio/netuio_queue.c:323:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1442: FILE: windows/netuio/netuio_queue.c:323:
+ goto end;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1443: FILE: windows/netuio/netuio_queue.c:324:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1445: FILE: windows/netuio/netuio_queue.c:326:
+ if (bytes_returned != dpdk_pci_io_input->access_size) {$
WARNING:BRACES: braces {} are not necessary for single statement blocks
#1445: FILE: windows/netuio/netuio_queue.c:326:
+ if (bytes_returned != dpdk_pci_io_input->access_size) {
+ status = STATUS_INVALID_PARAMETER;
+ }
ERROR:CODE_INDENT: code indent should use tabs where possible
#1446: FILE: windows/netuio/netuio_queue.c:327:
+ status = STATUS_INVALID_PARAMETER;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1446: FILE: windows/netuio/netuio_queue.c:327:
+ status = STATUS_INVALID_PARAMETER;$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1447: FILE: windows/netuio/netuio_queue.c:328:
+ }$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1450: FILE: windows/netuio/netuio_queue.c:331:
+ WdfRequestCompleteWithInformation(Request, status, bytes_returned);$
WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1452: FILE: windows/netuio/netuio_queue.c:333:
+ return;$
ERROR:C99_COMMENTS: do not use C99 // comments
#1480: FILE: windows/netuio/netuio_queue.h:21:
+#endif // NETUIO_QUEUE_H
total: 235 errors, 478 warnings, 1275 lines checked
^ permalink raw reply [flat|nested] 3+ messages in thread
* [dpdk-test-report] |SUCCESS| pw79510 windows/netuio: add Windows NetUIO kernel driver
[not found] <1601592908-11262-1-git-send-email-navasile@linux.microsoft.com>
2020-10-01 22:57 ` [dpdk-test-report] |WARNING| pw79510 [PATCH v5] windows/netuio: add Windows NetUIO kernel driver checkpatch
@ 2020-10-02 0:55 ` 0-day Robot
1 sibling, 0 replies; 3+ messages in thread
From: 0-day Robot @ 2020-10-02 0:55 UTC (permalink / raw)
To: test-report; +Cc: robot
From: robot@bytheb.org
Test-Label: travis-robot
Test-Status: SUCCESS
http://dpdk.org/patch/79510
_Travis build: passed_
Build URL: https://travis-ci.com/ovsrobot/dpdk/builds/187723791
^ permalink raw reply [flat|nested] 3+ messages in thread
* [dpdk-test-report] |WARNING| pw79510 [PATCH] [v5] windows/netuio: add Windows NetUIO kernel driver
@ 2020-10-06 16:09 dpdklab
0 siblings, 0 replies; 3+ messages in thread
From: dpdklab @ 2020-10-06 16:09 UTC (permalink / raw)
To: Test Report; +Cc: dpdk-test-reports
[-- Attachment #1: Type: text/plain, Size: 2190 bytes --]
Test-Label: iol-testing
Test-Status: WARNING
http://dpdk.org/patch/79510
_Testing issues_
Submitter: Narcisa Ana Maria Vasile <navasile@linux.microsoft.com>
Date: Thursday, October 01 2020 22:55:08
DPDK git baseline: Repo:dpdk
Branch: master
CommitID:c2b603bdf433f0ae4c4a63dcaa660c8d1792bfe8
79510 --> testing fail
Test environment and result as below:
+---------------------+--------------------+----------------+
| Environment | dpdk_meson_compile | dpdk_unit_test |
+=====================+====================+================+
| openSUSE Leap 15 | PASS | SKIPPED |
+---------------------+--------------------+----------------+
| Windows Server 2019 | PASS | SKIPPED |
+---------------------+--------------------+----------------+
| FreeBSD 11.2 | PASS | SKIPPED |
+---------------------+--------------------+----------------+
| Ubuntu 18.04 | PASS | PASS |
+---------------------+--------------------+----------------+
| CentOS 8 | PASS | SKIPPED |
+---------------------+--------------------+----------------+
| Fedora 31 | PASS | SKIPPED |
+---------------------+--------------------+----------------+
| Fedora Rawhide | WARN | SKIPPED |
+---------------------+--------------------+----------------+
| Arch Linux | PASS | SKIPPED |
+---------------------+--------------------+----------------+
openSUSE Leap 15
Kernel: 4.12.14
Compiler: gcc 7.4.1
Windows Server 2019
Kernel: 10.0
Compiler: clang 9.0
FreeBSD 11.2
Kernel: 11.2
Compiler: gcc 8.3
Ubuntu 18.04
Kernel: 4.15.0-generic
Compiler: gcc 7.4
CentOS 8
Kernel: 4.18.0.el8_0
Compiler: gcc 8.3.1
Fedora 31
Kernel: 5.3.16
Compiler: gcc 9.2.1
Fedora Rawhide
Kernel: 5.6.8
Compiler: gcc 10.2.1
Arch Linux
Kernel: latest
Compiler: gcc latest
To view detailed results, visit:
https://lab.dpdk.org/results/dashboard/patchsets/13418/
UNH-IOL DPDK Community Lab
To manage your email subscriptions, visit:
https://lab.dpdk.org/results/dashboard/preferences/subscriptions/
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-10-06 16:09 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1601592908-11262-1-git-send-email-navasile@linux.microsoft.com>
2020-10-01 22:57 ` [dpdk-test-report] |WARNING| pw79510 [PATCH v5] windows/netuio: add Windows NetUIO kernel driver checkpatch
2020-10-02 0:55 ` [dpdk-test-report] |SUCCESS| pw79510 " 0-day Robot
2020-10-06 16:09 [dpdk-test-report] |WARNING| pw79510 [PATCH] [v5] " dpdklab
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).