https://jira.whamcloud.com This file is an XML representation of an issue en-us 9.4.14 940014 05-12-2023 https://jira.whamcloud.com/browse/LU-4944 Lustre <p>This looks a bit similar to <a href="https://jira.whamcloud.com/browse/LU-3903" title="build fails with latest zfs source" class="issue-link" data-issue-key="LU-3903"><del>LU-3903</del></a>.</p> <p>I can no longer build lustre against the latest upstream zfs source pulled from the master branch of github.com/behlendorf/zfs.git. The internal API dsl_sync_task_do_nowait() used in osd-zfs code no longer exists.</p> <p>This is not currently a problem in our build and test framework where builds are done against an earlier, fixed, and frozen version of zfs source. It will become a problem the next time we sync up to the latest upstream zfs version.</p> LU-4944

build fails with latest zfs source

Bug Critical Resolved Fixed Alex Zhuravlev Bob Glossman zfs Wed, 23 Apr 2014 18:43:42 +0000 Tue, 6 May 2014 15:19:08 +0000 Tue, 6 May 2014 15:19:08 +0000 Lustre 2.6.0 Lustre 2.6.0 0 8 <p>It really should be our goal to have the Lustre 2.6.0 release build with the ZFS 0.6.3 code (or as close as we can get).</p> <p>I thought dsl_sync_task_do_nowait() was added for Lustre's use, but I don't know the details of why it was then removed. Maybe Brian can comment on the API change?</p> <p>The other question is whether 0.6.3 will be released within the next month or so? If yes, then we should shift our build/test over to an intermediate tag on the ZFS master in anticipation of that release, instead of sticking with the existing 0.6.2 release.</p> <p>there does appear to be a replacement API, dsl_sync_task_nowait(). However the params it's called with don't look identical. I think we need some expert mods to adapt to the new API. If we want to be flexible and do it both ways we probably need some autoconf support to detect &amp; use the correct API for the zfs version built with.</p> <p>As part of the synctask restructuring done upstream in Illumos dsl_sync_task_do_nowait() was replaced by dsl_sync_task_nowait(). The updated function removes the 'checkfunc' argument but since this wasn't used by Lustre it should be straight forward to use the new API. Adding a configure test to check for the symbol should be easy enough.</p> <p>Unfortunately, we can't trivially add the interface back to ZFS side because the callbacks were updated to share the same passed argument. The previous interface allowed you to pass a different argument for the checkfunc and the syncfunc. Sorry guys.</p> <p>Brian or others what is the Latest ZFS Commit that should work without Lustre Changes?</p> <p>These changes were merged just a few commits after 0.6.2 was tagged. They've been in the ZFS tree since early September 2013.</p> <p>*13fe019 Illumos #3464<br/> 6f1ffb0 Illumos #2882, #2883, #2900<br/> 0c28fb4 Tag zfs-0.6.2</p> <p>as an interim step I've worked out a patch that replaces dsl_sync_task_do_nowait() with dsl_sync_task_nowait() in osd-zfs code. It will be a bit more work to add a configure test, but this temporary patch will at least allow builds to complete. Not at all production worthy (IMHO).</p> <p><a href="http://review.whamcloud.com/#/c/10064/" class="external-link" target="_blank" rel="nofollow noopener">http://review.whamcloud.com/#/c/10064/</a></p> <p>Patch landed to Master.</p> Related LU-4606 Development Rank 1|hzwkuf: Rank (Obsolete) 13683 Severity