Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

5313 Allow I/Os to be aggregated across ZIO priority classes #3280

Closed
wants to merge 1 commit into from

Conversation

DeHackEd
Copy link
Contributor

Reviewed by: Andriy Gapon avg@FreeBSD.org
Reviewed by: Will Andrews willa@SpectraLogic.com
Reviewed by: Matt Ahrens mahrens@delphix.com
Reviewed by: George Wilson george@delphix.com
Approved by: Robert Mustacchi rm@joyent.com

References:
https://www.illumos.org/issues/5313
illumos/illumos-gate@fe319232

Ported-by: DHE git@dehacked.net

@DeHackEd
Copy link
Contributor Author

TODO: IllumOS's vdev_queue.c has functions in different orders. For synchronization we should re-order our code to match.

@DeHackEd DeHackEd force-pushed the zio-priority-aggregate branch from eb85ce7 to 28939f0 Compare April 11, 2015 22:33
@DeHackEd
Copy link
Contributor Author

Fix make checkstyle, one comment was misaligned

@behlendorf
Copy link
Contributor

Just for reference, the Fedora 19 failure in the latest test run is unrelated to this change,

Reviewed by: Andriy Gapon <avg@FreeBSD.org>
Reviewed by: Will Andrews <willa@SpectraLogic.com>
Reviewed by: Matt Ahrens <mahrens@delphix.com>
Reviewed by: George Wilson <george@delphix.com>
Approved by: Robert Mustacchi <rm@joyent.com>

References:
  https://www.illumos.org/issues/5313
  illumos/illumos-gate@fe319232

Ported-by: DHE <git@dehacked.net>
@DeHackEd DeHackEd force-pushed the zio-priority-aggregate branch from 28939f0 to e76757a Compare April 14, 2015 23:20
@DeHackEd
Copy link
Contributor Author

Fixed a copy/paste error in vdev_impl.h as indicated previously.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants