Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-11507

lustre doesn't build against zfs master: refcount_add missing

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • Lustre 2.12.0
    • Upstream
    • None
    • Centos 7
    • 3
    • 9223372036854775807

    Description

      The latest lustre will not build against ZFS master:

        CC [M]  /home/hutter/lustre-release/lustre/osd-zfs/osd_handler.o
       In file included from /home/hutter/lustre-release/lustre/osd-zfs/osd_handler.c:52:0:
       /home/hutter/lustre-release/lustre/osd-zfs/osd_internal.h: In function 'osd_sa_handle_get':
       /home/hutter/lustre-release/lustre/osd-zfs/osd_internal.h:929:2: error: implicit declaration of function 'refcount_add' [-Werror=implicit-function-declaration]
        refcount_add(&dn->dn_bonus->db_holds, osd_obj_tag);
      

      refcount_add was removed in ZFS in:

      Linux 4.19-rc3+ compat: Remove refcount_t compat (https://github.com/zfsonlinux/zfs/pull/7932).

      That patch is expected to be included in zfs-0.7.12 as well.

      Lustre should be updated to use zfs_refcount_add() if it exists, and then fall back to refcount_add if not.

      Attachments

        Issue Links

          Activity

            [LU-11507] lustre doesn't build against zfs master: refcount_add missing
            pjones Peter Jones made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-11393 [ LU-11393 ]
            pjones Peter Jones made changes -
            Fix Version/s New: Lustre 2.12.0 [ 13495 ]
            pjones Peter Jones made changes -
            Assignee Original: Nathaniel Clark [ utopiabound ] New: Tony Hutter [ hutter ]
            adilger Andreas Dilger made changes -
            Assignee Original: WC Triage [ wc-triage ] New: Nathaniel Clark [ utopiabound ]
            adilger Andreas Dilger made changes -
            Description Original: The latest lustre will not build against ZFS master:

            {{ }}
            {{  CC [M]  /home/hutter/lustre-release/lustre/osd-zfs/osd_handler.o}}
            {{ In file included from /home/hutter/lustre-release/lustre/osd-zfs/osd_handler.c:52:0:}}
            {{ /home/hutter/lustre-release/lustre/osd-zfs/osd_internal.h: In function 'osd_sa_handle_get':}}
            {{ /home/hutter/lustre-release/lustre/osd-zfs/osd_internal.h:929:2: error: implicit declaration of function 'refcount_add' [-Werror=implicit-function-declaration]}}
            {{  refcount_add(&dn->dn_bonus->db_holds, osd_obj_tag);}}
            {{  }}

            refcount_add was removed in ZFS in:

            Linux 4.19-rc3+ compat: Remove refcount_t compat ([https://github.com/zfsonlinux/zfs/pull/7932]).

            That patch is expected to be included in zfs-0.7.12 as well.

            Lustre should be updated to use zfs_refcount_add() if it exists, and then fall back to refcount_add if not.
            New: The latest lustre will not build against ZFS master:

            {noformat}
              CC [M]  /home/hutter/lustre-release/lustre/osd-zfs/osd_handler.o
             In file included from /home/hutter/lustre-release/lustre/osd-zfs/osd_handler.c:52:0:
             /home/hutter/lustre-release/lustre/osd-zfs/osd_internal.h: In function 'osd_sa_handle_get':
             /home/hutter/lustre-release/lustre/osd-zfs/osd_internal.h:929:2: error: implicit declaration of function 'refcount_add' [-Werror=implicit-function-declaration]
              refcount_add(&dn->dn_bonus->db_holds, osd_obj_tag);
            {noformat}

            refcount_add was removed in ZFS in:

            Linux 4.19-rc3+ compat: Remove refcount_t compat ([https://github.com/zfsonlinux/zfs/pull/7932]).

            That patch is expected to be included in zfs-0.7.12 as well.

            Lustre should be updated to use zfs_refcount_add() if it exists, and then fall back to refcount_add if not.
            hutter Tony Hutter made changes -
            Attachment New: refcount_add.diff [ 31186 ]
            hutter Tony Hutter created issue -

            People

              hutter Tony Hutter
              hutter Tony Hutter
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: