CVS log for src/sys/dev/dm/dm_pdev.c

Up to [cvs.NetBSD.org] / src / sys / dev / dm

Request diff between arbitrary revisions

Keyword substitution: kv
Default branch: MAIN

fix typos in sys/dev/dm code comments and documentation.
also remove some trailing space in documentation.

Merge changes from current as of 20200406

dm: Fix "table" output format of dm-linear and dm-stripe

The existing "table" output showing device file path of pdev is
not compatible with dm in Linux kernel (and also DragonFlyBSD).
It should be showing "major:minor" instead.

taken-from: DragonFlyBSD

dm: Style cleanups (no functional changes)

dm: Fix memory leak in dm_pdev_rem()

Need to free dmp regardless of dm_pdev_rem() return value.
taken-from: DragonFlyBSD

Switch to vn_bdev_open* functions.

dm: Fix strange pointer declarations

Should be "type *name" or "type* name", but not "type * name".
taken-from: DragonFlyBSD

dm: Simplify list eviction code

taken-from: DragonFlyBSD

dm: Remove trailing whitespace

dm: Minor fixes/cleanups

dm: style + whitespace + indentation fixes

No functional changes.

dm: Make global dm locks static

These are all per-file locks which protect dev/pdev/target structure.
taken-from: DragonFlyBSD

dm: Include <sys/vnode.h> in dm.h

dm.h already depends on vnode, so have dm.h include <sys/vnode.h>
instead of other .c files.

taken-from: DragonFlyBSD

dm: Fix race on pdev create

List lookup and insert need to be atomic.
https://github.com/DragonFlyBSD/DragonFlyBSD/commit/ac816675c8321309b49d6a335f95c5388036803f

take-from: DragonFlyBSD

use some size_t, add whitespace between functions, rcsids.

update from HEAD

Sync with HEAD

remove checks for failure after memory allocation calls that cannot fail:

  kmem_alloc() with KM_SLEEP
  kmem_zalloc() with KM_SLEEP
  percpu_alloc()
  pserialize_create()
  psref_class_create()

all of these paths include an assertion that the allocation has not failed,
so callers should not assert that again.

sync with head

make dm aware of physical sector sizes.

For aggregates of multiple disks we use the largest sector size from
all disks. For standard power-of-2 sizes this is the same as the least
common multiple. We still require proper alignment of the targets in
the mapping table.

ok by haad@

Introduce struct pathbuf. This is an abstraction to hold a pathname
and the metadata required to interpret it. Callers of namei must now
create a pathbuf and pass it to NDINIT (instead of a string and a
uio_seg), then destroy the pathbuf after the namei session is
complete.

Update all namei call sites accordingly. Add a pathbuf(9) man page and
update namei(9).

The pathbuf interface also now appears in a couple of related
additional places that were passing string/uio_seg pairs that were
later fed into NDINIT. Update other call sites accordingly.

sync with head

Indent files remove unnecessary blank lines, white spaces and KNFize code.

Replace aprint_Verbose with aprint_debug to shutup unnecessary logs.

sync with head

Fix bug in kmem_alloc/kmem_free of params string. Params string was
allocated with length DM_MAX_PARAMS_SIZE and released with strlen + 1 size.

Disable KM_NOSLEEP allocation because we do not need them here there is
nothing critical in ioctl part of dm driver.

Bug reported by jak@.

Sync with HEAD.

Commit is split, to avoid a "too many arguments" protocol error.

sync with head.

Sync with HEAD.

file dm_pdev.c was added on branch yamt-nfs-mp on 2009-05-04 08:12:36 +0000

Ansify function definitions w/o arguments. Generated with sed.

Sync with HEAD.

Sync with HEAD.

file dm_pdev.c was added on branch nick-hppapmap on 2009-01-19 13:17:53 +0000

file dm_pdev.c was added on branch mjf-devfs2 on 2009-01-17 13:28:53 +0000

Merge the haad-dm branch to -current. This branch adds LVM functionality to
the base NetBSD system. It uses Linux LVM2 tools and our BSD licensed
device-mapper driver.

The device-mapper driver can be used to create virtual block devices which
maps virtual blocks to real with target mapping called target. Currently
these targets are available a linear, zero, error and a snapshot (this is
work in progress and doesn't work yet).

The lvm2tools adds lvm and dmsetup binary to based system, where the lvm
tool is used to manage and administer whole LVM and the dmestup is used to
communicate iwith device-mapper kernel driver. With these tools also
a libdevmapper library is instaled to the base system.

Building of tools and driver is currently disable and can be enabled with
MKLVM=yes in mk.conf. I will add sets lists and rc.d script soon.

Oked by agc@ and cube@.

Fix copyright in TNF licenses.

Use typedef in all structures in dm driver and use them in source code.
No functional change expected.

Rewrite locking in dm driver for last time. Replace rwlock with mutex/cv.
Move table lists to separate structure called table_head and access them
through dm_table interface. Thx go@, rmind@ and Dusan Bernat for help and
suggestions.

Add new dm_target_*_deps function which will get all dependiences from
selected target. It is used to get device dependencies during
dm_table_deps_ioctl. Remove dm_dev::pdevs list which was really hard to
manage and wasn't use for anything usefull.

Add new version of snapshot code for nowonlyinitial configuration of
snapshots (parsing parameters strings and opening devices was added).

Fixonebug in dm_pdev where was one SLIST_ENTRY used to for global
pdev list and for per device pdev list. This bug was hidden for a
long time because I haven't used devices with more than one pdev.

Simplify locking remove mutexes from pdev, target part of dm and only allow
one ioctl command to be in driver at time. Ioctl interface of dm device is
not performance critical so I don't need to paralelize dm driver so much.

Add new dev_type --> DM_DELETING_DEV which is set to device during
dm_device_remove_ioctl. To disable any incoming IO which will come in time
window between geting a rw_lock and removing device from global dev list.

XXX. I can't remove mutex from dm_dev.c also because dm_lookup_minor is
called from device-mapper and not from dm_ioctl.c

Add dm_pdev_lookup_name_list which is used to lookup pdev in device pdev list.

Add new reference counter to struct pdev which is used from device pdev list.
When pdev is inserted from target_init routine increment device pdev list ref_cnt
and remove pdev from list only when this ref. counter is 0.

Avoid problem when to tables with same device can remove pdev from this list.

Fix memory leak and use vn_close to close pdev vnode in dm_pdev_rem.

With this change dmsetup remove works again.

Add dm_pdev_destroy routine. Change behaviour of dm_pdev_decr to decrement
only reference counter for one dm_pdev and not for all pdevs in a list.

Change dm_pdev_rem to static because it is only used in dm.pdev.c. Add note
to comment that dm_pdev_rem must be called with held pdev mutex.

Add $NetBSD$ tag to all dm driver sources.

Fix locking issues  find by @blymn. dm_pdev_insert dmp->ref_cnt was
incremented without held mutex. Hold mutex during dm_pdev_decr run.

KNFize my sources add space after comma in function parameters.
Wrap to 80 chars per line.

Import of device-mapper driver. This driver is BSD rewrite of linux dm driver.
For now only error, linear and zero targets are supported. This driver uses NetBSD
specific ioctl protocola based on proplib.

I was able to create logical volume (with lvm2tools lvcreate utility) with this
version of driver, newfs it and mount it.

Add dmgetdefaultdisklabel to get virtual disklabel for Logical Volume device. Add snapshot targets (snapshot, snapshot-origin), add dm_type to dm_dev structure to identify type of device. e.g. we can have mirrored disk device with snapshot on them and spare disk on them. When driver want to work with snapshot devices, it looks to upcalls list and finds all DM_SNAPSHOT devices.