Sorry for the noise. I cleaned everything up, untarred a fresh copy of lustre1.8.5, applied the patch, configured, and successfully made patches. I'm not sure what went wrong last time. <div><br clear="all">Aaron<br><br>

<div class="gmail_quote">On Thu, May 19, 2011 at 12:14 PM, Johann Lombardi <span dir="ltr"><<a href="mailto:johann@whamcloud.com">johann@whamcloud.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 Thu, May 19, 2011 at 11:51:49AM -0400, Aaron Everett wrote:<br>
> I'm getting a build error:<br>
> make[5]: Entering directory `/usr/src/kernels/2.6.18-238.9.1.el5-x86_64'<br>
> /usr/src/redhat/BUILD/lustre-1.8.5/lustre/mdc/mdc_lib.c:828: error:<br>
> conflicting types for 'mdc_getattr_pack'<br>
> /usr/src/redhat/BUILD/lustre-1.8.5/lustre/mdc/mdc_internal.h:56: error:<br>
> previous declaration of 'mdc_getattr_pack' was here<br>
<br>
</div>Weird, the patch does not modify mdc_getattr_pack() at all.<br>
It applies cleanly to 1.8.5 for me and i can successfully build it.<br>
How did you pick up the patch? Have you made any changes to mdc_getattr_pack()?<br>
Please find attached the patch I used (extracted from git).<br>
<div class="im"><br>
>      55 void mdc_getattr_pack(struct ptlrpc_request *req, int offset, __u64<br>
> valid,<br>
>      56                       int flags, struct mdc_op_data *data);<br>
><br>
> mdc_lib.c (patched version downloaded from link above):<br>
>     828 {<br>
>     829         if (mdc_req_is_2_0_server(req))<br>
>     830                 mdc_getattr_pack_20(req, offset, valid, flags, data,<br>
> ea_size);<br>
<br>
</div>The patch from bugzilla ticket 24048 added a new argument to mdc_getattr_pack(), but that's not the patch i pointed you at.<br>
<div class="im"><br>
>     831         else<br>
>     832                 mdc_getattr_pack_18(req, offset, valid, flags,<br>
> data);<br>
>     833 }<br>
><br>
> Upon closer inspection, it appears this patch is for Lustre 1.8.6, while<br>
> we're running 1.8.5.<br>
<br>
</div>The patch i attached should really work with 1.8.5.<br>
<div class="im"><br>
> Is there a download location for lustre source for<br>
> 1.8.6? I don't see it on <a href="http://lustre.org" target="_blank">lustre.org</a>.<br>
<br>
</div>AFAIK, 1.8.6 has not been released yet.<br>
<div><div></div><div class="h5"><br>
Johann<br>
--<br>
Johann Lombardi<br>
Whamcloud, Inc.<br>
<a href="http://www.whamcloud.com" target="_blank">www.whamcloud.com</a><br>
</div></div></blockquote></div><br></div>