Lustre devel - May 2007 - [Bug 12411] Build patchless client packages

Please don''t reply to lustre-devel. Instead, comment in Bugzilla by
using the following link:
https://bugzilla.lustre.org/show_bug.cgi?id=12411


>From Andreas:
It should be possible (and normal for 1.6.x) that a "patchless" client
run
on a patched server.  For clients the biggest part of patched vs patchless
is whether the vfs_intent patch is present or not.

To be honest, I''d rather have the "patchless" support be
determined by
whether vfs_intent or vfs_nointent is patched into the kernel instead
of "lustre_version.h" (which I think we should keep for the patched
servers).

Please don''t reply to lustre-devel. Instead, comment in Bugzilla by
using the following link:
https://bugzilla.lustre.org/show_bug.cgi?id=12411

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |adilger@clusterfs.com


Should we actually _remove_ the client patches entirely?  If we leave them in,
people could still use them by editing the series file, but I''d rather
take them
out entirely so we don''t need to maintain them.  Already, I''ll
need to update
and reorder some of the client patches if we keep them in.

Also, since we plan to stop testing patched clients, I don''t think
it''s
something we should support.

Please don''t reply to lustre-devel. Instead, comment in Bugzilla by
using the following link:
https://bugzilla.lustre.org/show_bug.cgi?id=12411



(In reply to comment #12)
> I think we should remove the vfs patches entirely.  We can always get them
from
> CVS, and if we leave them in the package someone will think they need them.
OK.
> There are a couple of small patches that might make sense to keep for the
> "patchless" clients running on the server kernels, one of them is
> export-show_stack.      
You mean export-show_task?  That''s what we use when dumping the stack. 
I left
it in the series since it''s also useful on servers.