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

[master re-sync of #2479, buildbot] zvol create minors check snapdev before traversing snapshots #3663

Conversation

kernelOfTruth
Copy link
Contributor

This is to address issue #2217 scalability issues with zvol_create_minors(pool). The pull request has been re-submitted without changes following local (forked) repo reorg.

#2479 (zvol_create_minors(): check snapdev before traversing snapshots #2479)

also related to #2388

@kernelOfTruth
Copy link
Contributor Author

I'm starting to use snapshots more and more, thus this would be good to have,

[also I'm currently updating the system kernel so taking a look at what needs to be changed with ZFS, too]

@kernelOfTruth kernelOfTruth changed the title [master re-sync, buildbot] zvol create minors check snapdev before traversing snapshots [master re-sync, buildbot] zvol create minors check snapdev before traversing snapshots #2479 Aug 6, 2015
@kernelOfTruth kernelOfTruth changed the title [master re-sync, buildbot] zvol create minors check snapdev before traversing snapshots #2479 [master re-sync of #2479, buildbot] zvol create minors check snapdev before traversing snapshots Aug 6, 2015
@behlendorf
Copy link
Contributor

@kernelOfTruth of truth could you fix the style issues in these patches and force update the patch stack. I agree, this or something like this would be good to have merged.

@kernelOfTruth kernelOfTruth force-pushed the zfs_master_30.07.2015_2479-zvol_create_minors-check-snapdev-before-traversing-snapshots_2 branch from f04207d to 89d5a97 Compare August 19, 2015 10:08
References:
openzfs#2479
openzfs@79c76d5 (Change KM_PUSHPAGE -> KM_SLEEP)

Porting notes:
@@ __zvol_create_minor(const char *name, boolean_t ignore_snapdev)

[which got changed to]

@@ __zvol_create_minor(const char *name)

doi = kmem_alloc(sizeof (dmu_object_info_t), KM_PUSHPAGE);

[replaced with]

doi = kmem_alloc(sizeof (dmu_object_info_t), KM_SLEEP);

Caused by

79c76d5
Change KM_PUSHPAGE -> KM_SLEEP
(upstream sync).

Fixing cstyle error:

"unparenthesized return expression" message in

linux 1509
@kernelOfTruth kernelOfTruth force-pushed the zfs_master_30.07.2015_2479-zvol_create_minors-check-snapdev-before-traversing-snapshots_2 branch from 89d5a97 to 54bbab4 Compare August 19, 2015 10:09
References:
openzfs#2479
openzfs@79c76d5 (Change KM_PUSHPAGE -> KM_SLEEP)

@@ __zvol_create_minor(const char *name)

    doi = kmem_alloc(sizeof (dmu_object_info_t), KM_PUSHPAGE);

    [replaced with]

    doi = kmem_alloc(sizeof (dmu_object_info_t), KM_SLEEP);

    Caused by

    79c76d5
    Change KM_PUSHPAGE -> KM_SLEEP
    (upstream sync).

Fixing cstyle errors:

space or tab at end of line

indent by spaces instead of tabs

missing space between keyword and paren

unparenthesized return expression
@kernelOfTruth kernelOfTruth force-pushed the zfs_master_30.07.2015_2479-zvol_create_minors-check-snapdev-before-traversing-snapshots_2 branch from 54bbab4 to 3c1758e Compare August 19, 2015 11:57
@kernelOfTruth
Copy link
Contributor Author

@behlendorf done :)

@behlendorf behlendorf added the Component: ZVOL ZFS Volumes label Aug 31, 2015
@kernelOfTruth
Copy link
Contributor Author

superseded by #3830 "issues #2217, #3681 - set of commits dealing with zvol__minor_() processing"

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

Successfully merging this pull request may close these issues.

3 participants