Project

General

Profile

Bug #12518

Sync with openzfs

Added by Gernot Strasser 8 months ago. Updated 7 months ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Difficulty:
Medium
Tags:
Gerrit CR:

Description

Hello,

I would really love to see Illumos staying in sync with openzfs as far as possible.
Besides persistent L2ARC I am looking mostly at performance related patches like

https://github.com/openzfs/zfs/commit/20f287855a61a64a90e01edfd551ff28f7abdb5c
https://github.com/openzfs/zfs/commit/0929c4de398606f8305057ca540cf577e6771c30
https://github.com/openzfs/zfs/commit/1dc32a67e93bbc8d650943f1a460abb9ff6c5083
https://github.com/openzfs/zfs/commit/b3212d2fa6ab8d7d8373373e8a6b8acbbf45508e

Not even sure all of them apply to Illumos at all.
Much to my regret I can only offer help concerning testing.


Related issues

Related to illumos gate - Bug #12586: zvol_write() can use dmu_tx_hold_write_by_dnode()ClosedJason King

Actions
#1

Updated by Jason King 8 months ago

Those certainly look like they could help with things like bhyve.. if no one else gets to them, I'll try to take a look soon..

#2

Updated by Joshua M. Clulow 8 months ago

In a bigger sense, now that support for FreeBSD has been added to the central OpenZFS repository, we should look at reporting it back to illumos so that we can be officially supported there as well.

#3

Updated by Gernot Strasser 8 months ago

I am all in for that!

#4

Updated by Jason King 7 months ago

The last commit doesn't appear to be useful for illumos -- on illumos taskq_member() just checks a field of the current kthread_t, so it's already pretty lightweight -- the fix appears to be specific to the port of the taskq framework to ZoL.

The first one appears useful (and a pretty simple change).. I've filed #12586 for that..

#5

Updated by Jason King 7 months ago

  • Related to Bug #12586: zvol_write() can use dmu_tx_hold_write_by_dnode() added
#6

Updated by Jason King 7 months ago

The second commit on the list seems specific to how Linux queues I/Os and doesn't seem relevant to illumos.

Also available in: Atom PDF