<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<meta content="text/html; charset=UTF-8">
<style type="text/css" style="">
<!--
p
{margin-top:0;
margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Arial,Helvetica,sans-serif">
<p>Dan,</p>
<p><br>
</p>
<p>This is a A) a still fairly rarely used crypto feature [A full and usable implementation of it is relatively recent, and this is supporting code that I believe is older.]<span style="font-size:12pt">, and B) per that link, the crash only happens when you
change your shared key during operation. So, I'd guess it's a pretty rare thing to do.</span></p>
<p><span style="font-size:12pt"><br>
</span></p>
<p><span style="font-size:12pt">- Patrick</span></p>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> lustre-devel <lustre-devel-bounces@lists.lustre.org> on behalf of Dan Carpenter <dan.carpenter@oracle.com><br>
<b>Sent:</b> Tuesday, January 31, 2017 2:54:29 AM<br>
<b>To:</b> James Simmons<br>
<b>Cc:</b> devel@driverdev.osuosl.org; Greg Kroah-Hartman; Linux Kernel Mailing List; Oleg Drokin; Lustre Development List<br>
<b>Subject:</b> Re: [lustre-devel] [PATCH 21/60] staging: lustre: ptlrpc: correct use of list_add_tail()</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">On Sat, Jan 28, 2017 at 07:04:49PM -0500, James Simmons wrote:<br>
> From: "John L. Hammond" <john.hammond@intel.com><br>
> <br>
> In sptlrpc_gc_add_sec() swap the arguments to list_add_tail() so that<br>
> it does what we meant it to do.<br>
> <br>
<br>
Huh... This is from before lustre was merged into staging. What are<br>
the user visible effects of this bug? I would have expected it to get<br>
caught earlier.<br>
<br>
<br>
> Signed-off-by: John L. Hammond <john.hammond@intel.com><br>
> Intel-bug-id: <a href="https://jira.hpdd.intel.com/browse/LU-8270">https://jira.hpdd.intel.com/browse/LU-8270</a><br>
<br>
I bet the answer to my question is on this link but I'm reviewing<br>
offline right now. Plus that's not where the bug description belongs.<br>
<br>
regards,<br>
dan carpenter<br>
<br>
_______________________________________________<br>
lustre-devel mailing list<br>
lustre-devel@lists.lustre.org<br>
<a href="http://lists.lustre.org/listinfo.cgi/lustre-devel-lustre.org">http://lists.lustre.org/listinfo.cgi/lustre-devel-lustre.org</a><br>
</div>
</span></font>
</body>
</html>