Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.12.0
    • None
    • 3
    • 9223372036854775807

    Description

      While Mike Marciniszyn was testing the TID-RDMA he tried to build Lustre against upstream. And ran into this build error:

      make[3]: Entering directory `/nfs/sc/disks/fabric_work/mmarciniszyn/repos/wfr/wf                                                                                                                                                             r-linux-devel'
        CC [M]  /nfs/sc/disks/fabric_work/mmarciniszyn/repos/lustre-release/libcfs/libcfs/linux/linux-tracefile.o
        CC [M]  /nfs/sc/disks/fabric_work/mmarciniszyn/repos/lustre-release/libcfs/libcfs/linux/linux-debug.o
      In file included from /nfs/sc/disks/fabric_work/mmarciniszyn/repos/lustre-release/libcfs/libcfs/linux/linux-debug.c:50:0:
      ./arch/x86/include/asm/uaccess.h: In function ‘set_fs’:
      ./arch/x86/include/asm/uaccess.h:31:9: error: dereferencing pointer to incomplete type
        current->thread.addr_limit = fs; 
      

      Attachments

        Issue Links

          Activity

            [LU-10560] Fixes for 4.14 kernel

            Yang Sheng (yang.sheng@intel.com) uploaded a new patch: https://review.whamcloud.com/32268
            Subject: LU-10560 llite: remove assigning f_version in ll_readir.
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 1de9c46e2dbb64cf788f40e52bf829b334b9da0c

            gerrit Gerrit Updater added a comment - Yang Sheng (yang.sheng@intel.com) uploaded a new patch: https://review.whamcloud.com/32268 Subject: LU-10560 llite: remove assigning f_version in ll_readir. Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 1de9c46e2dbb64cf788f40e52bf829b334b9da0c

            We have a patch for ldiskfs under this ticket so don't close.

            simmonsja James A Simmons added a comment - We have a patch for ldiskfs under this ticket so don't close.

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/31948/
            Subject: LU-10560 libcfs: Fix inconsistent declaration for cfs_kernel_write()
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: f19699186d29ba8561d67b4457e98c018ea26317

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/31948/ Subject: LU-10560 libcfs: Fix inconsistent declaration for cfs_kernel_write() Project: fs/lustre-release Branch: master Current Patch Set: Commit: f19699186d29ba8561d67b4457e98c018ea26317
            pjones Peter Jones added a comment -

            Ok. 4.15 work is happening under LU-10805 but it sounds like this last change should be enough to support 4.14 - thanks!

            pjones Peter Jones added a comment - Ok. 4.15 work is happening under LU-10805 but it sounds like this last change should be enough to support 4.14 - thanks!

            I noted that that the error in the backport in https://review.whamcloud.com/#/c/31928 and that is probably due to the const discrepancy.

            This is the only issue I have with 4.14.x kernel and it as it is working fine for our testing.

            There are issues with 4.15.x, but i haven't gotten to those yet.   Some notables:

            • timer API changes
            • const nits

            My objective here is to do some Lustre testing with our upstream work for OPA (lnet_selftest, lnet_ping)

            mmarcini Mike Marciniszyn (Inactive) added a comment - I noted that that the error in the backport in https://review.whamcloud.com/#/c/31928  and that is probably due to the const discrepancy. This is the only issue I have with 4.14.x kernel and it as it is working fine for our testing. There are issues with 4.15.x, but i haven't gotten to those yet.   Some notables: timer API changes const nits My objective here is to do some Lustre testing with our upstream work for OPA (lnet_selftest, lnet_ping)
            pjones Peter Jones added a comment -

            mmarcini is this now the last patch or are there still more to come?

            pjones Peter Jones added a comment - mmarcini is this now the last patch or are there still more to come?

            Mike Marciniszyn (mike.marciniszyn@intel.com) uploaded a new patch: https://review.whamcloud.com/31948
            Subject: LU-10560 libcfs: Fix inconsistent const for cfg_kernel_write()
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 874d454c7aeafb849862778dc6f163cc9c1938d7

            gerrit Gerrit Updater added a comment - Mike Marciniszyn (mike.marciniszyn@intel.com) uploaded a new patch: https://review.whamcloud.com/31948 Subject: LU-10560 libcfs: Fix inconsistent const for cfg_kernel_write() Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 874d454c7aeafb849862778dc6f163cc9c1938d7

            Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31928
            Subject: LU-10560 libcfs: Use kernel_write when appropriate
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set: 1
            Commit: 94d96caa0986d6ff539d75496f3443f159f32313

            gerrit Gerrit Updater added a comment - Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31928 Subject: LU-10560 libcfs: Use kernel_write when appropriate Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 94d96caa0986d6ff539d75496f3443f159f32313

            Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31927
            Subject: LU-10560 libcfs: handle rename to wait_queue_entry_t
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set: 1
            Commit: 13e2315bf370f67845770a34c658a83ab26a9720

            gerrit Gerrit Updater added a comment - Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31927 Subject: LU-10560 libcfs: handle rename to wait_queue_entry_t Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 13e2315bf370f67845770a34c658a83ab26a9720

            Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31926
            Subject: LU-10560 lustre_compat: Convert GFP_TEMPORARY to GFP_KERNEL
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set: 1
            Commit: e22e1cd2631e3c4a85577d001d5b0571fe8f77eb

            gerrit Gerrit Updater added a comment - Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31926 Subject: LU-10560 lustre_compat: Convert GFP_TEMPORARY to GFP_KERNEL Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: e22e1cd2631e3c4a85577d001d5b0571fe8f77eb

            Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31925
            Subject: LU-10560 llite: remove extra headers from rw26.c
            Project: fs/lustre-release
            Branch: b2_10
            Current Patch Set: 1
            Commit: ff9ab6a368ce3bf2187b047051b7eefc7a997ae0

            gerrit Gerrit Updater added a comment - Minh Diep (minh.diep@intel.com) uploaded a new patch: https://review.whamcloud.com/31925 Subject: LU-10560 llite: remove extra headers from rw26.c Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: ff9ab6a368ce3bf2187b047051b7eefc7a997ae0

            People

              simmonsja James A Simmons
              ashehata Amir Shehata (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              11 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: