mirror of
https://git.proxmox.com/git/mirror_zfs.git
synced 2024-11-18 18:31:00 +03:00
ba6a24026c
Early versions of ZFS coordinated the creation and destruction of device minors from userspace. This was inherently racy and in late 2009 these ioctl()s were removed leaving everything up to the kernel. This significantly simplified the code. However, we never picked up these changes in ZoL since we'd already significantly adjusted this code for Linux. This patch aims to rectify that by finally removing ZFC_IOC_*_MINOR ioctl()s and moving all the functionality down in to the kernel. Since this cleanup will change the kernel/user ABI it's being done in the same tag as the previous libzfs_core ABI changes. This will minimize, but not eliminate, the disruption to end users. Once merged ZoL, Illumos, and FreeBSD will basically be back in sync in regards to handling ZVOLs in the common code. While each platform must have its own custom zvol.c implemenation the interfaces provided are consistent. NOTES: 1) This patch introduces one subtle change in behavior which could not be easily avoided. Prior to this change callers of 'zfs create -V ...' were guaranteed that upon exit the /dev/zvol/ block device link would be created or an error returned. That's no longer the case. The utilities will no longer block waiting for the symlink to be created. Callers are now responsible for blocking, this is why a 'udev_wait' call was added to the 'label' function in scripts/common.sh. 2) The read-only behavior of a ZVOL now solely depends on if the ZVOL_RDONLY bit is set in zv->zv_flags. The redundant policy setting in the gendisk structure was removed. This both simplifies the code and allows us to safely leverage set_disk_ro() to issue a KOBJ_CHANGE uevent. See the comment in the code for futher details on this. 3) Because __zvol_create_minor() and zvol_alloc() may now be called in a sync task they must use KM_PUSHPAGE. References: illumos/illumos-gate@681d9761e8 Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Signed-off-by: Ned Bass <bass6@llnl.gov> Signed-off-by: Tim Chase <tim@chase2k.com> Closes #1969
55 lines
1.8 KiB
C
55 lines
1.8 KiB
C
/*
|
|
* CDDL HEADER START
|
|
*
|
|
* The contents of this file are subject to the terms of the
|
|
* Common Development and Distribution License (the "License").
|
|
* You may not use this file except in compliance with the License.
|
|
*
|
|
* You can obtain a copy of the license at usr/src/OPENSOLARIS.LICENSE
|
|
* or http://www.opensolaris.org/os/licensing.
|
|
* See the License for the specific language governing permissions
|
|
* and limitations under the License.
|
|
*
|
|
* When distributing Covered Code, include this CDDL HEADER in each
|
|
* file and include the License file at usr/src/OPENSOLARIS.LICENSE.
|
|
* If applicable, add the following below this CDDL HEADER, with the
|
|
* fields enclosed by brackets "[]" replaced with your own identifying
|
|
* information: Portions Copyright [yyyy] [name of copyright owner]
|
|
*
|
|
* CDDL HEADER END
|
|
*/
|
|
|
|
/*
|
|
* Copyright (c) 2006, 2010, Oracle and/or its affiliates. All rights reserved.
|
|
*/
|
|
|
|
#ifndef _SYS_ZVOL_H
|
|
#define _SYS_ZVOL_H
|
|
|
|
#include <sys/zfs_context.h>
|
|
|
|
#define ZVOL_OBJ 1ULL
|
|
#define ZVOL_ZAP_OBJ 2ULL
|
|
|
|
#ifdef _KERNEL
|
|
|
|
extern int zvol_check_volsize(uint64_t volsize, uint64_t blocksize);
|
|
extern int zvol_check_volblocksize(uint64_t volblocksize);
|
|
extern int zvol_get_stats(objset_t *os, nvlist_t *nv);
|
|
extern boolean_t zvol_is_zvol(const char *);
|
|
extern void zvol_create_cb(objset_t *os, void *arg, cred_t *cr, dmu_tx_t *tx);
|
|
extern int zvol_create_minor(const char *name);
|
|
extern int zvol_create_minors(const char *name);
|
|
extern int zvol_remove_minor(const char *name);
|
|
extern void zvol_remove_minors(const char *name);
|
|
extern void zvol_rename_minors(const char *oldname, const char *newname);
|
|
extern int zvol_set_volsize(const char *, uint64_t);
|
|
extern int zvol_set_volblocksize(const char *, uint64_t);
|
|
extern int zvol_set_snapdev(const char *, uint64_t);
|
|
|
|
extern int zvol_init(void);
|
|
extern void zvol_fini(void);
|
|
|
|
#endif /* _KERNEL */
|
|
#endif /* _SYS_ZVOL_H */
|