<feed xmlns='http://www.w3.org/2005/Atom'>
<title>netbsd/sys/dev/scsipi/ses.c, branch lockdoc-9.3</title>
<subtitle>NetBSD fork for lockdoc analysis</subtitle>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/'/>
<entry>
<title>Pull up following revision(s) (requested by riastradh in ticket #1485):</title>
<updated>2022-08-03T10:55:44+00:00</updated>
<author>
<name>martin</name>
<email>martin@NetBSD.org</email>
</author>
<published>2022-08-03T10:55:44+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=c302d5ea31d36c462babff41e49adeb9467d8bf5'/>
<id>c302d5ea31d36c462babff41e49adeb9467d8bf5</id>
<content type='text'>
	sys/dev/pci/if_iwi.c: revision 1.117
	sys/dev/raidframe/rf_netbsdkintf.c: revision 1.401
	sys/dev/scsipi/ses.c: revision 1.52
	sys/dev/isa/mcd.c: revision 1.121

sys/dev: Memset zero before copyout.

Just in case of uninitialized padding which would lead to kernel
stack disclosure.  If the compiler can prove the memset redundant
then it can optimize it away; otherwise better safe than sorry.

I think the iwi(4), mcd(4), and ses(4) changes actually plug leaks;
the raidframe(4) change probably doesn't (but doesn't hurt).</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
	sys/dev/pci/if_iwi.c: revision 1.117
	sys/dev/raidframe/rf_netbsdkintf.c: revision 1.401
	sys/dev/scsipi/ses.c: revision 1.52
	sys/dev/isa/mcd.c: revision 1.121

sys/dev: Memset zero before copyout.

Just in case of uninitialized padding which would lead to kernel
stack disclosure.  If the compiler can prove the memset redundant
then it can optimize it away; otherwise better safe than sorry.

I think the iwi(4), mcd(4), and ses(4) changes actually plug leaks;
the raidframe(4) change probably doesn't (but doesn't hurt).</pre>
</div>
</content>
</entry>
<entry>
<title> s/ are are / are /</title>
<updated>2019-03-08T08:35:58+00:00</updated>
<author>
<name>msaitoh</name>
<email>msaitoh@NetBSD.org</email>
</author>
<published>2019-03-08T08:35:58+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=3cc9fe3c529313076f82ce38237c125303253d7b'/>
<id>3cc9fe3c529313076f82ce38237c125303253d7b</id>
<content type='text'>
 s/ a a / a /</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
 s/ a a / a /</pre>
</div>
</content>
</entry>
<entry>
<title>Make scsipi framework MPSAFE.</title>
<updated>2016-11-20T15:37:19+00:00</updated>
<author>
<name>mlelstv</name>
<email>mlelstv@NetBSD.org</email>
</author>
<published>2016-11-20T15:37:19+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=ec70ca9e5859e315fce49a82349c9939d54cd1de'/>
<id>ec70ca9e5859e315fce49a82349c9939d54cd1de</id>
<content type='text'>
Data structures are now protected by a per-adapter mutex at IPL_BIO
that is created by the scsibus or atapibus instance when the adapter
is configured.
The enable reference counter and the channel freeze counter which are
currently used by HBA code before the adapter is configured, are made
atomic.
The target drivers are now all tagged as D_MPSAFE.

Almost all HBA drivers still require the kernel lock to present,
so all callbacks into HBA code are still protected by kernel lock
unless the driver is tagged as SCSIPI_ADAPT_MPSAFE.

TODO: refactor sd and cd to use dksubr.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Data structures are now protected by a per-adapter mutex at IPL_BIO
that is created by the scsibus or atapibus instance when the adapter
is configured.
The enable reference counter and the channel freeze counter which are
currently used by HBA code before the adapter is configured, are made
atomic.
The target drivers are now all tagged as D_MPSAFE.

Almost all HBA drivers still require the kernel lock to present,
so all callbacks into HBA code are still protected by kernel lock
unless the driver is tagged as SCSIPI_ADAPT_MPSAFE.

TODO: refactor sd and cd to use dksubr.</pre>
</div>
</content>
</entry>
<entry>
<title>- Use aprint*() more in xxx_attach().</title>
<updated>2016-07-14T10:19:05+00:00</updated>
<author>
<name>msaitoh</name>
<email>msaitoh@NetBSD.org</email>
</author>
<published>2016-07-14T10:19:05+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=d619634b879a03bdc3c689d78d1e713f0584225b'/>
<id>d619634b879a03bdc3c689d78d1e713f0584225b</id>
<content type='text'>
- Add missing aprint_naive("\n").
- Remove extra spaces and tabs.
- KNF.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
- Add missing aprint_naive("\n").
- Remove extra spaces and tabs.
- KNF.</pre>
</div>
</content>
</entry>
<entry>
<title>would you like some freshly ground _KERNEL_OPT with that?</title>
<updated>2015-08-24T23:13:15+00:00</updated>
<author>
<name>pooka</name>
<email>pooka@NetBSD.org</email>
</author>
<published>2015-08-24T23:13:15+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=2c98eb8ad77c9eea69cb44eb924aa661759268a7'/>
<id>2c98eb8ad77c9eea69cb44eb924aa661759268a7</id>
<content type='text'>
yes?  excellent choice, sir/madam.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
yes?  excellent choice, sir/madam.</pre>
</div>
</content>
</entry>
<entry>
<title>Add d_discard to all struct cdevsw instances I could find.</title>
<updated>2014-07-25T08:10:31+00:00</updated>
<author>
<name>dholland</name>
<email>dholland@NetBSD.org</email>
</author>
<published>2014-07-25T08:10:31+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=1a918832aa7d8748d0b53b202917f36e6949d407'/>
<id>1a918832aa7d8748d0b53b202917f36e6949d407</id>
<content type='text'>
All have been set to "nodiscard"; some should get a real implementation.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
All have been set to "nodiscard"; some should get a real implementation.</pre>
</div>
</content>
</entry>
<entry>
<title>Change (mostly mechanically) every cdevsw/bdevsw I can find to use</title>
<updated>2014-03-16T05:20:22+00:00</updated>
<author>
<name>dholland</name>
<email>dholland@NetBSD.org</email>
</author>
<published>2014-03-16T05:20:22+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=76258fa0fe68b72536361d37e7b733d8bb10b786'/>
<id>76258fa0fe68b72536361d37e7b733d8bb10b786</id>
<content type='text'>
designated initializers.

I have not built every extant kernel so I have probably broken at
least one build; however I've also found and fixed some wrong
cdevsw/bdevsw entries so even if so I think we come out ahead.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
designated initializers.

I have not built every extant kernel so I have probably broken at
least one build; however I've also found and fixed some wrong
cdevsw/bdevsw entries so even if so I think we come out ahead.</pre>
</div>
</content>
</entry>
<entry>
<title>Add detach support. From martin@.</title>
<updated>2013-04-07T18:50:33+00:00</updated>
<author>
<name>wiz</name>
<email>wiz@NetBSD.org</email>
</author>
<published>2013-04-07T18:50:33+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=5deb1dde9d52287c0127f76aeef21053b186abf6'/>
<id>5deb1dde9d52287c0127f76aeef21053b186abf6</id>
<content type='text'>
"Works for me."

Addresses PR 44283.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
"Works for me."

Addresses PR 44283.</pre>
</div>
</content>
</entry>
<entry>
<title>split device_t/softc for all remaining drivers.</title>
<updated>2012-10-27T17:17:22+00:00</updated>
<author>
<name>chs</name>
<email>chs@NetBSD.org</email>
</author>
<published>2012-10-27T17:17:22+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=5f819ca390688de9d67f1a48523a015b972c34ec'/>
<id>5f819ca390688de9d67f1a48523a015b972c34ec</id>
<content type='text'>
replace "struct device *" with "device_t".
use device_xname(), device_unit(), etc.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
replace "struct device *" with "device_t".
use device_xname(), device_unit(), etc.</pre>
</div>
</content>
</entry>
<entry>
<title>Retire varargs.h support. Move machine/stdarg.h logic into MI</title>
<updated>2011-07-17T20:54:30+00:00</updated>
<author>
<name>joerg</name>
<email>joerg@NetBSD.org</email>
</author>
<published>2011-07-17T20:54:30+00:00</published>
<link rel='alternate' type='text/html' href='http://git.infra.scholz.ruhr/netbsd/commit/?id=9f2c6cd595b50ba277a217ee608a101d3d113d35'/>
<id>9f2c6cd595b50ba277a217ee608a101d3d113d35</id>
<content type='text'>
sys/stdarg.h and expect compiler to provide proper builtins, defaulting
to the GCC interface. lint still has a special fallback.
Reduce abuse of _BSD_VA_LIST_ by defining __va_list by default and
derive va_list as required by standards.</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
sys/stdarg.h and expect compiler to provide proper builtins, defaulting
to the GCC interface. lint still has a special fallback.
Reduce abuse of _BSD_VA_LIST_ by defining __va_list by default and
derive va_list as required by standards.</pre>
</div>
</content>
</entry>
</feed>
