mirror of
https://github.com/freebsd/freebsd-src
synced 2024-09-06 17:18:32 +00:00
![Alexander Motin](/assets/img/avatar_default.png)
Before this patch CAM periph drivers called both disk_alloc() and disk_create() same time on periph creation. But then prevented disks from opening until the periph probe completion with cam_periph_hold(). As result, especially if disk misbehaves during the probe, GEOM event thread, triggered to taste the disk, got blocked on open attempt, potentially for a long time, unable to process other events. This patch moves disk_create() call from periph creation to the end of the probe. To allow disk_create() calls from non-sleepable CAM contexts some of its duties requiring memory allocations are moved either back to disk_alloc() or forward to g_disk_create(), so now disk_alloc() and disk_add_alias() are the only disk methods that require sleeping. If disk fails during the probe disk_create() may just be skipped, going directly to disk_destroy(). Other method calls during that time are just ignored. Since GEOM may now see the disks after CAM bus scan is already completed, introduce per-periph boot hold functions. Enclosure driver already had such mechanism, so just generalize it. Reviewed by: imp MFC after: 1 month Sponsored by: iXsystems, Inc. Differential Revision: https://reviews.freebsd.org/D35784
163 lines
4.9 KiB
C
163 lines
4.9 KiB
C
/*-
|
|
* SPDX-License-Identifier: BSD-3-Clause
|
|
*
|
|
* Copyright (c) 2003 Poul-Henning Kamp
|
|
* All rights reserved.
|
|
*
|
|
* This software was developed for the FreeBSD Project by Poul-Henning Kamp
|
|
* and NAI Labs, the Security Research Division of Network Associates, Inc.
|
|
* under DARPA/SPAWAR contract N66001-01-C-8035 ("CBOSS"), as part of the
|
|
* DARPA CHATS research program.
|
|
*
|
|
* Redistribution and use in source and binary forms, with or without
|
|
* modification, are permitted provided that the following conditions
|
|
* are met:
|
|
* 1. Redistributions of source code must retain the above copyright
|
|
* notice, this list of conditions and the following disclaimer.
|
|
* 2. Redistributions in binary form must reproduce the above copyright
|
|
* notice, this list of conditions and the following disclaimer in the
|
|
* documentation and/or other materials provided with the distribution.
|
|
* 3. The names of the authors may not be used to endorse or promote
|
|
* products derived from this software without specific prior written
|
|
* permission.
|
|
*
|
|
* THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
|
|
* ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
|
|
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
|
|
* ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
|
|
* FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
|
|
* DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
|
|
* OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
|
|
* HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
|
|
* LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
|
|
* OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
|
|
* SUCH DAMAGE.
|
|
*
|
|
* $FreeBSD$
|
|
*/
|
|
|
|
#ifndef _GEOM_GEOM_DISK_H_
|
|
#define _GEOM_GEOM_DISK_H_
|
|
|
|
#define DISK_RR_UNKNOWN 0
|
|
#define DISK_RR_NON_ROTATING 1
|
|
#define DISK_RR_MIN 0x0401
|
|
#define DISK_RR_MAX 0xfffe
|
|
|
|
#ifdef _KERNEL
|
|
|
|
#include <sys/queue.h>
|
|
#include <sys/_lock.h>
|
|
#include <sys/_mutex.h>
|
|
#include <sys/disk.h>
|
|
|
|
#define G_DISK_CLASS_NAME "DISK"
|
|
|
|
struct disk;
|
|
|
|
typedef int disk_open_t(struct disk *);
|
|
typedef int disk_close_t(struct disk *);
|
|
typedef void disk_strategy_t(struct bio *bp);
|
|
typedef int disk_getattr_t(struct bio *bp);
|
|
typedef void disk_gone_t(struct disk *);
|
|
typedef int disk_ioctl_t(struct disk *, u_long cmd, void *data,
|
|
int fflag, struct thread *td);
|
|
/* NB: disk_ioctl_t SHALL be cast'able to d_ioctl_t */
|
|
|
|
struct g_geom;
|
|
struct devstat;
|
|
|
|
typedef enum {
|
|
DISK_INIT_NONE,
|
|
DISK_INIT_CREATE,
|
|
DISK_INIT_START,
|
|
DISK_INIT_DONE
|
|
} disk_init_level;
|
|
|
|
struct disk_alias {
|
|
LIST_ENTRY(disk_alias) da_next;
|
|
const char *da_alias;
|
|
};
|
|
|
|
struct disk {
|
|
/* Fields which are private to geom_disk */
|
|
struct g_geom *d_geom;
|
|
struct devstat *d_devstat;
|
|
int d_goneflag;
|
|
int d_destroyed;
|
|
disk_init_level d_init_level;
|
|
|
|
/* Shared fields */
|
|
u_int d_flags;
|
|
const char *d_name;
|
|
u_int d_unit;
|
|
struct bio_queue_head *d_queue;
|
|
struct mtx *d_lock;
|
|
|
|
/* Disk methods */
|
|
disk_open_t *d_open;
|
|
disk_close_t *d_close;
|
|
disk_strategy_t *d_strategy;
|
|
disk_ioctl_t *d_ioctl;
|
|
dumper_t *d_dump;
|
|
disk_getattr_t *d_getattr;
|
|
disk_gone_t *d_gone;
|
|
|
|
/* Info fields from driver to geom_disk.c. Valid when open */
|
|
u_int d_sectorsize;
|
|
off_t d_mediasize;
|
|
u_int d_fwsectors;
|
|
u_int d_fwheads;
|
|
u_int d_maxsize;
|
|
off_t d_delmaxsize;
|
|
off_t d_stripeoffset;
|
|
off_t d_stripesize;
|
|
char d_ident[DISK_IDENT_SIZE];
|
|
char d_descr[DISK_IDENT_SIZE];
|
|
uint16_t d_hba_vendor;
|
|
uint16_t d_hba_device;
|
|
uint16_t d_hba_subvendor;
|
|
uint16_t d_hba_subdevice;
|
|
uint16_t d_rotation_rate;
|
|
char d_attachment[DISK_IDENT_SIZE];
|
|
|
|
/* Fields private to the driver */
|
|
void *d_drv1;
|
|
|
|
/* Fields private to geom_disk, to be moved on next version bump */
|
|
LIST_HEAD(,disk_alias) d_aliases;
|
|
struct g_event *d_cevent;
|
|
struct g_event *d_devent;
|
|
};
|
|
|
|
#define DISKFLAG_RESERVED 0x0001 /* Was NEEDSGIANT */
|
|
#define DISKFLAG_OPEN 0x0002
|
|
#define DISKFLAG_CANDELETE 0x0004
|
|
#define DISKFLAG_CANFLUSHCACHE 0x0008
|
|
#define DISKFLAG_UNMAPPED_BIO 0x0010
|
|
#define DISKFLAG_DIRECT_COMPLETION 0x0020
|
|
#define DISKFLAG_CANZONE 0x0080
|
|
#define DISKFLAG_WRITE_PROTECT 0x0100
|
|
|
|
struct disk *disk_alloc(void);
|
|
void disk_create(struct disk *disk, int version);
|
|
void disk_destroy(struct disk *disk);
|
|
void disk_gone(struct disk *disk);
|
|
void disk_attr_changed(struct disk *dp, const char *attr, int flag);
|
|
void disk_media_changed(struct disk *dp, int flag);
|
|
void disk_media_gone(struct disk *dp, int flag);
|
|
int disk_resize(struct disk *dp, int flag);
|
|
void disk_add_alias(struct disk *disk, const char *);
|
|
|
|
#define DISK_VERSION_00 0x58561059
|
|
#define DISK_VERSION_01 0x5856105a
|
|
#define DISK_VERSION_02 0x5856105b
|
|
#define DISK_VERSION_03 0x5856105c
|
|
#define DISK_VERSION_04 0x5856105d
|
|
#define DISK_VERSION_05 0x5856105e
|
|
#define DISK_VERSION_06 0x5856105f
|
|
#define DISK_VERSION DISK_VERSION_06
|
|
|
|
#endif /* _KERNEL */
|
|
#endif /* _GEOM_GEOM_DISK_H_ */
|