<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:x="urn:schemas-microsoft-com:office:excel" xmlns:p="urn:schemas-microsoft-com:office:powerpoint" xmlns:a="urn:schemas-microsoft-com:office:access" xmlns:dt="uuid:C2F41010-65B3-11d1-A29F-00AA00C14882" xmlns:s="uuid:BDC6E3F0-6DA3-11d1-A2A3-00AA00C14882" xmlns:rs="urn:schemas-microsoft-com:rowset" xmlns:z="#RowsetSchema" xmlns:b="urn:schemas-microsoft-com:office:publisher" xmlns:ss="urn:schemas-microsoft-com:office:spreadsheet" xmlns:c="urn:schemas-microsoft-com:office:component:spreadsheet" xmlns:odc="urn:schemas-microsoft-com:office:odc" xmlns:oa="urn:schemas-microsoft-com:office:activation" xmlns:html="http://www.w3.org/TR/REC-html40" xmlns:q="http://schemas.xmlsoap.org/soap/envelope/" xmlns:rtc="http://microsoft.com/officenet/conferencing" xmlns:D="DAV:" xmlns:Repl="http://schemas.microsoft.com/repl/" xmlns:mt="http://schemas.microsoft.com/sharepoint/soap/meetings/" xmlns:x2="http://schemas.microsoft.com/office/excel/2003/xml" xmlns:ppda="http://www.passport.com/NameSpace.xsd" xmlns:ois="http://schemas.microsoft.com/sharepoint/soap/ois/" xmlns:dir="http://schemas.microsoft.com/sharepoint/soap/directory/" xmlns:ds="http://www.w3.org/2000/09/xmldsig#" xmlns:dsp="http://schemas.microsoft.com/sharepoint/dsp" xmlns:udc="http://schemas.microsoft.com/data/udc" xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:sub="http://schemas.microsoft.com/sharepoint/soap/2002/1/alerts/" xmlns:ec="http://www.w3.org/2001/04/xmlenc#" xmlns:sp="http://schemas.microsoft.com/sharepoint/" xmlns:sps="http://schemas.microsoft.com/sharepoint/soap/" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:udcs="http://schemas.microsoft.com/data/udc/soap" xmlns:udcxf="http://schemas.microsoft.com/data/udc/xmlfile" xmlns:udcp2p="http://schemas.microsoft.com/data/udc/parttopart" xmlns:wf="http://schemas.microsoft.com/sharepoint/soap/workflow/" xmlns:dsss="http://schemas.microsoft.com/office/2006/digsig-setup" xmlns:dssi="http://schemas.microsoft.com/office/2006/digsig" xmlns:mdssi="http://schemas.openxmlformats.org/package/2006/digital-signature" xmlns:mver="http://schemas.openxmlformats.org/markup-compatibility/2006" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns:mrels="http://schemas.openxmlformats.org/package/2006/relationships" xmlns:spwp="http://microsoft.com/sharepoint/webpartpages" xmlns:ex12t="http://schemas.microsoft.com/exchange/services/2006/types" xmlns:ex12m="http://schemas.microsoft.com/exchange/services/2006/messages" xmlns:pptsl="http://schemas.microsoft.com/sharepoint/soap/SlideLibrary/" xmlns:spsl="http://microsoft.com/webservices/SharePointPortalServer/PublishedLinksService" xmlns:Z="urn:schemas-microsoft-com:" xmlns:st="" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv=Content-Type content="text/html; charset=us-ascii">
<meta name=Generator content="Microsoft Word 12 (filtered medium)">
<style>
<!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-compose;
font-family:"Calibri","sans-serif";
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.25in 1.0in 1.25in;}
div.WordSection1
{page:WordSection1;}
-->
</style>
<!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang=EN-US link=blue vlink=purple>
<div class=WordSection1>
<p class=MsoNormal>I got a BS while disabling the IPoIB driver, the reason for
that is trying to delete WDF device that is invalid. I guess that the object
already deleted but I’m not sure.<o:p></o:p></p>
<p class=MsoNormal><o:p> </o:p></p>
<p class=MsoNormal>Is WmPowerD0Exit and WmPowerD0Entry functions are
thread safe? In other words is only one instance of the function can be run in
same time?<o:p></o:p></p>
<p class=MsoNormal><o:p> </o:p></p>
<p class=MsoNormal>In order to catch the problem I added some ASSERT in the
code and set the pointer to NULL after deleting the object.<o:p></o:p></p>
<p class=MsoNormal><o:p> </o:p></p>
<p class=MsoNormal>Attached is the patch<o:p></o:p></p>
<p class=MsoNormal><o:p> </o:p></p>
<p class=MsoNormal>Index: winmad/kernel/wm_driver.c<o:p></o:p></p>
<p class=MsoNormal>===================================================================<o:p></o:p></p>
<p class=MsoNormal>--- winmad/kernel/wm_driver.c (revision
6842)<o:p></o:p></p>
<p class=MsoNormal>+++ winmad/kernel/wm_driver.c (working
copy)<o:p></o:p></p>
<p class=MsoNormal>@@ -41,7 +41,7 @@<o:p></o:p></p>
<p class=MsoNormal> WDF_DECLARE_CONTEXT_TYPE_WITH_NAME(WM_IB_DEVICE,
WmIbDeviceGetContext)<o:p></o:p></p>
<p class=MsoNormal> WDF_DECLARE_CONTEXT_TYPE_WITH_NAME(WM_PROVIDER,
WmProviderGetContext)<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>-WDFDEVICE ControlDevice;<o:p></o:p></p>
<p class=MsoNormal>+WDFDEVICE ControlDevice
= NULL;<o:p></o:p></p>
<p class=MsoNormal> static LIST_ENTRY DevList;<o:p></o:p></p>
<p class=MsoNormal> static LIST_ENTRY ProvList;<o:p></o:p></p>
<p class=MsoNormal> static KGUARDED_MUTEX Lock;<o:p></o:p></p>
<p class=MsoNormal>@@ -195,11 +195,14 @@<o:p></o:p></p>
<p class=MsoNormal> WdfDeviceInitSetFileObjectConfig(pinit,
&fileconfig, &attr);<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal> WDF_OBJECT_ATTRIBUTES_INIT(&attr);<o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
== NULL);<o:p></o:p></p>
<p class=MsoNormal> status
= WdfDeviceCreate(&pinit, &attr, &ControlDevice);<o:p></o:p></p>
<p class=MsoNormal> if
(!NT_SUCCESS(status)) {<o:p></o:p></p>
<p class=MsoNormal> goto
err1;<o:p></o:p></p>
<p class=MsoNormal> }<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>+ <o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
!= NULL);<o:p></o:p></p>
<p class=MsoNormal> status
= WdfDeviceCreateSymbolicLink(ControlDevice, &symlink);<o:p></o:p></p>
<p class=MsoNormal> if
(!NT_SUCCESS(status)) {<o:p></o:p></p>
<p class=MsoNormal> goto
err2;<o:p></o:p></p>
<p class=MsoNormal>@@ -220,6 +223,7 @@<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal> err2:<o:p></o:p></p>
<p class=MsoNormal> WdfObjectDelete(ControlDevice);<o:p></o:p></p>
<p class=MsoNormal>+ ControlDevice
= NULL;<o:p></o:p></p>
<p class=MsoNormal> return;<o:p></o:p></p>
<p class=MsoNormal> err1:<o:p></o:p></p>
<p class=MsoNormal> WdfDeviceInitFree(pinit);<o:p></o:p></p>
<p class=MsoNormal>@@ -340,14 +344,12 @@<o:p></o:p></p>
<p class=MsoNormal> WM_REGISTRATION *reg;<o:p></o:p></p>
<p class=MsoNormal> LIST_ENTRY *entry;<o:p></o:p></p>
<p class=MsoNormal> BOOLEAN destroy;<o:p></o:p></p>
<p class=MsoNormal>- WDFDEVICE ctrldev;<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal> pdev
= WmIbDeviceGetContext(Device);<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal> KeAcquireGuardedMutex(&Lock);<o:p></o:p></p>
<p class=MsoNormal> RemoveEntryList(&pdev->Entry);<o:p></o:p></p>
<p class=MsoNormal> destroy
= IsListEmpty(&DevList);<o:p></o:p></p>
<p class=MsoNormal>- ctrldev
= ControlDevice;<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal> for
(entry = ProvList.Flink; entry != &ProvList; entry = entry->Flink) {<o:p></o:p></p>
<p class=MsoNormal> prov
= CONTAINING_RECORD(entry, WM_PROVIDER, Entry);<o:p></o:p></p>
<p class=MsoNormal>@@ -365,7 +367,8 @@<o:p></o:p></p>
<p class=MsoNormal> }<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal> if
(destroy) {<o:p></o:p></p>
<p class=MsoNormal>- WdfObjectDelete(ctrldev);<o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
!= NULL);<o:p></o:p></p>
<p class=MsoNormal>+ WdfObjectDelete(ControlDevice);<o:p></o:p></p>
<p class=MsoNormal> }<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal> return
STATUS_SUCCESS;<o:p></o:p></p>
<p class=MsoNormal>Index: winmad/kernel/wm_provider.c<o:p></o:p></p>
<p class=MsoNormal>===================================================================<o:p></o:p></p>
<p class=MsoNormal>--- winmad/kernel/wm_provider.c (revision
6842)<o:p></o:p></p>
<p class=MsoNormal>+++ winmad/kernel/wm_provider.c (working
copy)<o:p></o:p></p>
<p class=MsoNormal>@@ -64,6 +64,8 @@<o:p></o:p></p>
<p class=MsoNormal> pProvider->Exclusive
= 0;<o:p></o:p></p>
<p class=MsoNormal> KeInitializeEvent(&pProvider->ExclusiveEvent,
SynchronizationEvent, FALSE);<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
!= NULL);<o:p></o:p></p>
<p class=MsoNormal>+ <o:p></o:p></p>
<p class=MsoNormal> WDF_IO_QUEUE_CONFIG_INIT(&config,
WdfIoQueueDispatchManual);<o:p></o:p></p>
<p class=MsoNormal> status
= WdfIoQueueCreate(ControlDevice, &config,<o:p></o:p></p>
<p class=MsoNormal>
WDF_NO_OBJECT_ATTRIBUTES, &pProvider->ReadQueue);<o:p></o:p></p>
<p class=MsoNormal>Index: winverbs/kernel/wv_cq.c<o:p></o:p></p>
<p class=MsoNormal>===================================================================<o:p></o:p></p>
<p class=MsoNormal>--- winverbs/kernel/wv_cq.c (revision
6842)<o:p></o:p></p>
<p class=MsoNormal>+++ winverbs/kernel/wv_cq.c (working copy)<o:p></o:p></p>
<p class=MsoNormal>@@ -97,6 +97,8 @@<o:p></o:p></p>
<p class=MsoNormal> cq->Ref
= 1;<o:p></o:p></p>
<p class=MsoNormal> KeInitializeEvent(&cq->Event,
NotificationEvent, FALSE);<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
!= NULL);<o:p></o:p></p>
<p class=MsoNormal>+<o:p></o:p></p>
<p class=MsoNormal> WDF_IO_QUEUE_CONFIG_INIT(&config,
WdfIoQueueDispatchManual);<o:p></o:p></p>
<p class=MsoNormal> status
= WdfIoQueueCreate(ControlDevice, &config,<o:p></o:p></p>
<p class=MsoNormal>
WDF_NO_OBJECT_ATTRIBUTES, &cq->Queue);<o:p></o:p></p>
<p class=MsoNormal>Index: winverbs/kernel/wv_device.c<o:p></o:p></p>
<p class=MsoNormal>===================================================================<o:p></o:p></p>
<p class=MsoNormal>--- winverbs/kernel/wv_device.c (revision
6842)<o:p></o:p></p>
<p class=MsoNormal>+++ winverbs/kernel/wv_device.c (working
copy)<o:p></o:p></p>
<p class=MsoNormal>@@ -216,6 +216,8 @@<o:p></o:p></p>
<p class=MsoNormal> return
STATUS_NO_MEMORY;<o:p></o:p></p>
<p class=MsoNormal> }<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
!= NULL);<o:p></o:p></p>
<p class=MsoNormal>+<o:p></o:p></p>
<p class=MsoNormal> WDF_IO_QUEUE_CONFIG_INIT(&config,
WdfIoQueueDispatchManual);<o:p></o:p></p>
<p class=MsoNormal> for
(i = 0; i < pDevice->PortCount; i++) {<o:p></o:p></p>
<p class=MsoNormal> pDevice->pPorts[i].Flags
= 0;<o:p></o:p></p>
<p class=MsoNormal>Index: winverbs/kernel/wv_driver.c<o:p></o:p></p>
<p class=MsoNormal>===================================================================<o:p></o:p></p>
<p class=MsoNormal>--- winverbs/kernel/wv_driver.c (revision
6842)<o:p></o:p></p>
<p class=MsoNormal>+++ winverbs/kernel/wv_driver.c (working
copy)<o:p></o:p></p>
<p class=MsoNormal>@@ -48,7 +48,7 @@<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal> WDF_DECLARE_CONTEXT_TYPE_WITH_NAME(WV_RDMA_DEVICE,
WvRdmaDeviceGetContext)<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>-WDFDEVICE ControlDevice;<o:p></o:p></p>
<p class=MsoNormal>+WDFDEVICE ControlDevice
= NULL;<o:p></o:p></p>
<p class=MsoNormal> static LIST_ENTRY DevList;<o:p></o:p></p>
<p class=MsoNormal> static LIST_ENTRY ProvList;<o:p></o:p></p>
<p class=MsoNormal> static KGUARDED_MUTEX Lock;<o:p></o:p></p>
<p class=MsoNormal>@@ -565,6 +565,8 @@<o:p></o:p></p>
<p class=MsoNormal> ifc.InterfaceHeader.InterfaceReference
= WdfDeviceInterfaceReferenceNoOp;<o:p></o:p></p>
<p class=MsoNormal> ifc.InterfaceHeader.InterfaceDereference
= WdfDeviceInterfaceReferenceNoOp;<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
!= NULL);<o:p></o:p></p>
<p class=MsoNormal>+<o:p></o:p></p>
<p class=MsoNormal> ifc.WdfDevice
= (ULONG_PTR)ControlDevice;<o:p></o:p></p>
<p class=MsoNormal> ifc.WdmDevice
= WdfDeviceWdmGetDeviceObject(ControlDevice);<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>@@ -608,12 +610,16 @@<o:p></o:p></p>
<p class=MsoNormal> WDF_OBJECT_ATTRIBUTES_INIT_CONTEXT_TYPE(&attr,
WV_PROVIDER);<o:p></o:p></p>
<p class=MsoNormal> WdfDeviceInitSetFileObjectConfig(pinit,
&fileconfig, &attr);<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
== NULL);<o:p></o:p></p>
<p class=MsoNormal>+<o:p></o:p></p>
<p class=MsoNormal> WDF_OBJECT_ATTRIBUTES_INIT(&attr);<o:p></o:p></p>
<p class=MsoNormal> status
= WdfDeviceCreate(&pinit, &attr, &ControlDevice);<o:p></o:p></p>
<p class=MsoNormal> if
(!NT_SUCCESS(status)) {<o:p></o:p></p>
<p class=MsoNormal> goto
err1;<o:p></o:p></p>
<p class=MsoNormal> }<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
!= NULL);<o:p></o:p></p>
<p class=MsoNormal>+ <o:p></o:p></p>
<p class=MsoNormal> status
= WdfDeviceCreateSymbolicLink(ControlDevice, &symlink);<o:p></o:p></p>
<p class=MsoNormal> if
(!NT_SUCCESS(status)) {<o:p></o:p></p>
<p class=MsoNormal> goto
err2;<o:p></o:p></p>
<p class=MsoNormal>@@ -633,6 +639,7 @@<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal> err2:<o:p></o:p></p>
<p class=MsoNormal> WdfObjectDelete(ControlDevice);<o:p></o:p></p>
<p class=MsoNormal>+ ControlDevice
= NULL;<o:p></o:p></p>
<p class=MsoNormal> return;<o:p></o:p></p>
<p class=MsoNormal> err1:<o:p></o:p></p>
<p class=MsoNormal> WdfDeviceInitFree(pinit);<o:p></o:p></p>
<p class=MsoNormal>@@ -723,7 +730,6 @@<o:p></o:p></p>
<p class=MsoNormal> KeAcquireGuardedMutex(&Lock);<o:p></o:p></p>
<p class=MsoNormal> RemoveEntryList(&pdev->Entry);<o:p></o:p></p>
<p class=MsoNormal> destroy
= IsListEmpty(&DevList);<o:p></o:p></p>
<p class=MsoNormal>- ctrldev
= ControlDevice;<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal> for
(entry = ProvList.Flink; entry != &ProvList; entry = entry->Flink) {<o:p></o:p></p>
<p class=MsoNormal> prov
= CONTAINING_RECORD(entry, WV_PROVIDER, Entry);<o:p></o:p></p>
<p class=MsoNormal>@@ -744,7 +750,10 @@<o:p></o:p></p>
<p class=MsoNormal> IbCmInterface.InterfaceHeader.InterfaceDereference(IbCmInterface.<o:p></o:p></p>
<p class=MsoNormal>
InterfaceHeader.Context);<o:p></o:p></p>
<p class=MsoNormal> }<o:p></o:p></p>
<p class=MsoNormal>- WdfObjectDelete(ctrldev);<o:p></o:p></p>
<p class=MsoNormal>+<o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
!+ NULL);<o:p></o:p></p>
<p class=MsoNormal>+ WdfObjectDelete(ControlDevice);<o:p></o:p></p>
<p class=MsoNormal>+ ControlDevice
= NULL;<o:p></o:p></p>
<p class=MsoNormal> }<o:p></o:p></p>
<p class=MsoNormal> }<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>Index: winverbs/kernel/wv_ep.c<o:p></o:p></p>
<p class=MsoNormal>===================================================================<o:p></o:p></p>
<p class=MsoNormal>--- winverbs/kernel/wv_ep.c (revision
6842)<o:p></o:p></p>
<p class=MsoNormal>+++ winverbs/kernel/wv_ep.c (working copy)<o:p></o:p></p>
<p class=MsoNormal>@@ -101,6 +101,8 @@<o:p></o:p></p>
<p class=MsoNormal> KeInitializeEvent(&ep->Event,
NotificationEvent, FALSE);<o:p></o:p></p>
<p class=MsoNormal> InitializeListHead(&ep->Entry);<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
!= NULL);<o:p></o:p></p>
<p class=MsoNormal>+ <o:p></o:p></p>
<p class=MsoNormal> WDF_IO_QUEUE_CONFIG_INIT(&config,
WdfIoQueueDispatchManual);<o:p></o:p></p>
<p class=MsoNormal> status
= WdfIoQueueCreate(ControlDevice, &config,<o:p></o:p></p>
<p class=MsoNormal>
WDF_NO_OBJECT_ATTRIBUTES, &ep->Queue);<o:p></o:p></p>
<p class=MsoNormal>Index: winverbs/kernel/wv_srq.c<o:p></o:p></p>
<p class=MsoNormal>===================================================================<o:p></o:p></p>
<p class=MsoNormal>--- winverbs/kernel/wv_srq.c (revision
6842)<o:p></o:p></p>
<p class=MsoNormal>+++ winverbs/kernel/wv_srq.c (working copy)<o:p></o:p></p>
<p class=MsoNormal>@@ -109,6 +109,8 @@<o:p></o:p></p>
<p class=MsoNormal> srq->Ref
= 1;<o:p></o:p></p>
<p class=MsoNormal> KeInitializeEvent(&srq->Event,
NotificationEvent, FALSE);<o:p></o:p></p>
<p class=MsoNormal> <o:p></o:p></p>
<p class=MsoNormal>+ ASSERT(ControlDevice
!= NULL);<o:p></o:p></p>
<p class=MsoNormal>+<o:p></o:p></p>
<p class=MsoNormal> WDF_IO_QUEUE_CONFIG_INIT(&config,
WdfIoQueueDispatchManual);<o:p></o:p></p>
<p class=MsoNormal> status
= WdfIoQueueCreate(ControlDevice, &config,<o:p></o:p></p>
<p class=MsoNormal>
WDF_NO_OBJECT_ATTRIBUTES, &srq->Queue);<o:p></o:p></p>
</div>
</body>
</html>