<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Viresh,</div><div class="gmail_default" style="font-size:small"><br></div></div><div><div class="gmail_default" style="font-size:small">These new tests look good, only a few comments/questions below:</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Patch 1/10:</div><div class="gmail_default" style="font-size:small">1. The git summary should be updated too (someone who push patch can help do that:).</div><div class="gmail_default" style="font-size:small">2. Maybe better to replace the TWARN by TINFO? Since tst_is_mounted() as a general function to check if mount success, sometimes we just need the return status then do next work(I tend to leave the waring or break operate to LTP users:).</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Patch 5/10, 9/10: </div><div class="gmail_default" style="font-size:small">May I ask why we use "sync" as the key value in fsconfig()? I ask this because it can get rid of the errors we found in XFS test before.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Patch 9/10, 10/10:</div><div class="gmail_default" style="font-size:small">I guess that'd be better if we put the 'ismounted = 1' at the behind of tst_is_mounted(), do you feel the code sequence looks strange which we set 'ismounted' to 1 then do mount checking?</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Ack for the whole patchset v3 (+ follow some modification for above comments): </div><div class="gmail_default" style="font-size:small">    Acked-by: Li Wang <<a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a>></div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>