| 19bd59e4 | 27-Nov-2016 |
Tomohiro Kusumi <kusumi.tomohiro@gmail.com> |
sys/dev/disk/dm: Make init/destroy handlers optional
Some targets don't need to do anything in init/destroy handler,
as there is no private data to init/cleanup, so make them optional.
table_en->ta
sys/dev/disk/dm: Make init/destroy handlers optional
Some targets don't need to do anything in init/destroy handler,
as there is no private data to init/cleanup, so make them optional.
table_en->target_config can be set to NULL regardless of destroy
handler or its result.
show more ...
|
| 41a68322 | 26-Nov-2016 |
Tomohiro Kusumi <kusumi.tomohiro@gmail.com> |
sys/dev/disk/dm: Remove dm_dev::dev_type
This is unused, and also not necessary.
dm core shouldn't need to be aware of target driver type.
Target dependent actions are handled by target dependent h
sys/dev/disk/dm: Remove dm_dev::dev_type
This is unused, and also not necessary.
dm core shouldn't need to be aware of target driver type.
Target dependent actions are handled by target dependent handlers.
dm targets have oop-like structure, so we don't want/need
dm core to be able to do things like below.
Also see d471f1f9 and 49784e7d.
switch (dev->dev_type) {
case DM_LINEAR_DEV:
do_something_specific_to_linear(); break;
case DM_STRIPE_DEV:
do_something_specific_to_striped(); break;
case ...:
...; break;
}
show more ...
|
| 2ca88594 | 01-Oct-2016 |
Tomohiro Kusumi <kusumi.tomohiro@gmail.com> |
sys/dev/disk/dm: Remove default:break for switch enums on kldload
This could be reverted as compilers may warn if not all enum elements
are covered, but some dm targets don't have default case on lo
sys/dev/disk/dm: Remove default:break for switch enums on kldload
This could be reverted as compilers may warn if not all enum elements
are covered, but some dm targets don't have default case on loading,
and it's been working. Compiles with LINT64.
show more ...
|