Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 26 Apr 2008 01:19:01 -0400
From:      Coleman Kane <cokane@FreeBSD.org>
To:        freebsd-stable@freebsd.org
Cc:        imp@FreeBSD.org, rwatson@FreeBSD.org
Subject:   MFC Candidate: convert ffs_softdep.c over to callout(9)
Message-ID:  <1209187141.2508.6.camel@localhost>

next in thread | raw e-mail | index | archive | help

--=-gR8tcWH0GcBt/Ajzq+Ao
Content-Type: multipart/mixed; boundary="=-DuJHNqKHGzeS18peO88Q"


--=-DuJHNqKHGzeS18peO88Q
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable

Hi stable users,

Recently I committed revision 1.219 of src/sys/ufs/ffs/ffs_softdep.c to
remove the use of old-style timeout(9) calls in the softdep code and
replace them with the new callout(9) API, to further MPSAFE-ness of the
softdep code.

I am attaching a patch to RELENG_7 that I'd like some people to test,
which is just an MFC of this code. Can I get any testers (so that I can
MFC this)?

--=20
Coleman Kane

--=-DuJHNqKHGzeS18peO88Q
Content-Disposition: attachment;
	filename=ffs_softdep.c-newcallout-releng_7.diff
Content-Transfer-Encoding: base64
Content-Type: text/x-patch; name=ffs_softdep.c-newcallout-releng_7.diff;
	charset=UTF-8

ZGlmZiAtLWdpdCBhL3N5cy91ZnMvZmZzL2Zmc19zb2Z0ZGVwLmMgYi9zeXMvdWZzL2Zmcy9mZnNf
c29mdGRlcC5jDQppbmRleCA2NjhmMDhmLi5mMWIwMWQyIDEwMDY0NA0KLS0tIHN5cy91ZnMvZmZz
L2Zmc19zb2Z0ZGVwLmMNCisrKyBzeXMvdWZzL2Zmcy9mZnNfc29mdGRlcC5jDQpAQCAtNjYxLDcg
KzY2MSw3IEBAIHN0YXRpYyBpbnQgbWF4aW5kaXJkZXBzID0gNTA7CS8qIG1heCBudW1iZXIgb2Yg
aW5kaXJkZXBzIGJlZm9yZSBzbG93ZG93biAqLw0KIHN0YXRpYyBpbnQgdGlja2RlbGF5ID0gMjsJ
LyogbnVtYmVyIG9mIHRpY2tzIHRvIHBhdXNlIGR1cmluZyBzbG93ZG93biAqLw0KIHN0YXRpYyBp
bnQgcHJvY193YWl0aW5nOwkvKiB0cmFja3Mgd2hldGhlciB3ZSBoYXZlIGEgdGltZW91dCBwb3N0
ZWQgKi8NCiBzdGF0aWMgaW50ICpzdGF0X2NvdW50cDsJLyogc3RhdGlzdGljIHRvIGNvdW50IGlu
IHByb2Nfd2FpdGluZyB0aW1lb3V0ICovDQotc3RhdGljIHN0cnVjdCBjYWxsb3V0X2hhbmRsZSBo
YW5kbGU7IC8qIGhhbmRsZSBvbiBwb3N0ZWQgcHJvY193YWl0aW5nIHRpbWVvdXQgKi8NCitzdGF0
aWMgc3RydWN0IGNhbGxvdXQgc29mdGRlcF9jYWxsb3V0Ow0KIHN0YXRpYyBpbnQgcmVxX3BlbmRp
bmc7DQogc3RhdGljIGludCByZXFfY2xlYXJfaW5vZGVkZXBzOwkvKiBzeW5jZXIgcHJvY2VzcyBm
bHVzaCBzb21lIGlub2RlZGVwcyAqLw0KICNkZWZpbmUgRkxVU0hfSU5PREVTCQkxDQpAQCAtMTM5
Miw2ICsxMzkyLDkgQEAgc29mdGRlcF9pbml0aWFsaXplKCkNCiAJYmlvb3BzLmlvX2NvbXBsZXRl
ID0gc29mdGRlcF9kaXNrX3dyaXRlX2NvbXBsZXRlOw0KIAliaW9vcHMuaW9fZGVhbGxvY2F0ZSA9
IHNvZnRkZXBfZGVhbGxvY2F0ZV9kZXBlbmRlbmNpZXM7DQogCWJpb29wcy5pb19jb3VudGRlcHMg
PSBzb2Z0ZGVwX2NvdW50X2RlcGVuZGVuY2llczsNCisNCisJLyogSW5pdGlhbGl6ZSB0aGUgY2Fs
bG91dCB3aXRoIGFuIG10eC4gKi8NCisJY2FsbG91dF9pbml0X210eCgmc29mdGRlcF9jYWxsb3V0
LCAmbGssIDApOw0KIH0NCiANCiAvKg0KQEAgLTE0MDIsNiArMTQwNSw3IEBAIHZvaWQNCiBzb2Z0
ZGVwX3VuaW5pdGlhbGl6ZSgpDQogew0KIA0KKwljYWxsb3V0X2RyYWluKCZzb2Z0ZGVwX2NhbGxv
dXQpOw0KIAloYXNoZGVzdHJveShwYWdlZGVwX2hhc2h0YmwsIE1fUEFHRURFUCwgcGFnZWRlcF9o
YXNoKTsNCiAJaGFzaGRlc3Ryb3koaW5vZGVkZXBfaGFzaHRibCwgTV9JTk9ERURFUCwgaW5vZGVk
ZXBfaGFzaCk7DQogCWhhc2hkZXN0cm95KG5ld2Jsa19oYXNodGJsLCBNX05FV0JMSywgbmV3Ymxr
X2hhc2gpOw0KQEAgLTU4NTYsOCArNTg2MCwxMCBAQCByZXF1ZXN0X2NsZWFudXAobXAsIHJlc291
cmNlKQ0KIAkgKiBXZSB3YWl0IGF0IG1vc3QgdGlja2RlbGF5IGJlZm9yZSBwcm9jZWVkaW5nIGlu
IGFueSBjYXNlLg0KIAkgKi8NCiAJcHJvY193YWl0aW5nICs9IDE7DQotCWlmIChoYW5kbGUuY2Fs
bG91dCA9PSBOVUxMKQ0KLQkJaGFuZGxlID0gdGltZW91dChwYXVzZV90aW1lciwgMCwgdGlja2Rl
bGF5ID4gMiA/IHRpY2tkZWxheSA6IDIpOw0KKwlpZiAoY2FsbG91dF9wZW5kaW5nKCZzb2Z0ZGVw
X2NhbGxvdXQpID09IEZBTFNFKQ0KKwkJY2FsbG91dF9yZXNldCgmc29mdGRlcF9jYWxsb3V0LCB0
aWNrZGVsYXkgPiAyID8gdGlja2RlbGF5IDogMiwNCisJCSAgICBwYXVzZV90aW1lciwgMCk7DQor
DQogCW1zbGVlcCgoY2FkZHJfdCkmcHJvY193YWl0aW5nLCAmbGssIFBQQVVTRSwgInNvZnR1cGRh
dGUiLCAwKTsNCiAJcHJvY193YWl0aW5nIC09IDE7DQogCXJldHVybiAoMSk7DQpAQCAtNTg3Miwx
NCArNTg3OCwxNSBAQCBwYXVzZV90aW1lcihhcmcpDQogCXZvaWQgKmFyZzsNCiB7DQogDQotCUFD
UVVJUkVfTE9DSygmbGspOw0KKwkvKg0KKwkgKiBUaGUgY2FsbG91dF8gQVBJIGhhcyBhY3F1aXJl
ZCBtdHggYW5kIHdpbGwgaG9sZCBpdCBhcm91bmQgdGhpcw0KKwkgKiBmdW5jdGlvbiBjYWxsLg0K
KwkgKi8NCiAJKnN0YXRfY291bnRwICs9IDE7DQogCXdha2V1cF9vbmUoJnByb2Nfd2FpdGluZyk7
DQogCWlmIChwcm9jX3dhaXRpbmcgPiAwKQ0KLQkJaGFuZGxlID0gdGltZW91dChwYXVzZV90aW1l
ciwgMCwgdGlja2RlbGF5ID4gMiA/IHRpY2tkZWxheSA6IDIpOw0KLQllbHNlDQotCQloYW5kbGUu
Y2FsbG91dCA9IE5VTEw7DQotCUZSRUVfTE9DSygmbGspOw0KKwkJY2FsbG91dF9yZXNldCgmc29m
dGRlcF9jYWxsb3V0LCB0aWNrZGVsYXkgPiAyID8gdGlja2RlbGF5IDogMiwNCisJCSAgICBwYXVz
ZV90aW1lciwgMCk7DQogfQ0KIA0KIC8qDQo=


--=-DuJHNqKHGzeS18peO88Q--

--=-gR8tcWH0GcBt/Ajzq+Ao
Content-Type: application/pgp-signature; name=signature.asc
Content-Description: This is a digitally signed message part

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.8 (FreeBSD)

iEYEABECAAYFAkgSuz8ACgkQcMSxQcXat5c7YQCaAygTUuRo5l8ByaYYZx1SiZ6/
LMIAn2usEmKsZrF769K8S0WB4bcC1jww
=7jVk
-----END PGP SIGNATURE-----

--=-gR8tcWH0GcBt/Ajzq+Ao--




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1209187141.2508.6.camel>