<feed xmlns='http://www.w3.org/2005/Atom'>
<title>netbsd/sys/dev/dm/dm_target_error.c, branch trunk</title>
<subtitle>NetBSD fork for lockdoc analysis</subtitle>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/'/>
<entry>
<title>dm: #if0 target's -&gt;upcall() handler</title>
<updated>2020-01-21T16:27:53+00:00</updated>
<author>
<name>tkusumi</name>
<email>tkusumi@NetBSD.org</email>
</author>
<published>2020-01-21T16:27:53+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=096b14febb05d34564c308e571683e5a5a225b94'/>
<id>096b14febb05d34564c308e571683e5a5a225b94</id>
<content type='text'>
This is part of NetBSD's dm design, but unimplemented
(all handlers return 0) and also unused.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This is part of NetBSD's dm design, but unimplemented
(all handlers return 0) and also unused.</pre>
</div>
</content>
</entry>
<entry>
<title>dm: Make target's -&gt;table() optional</title>
<updated>2019-12-23T16:17:35+00:00</updated>
<author>
<name>tkusumi</name>
<email>tkusumi@NetBSD.org</email>
</author>
<published>2019-12-23T16:17:35+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=1c72639d0a434ac8105c817b5167d90cdc5fbe29'/>
<id>1c72639d0a434ac8105c817b5167d90cdc5fbe29</id>
<content type='text'>
Since -&gt;info() (counter part of -&gt;table() in the original dm design
in Linux kernel in .status where both INFO and TABLE are optional)
is an optional handler, make -&gt;table() optional as well. Some
targets don't have anything to do in -&gt;table() just as in -&gt;info().

taken-from: DragonFlyBSD</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Since -&gt;info() (counter part of -&gt;table() in the original dm design
in Linux kernel in .status where both INFO and TABLE are optional)
is an optional handler, make -&gt;table() optional as well. Some
targets don't have anything to do in -&gt;table() just as in -&gt;info().

taken-from: DragonFlyBSD</pre>
</div>
</content>
</entry>
<entry>
<title>dm: Remove target's -&gt;deps() by implementing deps in dm core</title>
<updated>2019-12-21T11:59:03+00:00</updated>
<author>
<name>tkusumi</name>
<email>tkusumi@NetBSD.org</email>
</author>
<published>2019-12-21T11:59:03+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=87d1de18f98b3e78b5d086567959f7431f8fc42a'/>
<id>87d1de18f98b3e78b5d086567959f7431f8fc42a</id>
<content type='text'>
Retrieving device dependencies doesn't need to be target specific.
The reason it currently needs -&gt;deps() is because dm core doesn't
have data structure that allows table to walk through target's
underlying devices. Add struct dm_mapping to be able to do this,
and remove -&gt;deps()'s from targets which basically do the same thing.

    =====(A) before this commit
    table
        |                                     [dm core]
    -------------------------------------------------------
        |           pdev      pdev      pdev  [dm targets]
        v          ^         ^         ^
        target----/---------/---------/
        (void*)

    =====(B) this commit
    table----&gt;mapping--&gt;mapping--&gt;mapping--&gt;...
        |           |         |         |
        |           v         v         v     [dm core]
    -------------------------------------------------------
        |           pdev      pdev      pdev  [dm targets]
        v          ^         ^         ^
        target----/---------/---------/
        (void*)

taken-from: DragonFlyBSD</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Retrieving device dependencies doesn't need to be target specific.
The reason it currently needs -&gt;deps() is because dm core doesn't
have data structure that allows table to walk through target's
underlying devices. Add struct dm_mapping to be able to do this,
and remove -&gt;deps()'s from targets which basically do the same thing.

    =====(A) before this commit
    table
        |                                     [dm core]
    -------------------------------------------------------
        |           pdev      pdev      pdev  [dm targets]
        v          ^         ^         ^
        target----/---------/---------/
        (void*)

    =====(B) this commit
    table----&gt;mapping--&gt;mapping--&gt;mapping--&gt;...
        |           |         |         |
        |           v         v         v     [dm core]
    -------------------------------------------------------
        |           pdev      pdev      pdev  [dm targets]
        v          ^         ^         ^
        target----/---------/---------/
        (void*)

taken-from: DragonFlyBSD</pre>
</div>
</content>
</entry>
<entry>
<title>dm: Test # of args in target's -&gt;init()</title>
<updated>2019-12-18T14:31:35+00:00</updated>
<author>
<name>tkusumi</name>
<email>tkusumi@NetBSD.org</email>
</author>
<published>2019-12-18T14:31:35+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=fdf87adf0cac86f202f950ecd54d4364f50026b1'/>
<id>fdf87adf0cac86f202f950ecd54d4364f50026b1</id>
<content type='text'>
The # of args is part of target's spec.
Both Linux kernel and DragonFlyBSD test argc on ctr/init.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The # of args is part of target's spec.
Both Linux kernel and DragonFlyBSD test argc on ctr/init.</pre>
</div>
</content>
</entry>
<entry>
<title>dm: Enable dm-error and dm-zero target</title>
<updated>2019-12-16T14:26:23+00:00</updated>
<author>
<name>tkusumi</name>
<email>tkusumi@NetBSD.org</email>
</author>
<published>2019-12-16T14:26:23+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=f2dea84bd7e574bdeecd865e120432c33a0ecf67'/>
<id>f2dea84bd7e574bdeecd865e120432c33a0ecf67</id>
<content type='text'>
Add these two targets to dm.kmod.
These are generally available in Linux and DragonFlyBSD,
so enable them in NetBSD as well.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Add these two targets to dm.kmod.
These are generally available in Linux and DragonFlyBSD,
so enable them in NetBSD as well.</pre>
</div>
</content>
</entry>
<entry>
<title>dm: Style cleanups (no functional changes)</title>
<updated>2019-12-15T14:39:42+00:00</updated>
<author>
<name>tkusumi</name>
<email>tkusumi@NetBSD.org</email>
</author>
<published>2019-12-15T14:39:42+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=335d67065cfb38d4598e84a619006f80235e5596'/>
<id>335d67065cfb38d4598e84a619006f80235e5596</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>dm: Make targets' -&gt;sync() optional</title>
<updated>2019-12-15T10:12:45+00:00</updated>
<author>
<name>tkusumi</name>
<email>tkusumi@NetBSD.org</email>
</author>
<published>2019-12-15T10:12:45+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=7896be8aaa02cb2868056f7d9ed0dad88d9724f6'/>
<id>7896be8aaa02cb2868056f7d9ed0dad88d9724f6</id>
<content type='text'>
Apparently some targets have nothing to sync, so make it optional.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Apparently some targets have nothing to sync, so make it optional.</pre>
</div>
</content>
</entry>
<entry>
<title>dm: Make targets' -&gt;secsize() optional</title>
<updated>2019-12-15T09:42:29+00:00</updated>
<author>
<name>tkusumi</name>
<email>tkusumi@NetBSD.org</email>
</author>
<published>2019-12-15T09:42:29+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=e743ff438f64f91780a30c0b65f8fce630593334'/>
<id>e743ff438f64f91780a30c0b65f8fce630593334</id>
<content type='text'>
and make a caller assume secsize 0 if -&gt;secsize not present.
This allows a dummy function to be removed which was added in
"dm: Add dummy target -&gt;sync()/-&gt;secsize() to prevent panic on modload(8)".</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
and make a caller assume secsize 0 if -&gt;secsize not present.
This allows a dummy function to be removed which was added in
"dm: Add dummy target -&gt;sync()/-&gt;secsize() to prevent panic on modload(8)".</pre>
</div>
</content>
</entry>
<entry>
<title>dm: Rename targets' -&gt;status() to -&gt;table() given -&gt;info() exists</title>
<updated>2019-12-15T05:56:02+00:00</updated>
<author>
<name>tkusumi</name>
<email>tkusumi@NetBSD.org</email>
</author>
<published>2019-12-15T05:56:02+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=23ef6e5dd08de606f940a71b427520d20082fd32'/>
<id>23ef6e5dd08de606f940a71b427520d20082fd32</id>
<content type='text'>
Since now that dm targets in NetBSD have -&gt;info() for "status",
-&gt;status() should be renamed to -&gt;table() for "table",
given how dm target status was originally designed in Linux kernel.

taken-from: DragonFlyBSD</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Since now that dm targets in NetBSD have -&gt;info() for "status",
-&gt;status() should be renamed to -&gt;table() for "table",
given how dm target status was originally designed in Linux kernel.

taken-from: DragonFlyBSD</pre>
</div>
</content>
</entry>
<entry>
<title>dm: Remove unconditional debug prints in targets' -&gt;strategy()</title>
<updated>2019-12-14T10:02:35+00:00</updated>
<author>
<name>tkusumi</name>
<email>tkusumi@NetBSD.org</email>
</author>
<published>2019-12-14T10:02:35+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=6caba6c835bdbeb571dc27d847480163dad4b059'/>
<id>6caba6c835bdbeb571dc27d847480163dad4b059</id>
<content type='text'>
Having debug prints in -&gt;strategy() by default just to tell -&gt;strategy()
is called is overkill.
taken-from: DragonFlyBSD</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Having debug prints in -&gt;strategy() by default just to tell -&gt;strategy()
is called is overkill.
taken-from: DragonFlyBSD</pre>
</div>
</content>
</entry>
</feed>
