<div class="gmail_extra"><br><br>
<div class="gmail_quote">On Fri, Apr 27, 2012 at 5:15 PM, Christoph Hellwig <span dir="ltr"><<a href="mailto:hch@infradead.org" target="_blank">hch@infradead.org</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT:#ccc 1px solid;MARGIN:0px 0px 0px 0.8ex;PADDING-LEFT:1ex" class="gmail_quote">
<div class="im">On Fri, Apr 27, 2012 at 03:05:46PM +0800, <a href="mailto:yaohaiting.wujue@gmail.com">yaohaiting.wujue@gmail.com</a> wrote:<br>> Problems:<br>><br>> 1, race condition<br>><br>> get_ordered_sd_vnode_list gets the cache and try to use it, but<br>
> free_ordered_sd_vnode_list frees it.<br><br></div>So what the order by using a local variable.<br>
<div class="im"><br>> 2, no cache when I/O is not tense<br>><br>> create and free the cache frequently when I/O is not tense<br><br></div>See my "vnode mapping layer fixes and cleanups" series, which fixes this<br>
as a side effect.<br><br></blockquote>
<div> </div>
<div>I found this bug today and tried to fix it. Since there is no problem anymore, let's abandon the changes of this patch.</div>
<div> </div>
<div>Thanks</div>
<div>Haiti</div></div><br></div>