> On 8/11/17 4:38 PM, GerritHub wrote:
> > Frank Filz has uploaded this change for *review*.
> >
> > View Change <https://review.gerrithub.io/373991>
> >
> > Remove support_ex FSAL method
> >
> Frank, I reviewed the simple removes, and even this one, but got
> flummoxed where you change code to assume support_ex.  I don't see how
> this would be separate, or bisectible, from them.

Once FSAL_ZFS was removed, all in-tree FSALs claim support_ex, so any 
non-support_ex code paths are dead, and can be safely removed in bits and 
pieces and in (almost) any order (I did remove from top of stack down, so there 
should not be any linker failures).

> There were a couple of CEA -1 that passed cthon, and even pynfs.
> Whatever that bug (some counting of COMPOUND elements?), it doesn't
> seem related to these patches.  I'd ignore that -1.

I wasn't getting too worried about the CEA -1 verify yet...

Frank


---
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Nfs-ganesha-devel mailing list
Nfs-ganesha-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel

Reply via email to