I added EXPORT_SYMBOL_GPL for those function to get passed it. On Wed, May 30, 2018 at 3:27 PM, Olga Kornievskaia <aglo@xxxxxxxxx> wrote: > It doesn't compile. Am I missing something? > > ERROR: "nfs4_inode_make_writeable" [fs/nfs/nfs.ko] undefined! > ERROR: "nfs4_inode_return_delegation" [fs/nfs/nfs.ko] undefined! > > On Wed, May 30, 2018 at 2:37 PM, Olga Kornievskaia <aglo@xxxxxxxxx> wrote: >> Awesome thanks. >> >> On Wed, May 30, 2018 at 2:36 PM, Trond Myklebust >> <trondmy@xxxxxxxxxxxxxxx> wrote: >>> On Wed, 2018-05-30 at 14:25 -0400, Olga Kornievskaia wrote: >>>> Hi Trond, >>>> >>>> Do you by any chance have this patch set somewhere in your git >>>> branches? I'd like to test this against NetApp. >>>> >>>> Thank you. >>>> >>> >>> I've just pushed it out to my testing branch on git.linux-nfs.org. >>> >>>> On Wed, May 30, 2018 at 2:05 PM, Trond Myklebust <trondmy@xxxxxxxxx> >>>> wrote: >>>> > This patchset is something we've been sitting on for a couple of >>>> > years >>>> > in order to allow it to mature. It was mainly developed by Fred >>>> > Isaman >>>> > while he was working for Primary Data, but we've had a few bugfixes >>>> > come in since then. >>>> > The main reason for delaying the upstream merge was to ensure that >>>> > adding layoutget to the OPEN compound was safe (does not cause >>>> > existing >>>> > layouts to be automatically revoked) and that it gives us a real >>>> > performance benefit. >>>> > >>>> > Fred Isaman (14): >>>> > pnfs: Remove redundant assignment from nfs4_proc_layoutget(). >>>> > pnfs: Store return value of decode_layoutget for later processing >>>> > NFS4: move ctx into nfs4_run_open_task >>>> > pnfs: Add layout driver flag PNFS_LAYOUTGET_ON_OPEN >>>> > pnfs: refactor send_layoutget >>>> > pnfs: move allocations out of nfs4_proc_layoutget >>>> > pnfs: Add conditional encode/decode of LAYOUTGET within OPEN >>>> > compound >>>> > pnfs: Move nfs4_opendata into nfs4_fs.h >>>> > pnfs: Change pnfs_alloc_init_layoutget_args call signature >>>> > pnfs: Add LAYOUTGET to OPEN of a new file >>>> > pnfs: Add LAYOUTGET to OPEN of an existing file >>>> > pnfs: Stop attempting LAYOUTGET on OPEN on failure >>>> > pnfs: Add barrier to prevent lgopen using LAYOUTGET during recall >>>> > pnfs: Fix manipulation of NFS_LAYOUT_FIRST_LAYOUTGET >>>> > >>>> > Trond Myklebust (5): >>>> > NFSv4/pnfs: Ensure pnfs_parse_lgopen() won't try to parse >>>> > uninitialised data >>>> > NFSv4/pnfs: Don't switch off layoutget-on-open for transient >>>> > errors >>>> > pNFS: Don't send LAYOUTGET on OPEN for read, if we already have >>>> > cached >>>> > data >>>> > pnfs: Don't call commit on failed layoutget-on-open >>>> > pnfs: Don't release the sequence slot until we've processed >>>> > layoutget >>>> > on open >>>> > >>>> > fs/nfs/callback_proc.c | 2 + >>>> > fs/nfs/flexfilelayout/flexfilelayout.c | 1 + >>>> > fs/nfs/nfs4_fs.h | 25 +++ >>>> > fs/nfs/nfs4proc.c | 97 +++++---- >>>> > fs/nfs/nfs4state.c | 8 + >>>> > fs/nfs/nfs4xdr.c | 65 +++++- >>>> > fs/nfs/pnfs.c | 266 >>>> > +++++++++++++++++++++---- >>>> > fs/nfs/pnfs.h | 28 ++- >>>> > include/linux/nfs_fs_sb.h | 2 + >>>> > include/linux/nfs_xdr.h | 4 + >>>> > 10 files changed, 398 insertions(+), 100 deletions(-) >>>> > >>>> > -- >>>> > 2.17.0 >>>> > >>>> > -- >>>> > To unsubscribe from this list: send the line "unsubscribe linux- >>>> > nfs" in >>>> > the body of a message to majordomo@xxxxxxxxxxxxxxx >>>> > More majordomo info at http://vger.kernel.org/majordomo-info.html >>>> >>>> -- >>>> To unsubscribe from this list: send the line "unsubscribe linux-nfs" >>>> in >>>> the body of a message to majordomo@xxxxxxxxxxxxxxx >>>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> -- >>> Trond Myklebust >>> Linux NFS client maintainer, Hammerspace >>> trond.myklebust@xxxxxxxxxxxxxxx >>> -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
