<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 24, 2017 at 5:11 PM, Raghavendra G <span dir="ltr"><<a href="mailto:raghavendra@gluster.com" target="_blank">raghavendra@gluster.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Fri, Jul 21, 2017 at 6:39 PM, Vijay Bellur <span dir="ltr"><<a href="mailto:vbellur@redhat.com" target="_blank">vbellur@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><span>On Fri, Jul 21, 2017 at 3:26 AM, Raghavendra Gowdappa <span dir="ltr"><<a href="mailto:rgowdapp@redhat.com" target="_blank">rgowdapp@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
We've a bug [1], due to which read-ahead is completely disabled when the workload is read-only. One of the easy fix was to make read-ahead as an ancestor of open-behind in xlator graph (Currently its a descendant). A patch has been sent out by Rafi to do the same. As noted in one of the comments, one flip side of this solution is that small files (which are eligible to be cached by quick read) are cached twice - once each in read-ahead and quick-read - wasting up precious memory. However, there are no other simpler solutions for this issue. If you've concerns on the approach followed by [2] or have other suggestions please voice them out. Otherwise, I am planning to merge [2] for lack of better alternatives.<br></blockquote><div><br></div><div><br></div></span><div>Since the maximum size of files cached by quick-read is 64KB, can we have read-ahead kick in for offsets greater than 64KB?</div></div></div></div></blockquote><div><br></div></span><div>I got your point. We can enable read-ahead only for files whose size is greater than the size eligible for caching quick-read. IOW, read-ahead gets disabled if file size is less than 64KB. Thanks for the suggestion.<br></div></div></div></div></blockquote><div><br></div><div>I added a comment on the patch to move the xlators in reverse to the way the patch is currently doing. Milind I think implemented it. Will that lead to any problem?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div>Thanks,</div><div>Vijay</div></div></div></div><span class="">
<br>______________________________<wbr>_________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org" target="_blank">Gluster-devel@gluster.org</a><br>
<a href="http://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" target="_blank">http://lists.gluster.org/mailm<wbr>an/listinfo/gluster-devel</a><br></span></blockquote></div><span class="HOEnZb"><font color="#888888"><br><br clear="all"><br>-- <br><div class="m_5859304286038205615gmail_signature" data-smartmail="gmail_signature">Raghavendra G<br></div>
</font></span></div></div>
<br>______________________________<wbr>_________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org">Gluster-devel@gluster.org</a><br>
<a href="http://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" target="_blank">http://lists.gluster.org/<wbr>mailman/listinfo/gluster-devel</a><br></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Pranith<br></div></div>
</div></div>