4045 zfs write throttle & i/o scheduler performance work

[Andriy Gapon]

on 15/09/2013 20:08 Andriy Gapon said the following:
> 
> I wonder if anyone already has a patch for integrating "4045 zfs write throttle
> & i/o scheduler performance work" from Illumos to FreeBSD.  Or if anyone is
> working on this.
> 
> We would like to test this change in our environment as soon as possible.  So I
> could start working on it if needed.
> 
> BTW, it seems that "3581 spa_zio_taskq[ZIO_TYPE_FREE][ZIO_TASKQ_ISSUE]->tq_lock
> is piping hot" was not integrated into FreeBSD despite being committed to
> Illumos back in February.  I am not sure if we need the main change from this
> commit, but, as it is frequent with Illumos commits, it contains some changes to
> definitions of some macros, enums and structs.  So I think it is worth having
> that change merged, at least partially.  That would help with merge the future
> changing including "4045 zfs write throttle & i/o scheduler performance work".
>

OK, I've done some work on merging these changes.
Please review:
https://github.com/avg-I/freebsd/compare/illumos;merge-zfs-write-throttle
https://github.com/avg-I/freebsd/commits/illumos/merge-zfs-write-throttle
As I am not sure that I've got everything right, I will appreciate it.

Especially please see my concerns for this commit:
https://github.com/avg-I/freebsd/commit/5c7d4daa51c78477c8f0c9af6de7c99a81ff6418
I am sure that there is something to be concerned about :)

-- 
Andriy Gapon