<div dir="ltr">ah, another way by not building-in loggen, thanks.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 27, 2013 at 7:11 AM, Dave Chinner <span dir="ltr"><<a href="mailto:david@fromorbit.com" target="_blank">david@fromorbit.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Sun, May 26, 2013 at 07:11:03AM +0800, Zhi Yong Wu wrote:<br>
> I remove the lines related to h_chksum, then it can work well, so i<br>
> guess that i should send out one patch to fix it.<br>
<br>
</div>While it will make the code in loggen work the same (it just writes<br>
0 to the field but the buffer is already zeroed) as it does now,<br>
it doesn't necessarily mean it works correctly. The current kernels<br>
validate the CRC and throw warnings if the CRC isn't correct, and<br>
CRC enabled kernels will simply refuse to mount.<br>
<br>
As it is, my test systems don't build loggen because<br>
HAVE_XLOG_ASSIGN_LSN isn't set by configure. It only gets set if<br>
there's an xfs/libxfs.h header file found in an include path. My<br>
guess is that you've run 'make install-qa' in the xfsprogs package<br>
at some time in the past, and that's why loggen is trying to be<br>
built.<br>
<br>
So Michael's quick hack to remove it just to make it compile at this<br>
point is probably appropriate...<br>
<div class="HOEnZb"><div class="h5"><br>
Cheers,<br>
<br>
Dave.<br>
--<br>
Dave Chinner<br>
<a href="mailto:david@fromorbit.com">david@fromorbit.com</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><div>Regards,</div><div> </div><div>Zhi Yong Wu</div>
</div>