<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META content="text/html; charset=us-ascii" http-equiv=Content-Type>
<META name=GENERATOR content="MSHTML 8.00.6001.18812"></HEAD>
<BODY>
<DIV dir=ltr align=left><SPAN class=515565515-16092009><FONT color=#0000ff
size=2 face=Arial>Hello,</FONT></SPAN></DIV>
<DIV dir=ltr align=left><SPAN class=515565515-16092009><FONT color=#0000ff
size=2 face=Arial> This patch appears to have cured the sometimes BSOD on
power-down. After 53 uninstalls I have yet to witness a
BSOD!!</FONT></SPAN></DIV>
<DIV dir=ltr align=left><SPAN class=515565515-16092009><FONT color=#0000ff
size=2 face=Arial>There was a similar patch to winverbs (paged vs. non-paged PNP
struct allocation) which also cured a power-down BSOD, so it's not too
surprising this patch has the same effect.</FONT></SPAN></DIV>
<DIV dir=ltr align=left><SPAN class=515565515-16092009><FONT color=#0000ff
size=2 face=Arial>I will be including it in the WinOF 2.1 release (upcoming
RC5).</FONT></SPAN></DIV>
<DIV dir=ltr align=left><SPAN class=515565515-16092009><FONT color=#0000ff
size=2 face=Arial></FONT></SPAN> </DIV>
<DIV dir=ltr align=left><SPAN class=515565515-16092009><FONT color=#0000ff
size=2 face=Arial>stan.</FONT></SPAN></DIV><BR>
<DIV dir=ltr lang=en-us class=OutlookMessageHeader align=left>
<HR tabIndex=-1>
<FONT size=2 face=Tahoma><B>From:</B> ofw-bounces@lists.openfabrics.org
[mailto:ofw-bounces@lists.openfabrics.org] <B>On Behalf Of </B>Leonid
Keller<BR><B>Sent:</B> Tuesday, September 15, 2009 1:15 AM<BR><B>To:</B>
ofw_list<BR><B>Subject:</B> [ofw][patch][ibal] use non-pageable memory to
prevent possible problems on power down<BR></FONT><BR></DIV>
<DIV></DIV>
<DIV><FONT size=2 face=Arial><SPAN class=362050208-15092009>IBAL uses pageable
memory to create PnP context.</SPAN></FONT></DIV>
<DIV><FONT size=2 face=Arial><SPAN class=362050208-15092009>It can create
possible problems in power down flows at the time of system
contention.</SPAN></FONT></DIV>
<DIV><FONT size=2 face=Arial><SPAN
class=362050208-15092009></SPAN></FONT> </DIV>
<DIV><FONT size=2 face=Arial><SPAN class=362050208-15092009>We saw some similar
case at a customer.</SPAN></FONT></DIV>
<DIV><FONT size=2 face=Arial><SPAN class=362050208-15092009>There is no strong
evidence that that is what influenced, but with this patch IBAL will be more
safe and at no cost.</SPAN></FONT></DIV>
<DIV><FONT size=2 face=Arial><SPAN
class=362050208-15092009></SPAN></FONT> </DIV>
<DIV><FONT size=2 face=Arial><SPAN class=362050208-15092009>Found by Hobin Lee
(Xsigo).</SPAN></FONT></DIV>
<DIV><FONT size=2 face=Arial></FONT> </DIV>
<DIV><FONT size=2 face=Arial>Index:
kernel/al_pnp.c<BR>===================================================================<BR>---
kernel/al_pnp.c (revision 2421)<BR>+++ kernel/al_pnp.c (working
copy)<BR>@@ -587,7 +587,7 @@<BR> CL_ASSERT( p_reg
);<BR> <BR> /* No context exists for this port. Create
one. */<BR>- p_context = (al_pnp_context_t*)cl_pzalloc(
sizeof(al_pnp_context_t) );<BR>+ p_context = (al_pnp_context_t*)cl_zalloc(
sizeof(al_pnp_context_t) );<BR> if( !p_context
)<BR> {<BR> AL_PRINT( TRACE_LEVEL_ERROR,
AL_DBG_ERROR,<BR></FONT></DIV></BODY></HTML>