<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META http-equiv=Content-Type content="text/html; charset=us-ascii">
<META content="MSHTML 6.00.2900.5512" name=GENERATOR></HEAD>
<BODY>
<DIV><SPAN class=243512016-16092009><FONT face=Arial color=#0000ff
size=2>Applied to the trunk (2480).</FONT></SPAN></DIV><BR>
<BLOCKQUOTE dir=ltr
style="PADDING-LEFT: 5px; MARGIN-LEFT: 5px; BORDER-LEFT: #0000ff 2px solid; MARGIN-RIGHT: 0px">
<DIV class=OutlookMessageHeader lang=en-us dir=ltr align=left>
<HR tabIndex=-1>
<FONT face=Tahoma size=2><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 11:15 AM<BR><B>To:</B>
ofw_list<BR><B>Subject:</B> [ofw][patch][ibal] use non-pageable memory to
prevent possibleproblems on power down<BR></FONT><BR></DIV>
<DIV></DIV>
<DIV><FONT face=Arial size=2><SPAN class=362050208-15092009>IBAL uses pageable
memory to create PnP context.</SPAN></FONT></DIV>
<DIV><FONT face=Arial size=2><SPAN class=362050208-15092009>It can create
possible problems in power down flows at the time of system
contention.</SPAN></FONT></DIV>
<DIV><FONT face=Arial size=2><SPAN
class=362050208-15092009></SPAN></FONT> </DIV>
<DIV><FONT face=Arial size=2><SPAN class=362050208-15092009>We saw some
similar case at a customer.</SPAN></FONT></DIV>
<DIV><FONT face=Arial size=2><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 face=Arial size=2><SPAN
class=362050208-15092009></SPAN></FONT> </DIV>
<DIV><FONT face=Arial size=2><SPAN class=362050208-15092009>Found by Hobin Lee
(Xsigo).</SPAN></FONT></DIV>
<DIV><FONT face=Arial size=2></FONT> </DIV>
<DIV><FONT face=Arial size=2>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></BLOCKQUOTE></BODY></HTML>