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