Subject: Re: mount_xxx error reporting patch
To: Konrad Schroder <perseant@hhhh.org>
From: Jason Thorpe <thorpej@nas.nasa.gov>
List: tech-userlevel
Date: 06/24/1999 13:00:39
On Thu, 24 Jun 1999 11:06:54 -0700 (PDT)
Konrad Schroder <perseant@hhhh.org> wrote:
> The "mount" family of commands (others too, but I'm not going there now)
> has inconsistent and incorrect error reporting, specifically for the case
> of mount(2) failing. Many of these commands report this in the form
>
> err(1, "%s", "");
>
> which is dead wrong; others use variations which are not so obviously
> wrong but which leave out the mount point and so IMO don't give out enough
> information. I've regularized them in the attached patch, so that they
> all use (an equivalent of):
>
> err(1,"%s on %s", dev, dir);
>
> Note that I want to do this to all of them: they should all report that
> error the same way, regardless of whether they were "broken" before or
not. (It doesn't have to be exactly this format, but they really should
> all be consistent at least.)
>
> Please let me know if you have any objections, comments, etc.
I say go ahead and commit these changes.
>
> Konrad Schroder
> perseant@hhhh.org
>
> --0-1387222599-930244416=:12056
> Content-Type: TEXT/PLAIN; CHARSET=US-ASCII; NAME="mount_xxx.diff"
> Content-Transfer-Encoding: BASE64
> Content-ID: <Pine.NEB.3.96.990624110240.12056Q@hhhh.hitl.washington.edu>
> Content-Description:
>
> SW5kZXg6IG1vdW50X2Fkb3MvbW91bnRfYWRvcy5jDQo9PT09PT09PT09PT09
> PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09DQpSQ1MgZmlsZTogL2N2c3Jvb3QvYmFzZXNyYy9zYmluL21v
> dW50X2Fkb3MvbW91bnRfYWRvcy5jLHYNCnJldHJpZXZpbmcgcmV2aXNpb24g
> MS44DQpkaWZmIC11IC1yMS44IG1vdW50X2Fkb3MuYw0KLS0tIG1vdW50X2Fk
> b3MuYwkxOTk4LzAzLzAxIDAyOjIwOjExCTEuOA0KKysrIG1vdW50X2Fkb3Mu
> YwkxOTk5LzA2LzI0IDE3OjUwOjIxDQpAQCAtMTM0LDcgKzEzNCw3IEBADQog
> CX0NCiANCiAJaWYgKG1vdW50KE1PVU5UX0FET1NGUywgZGlyLCBtbnRmbGFn
> cywgJmFyZ3MpIDwgMCkNCi0JCWVycigxLCAibW91bnQiKTsNCisJCWVycigx
> LCAiJXMgb24gJXMiLCBkZXYsIGRpcik7DQogDQogCWV4aXQgKDApOw0KIH0N
> CkluZGV4OiBtb3VudF9jZDk2NjAvbW91bnRfY2Q5NjYwLmMNCj09PT09PT09
> PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09PT09PT0NClJDUyBmaWxlOiAvY3Zzcm9vdC9iYXNlc3JjL3Ni
> aW4vbW91bnRfY2Q5NjYwL21vdW50X2NkOTY2MC5jLHYNCnJldHJpZXZpbmcg
> cmV2aXNpb24gMS43DQpkaWZmIC11IC1yMS43IG1vdW50X2NkOTY2MC5jDQot
> LS0gbW91bnRfY2Q5NjYwLmMJMTk5OS8wNS8yOCAxMDoxMDoyNQkxLjcNCisr
> KyBtb3VudF9jZDk2NjAuYwkxOTk5LzA2LzI0IDE3OjUwOjIyDQpAQCAtMTI0
> LDcgKzEyNCw3IEBADQogCWFyZ3MuZmxhZ3MgPSBvcHRzOw0KIA0KIAlpZiAo
> bW91bnQoTU9VTlRfQ0Q5NjYwLCBkaXIsIG1udGZsYWdzLCAmYXJncykgPCAw
> KQ0KLQkJZXJyKDEsICIlcyIsICIiKTsNCisJCWVycigxLCAiJXMgb24gJXMi
> LCBkZXYsIGRpcik7DQogCWV4aXQoMCk7DQogfQ0KIA0KSW5kZXg6IG1vdW50
> X2ZkZXNjL21vdW50X2ZkZXNjLmMNCj09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0N
> ClJDUyBmaWxlOiAvY3Zzcm9vdC9iYXNlc3JjL3NiaW4vbW91bnRfZmRlc2Mv
> bW91bnRfZmRlc2MuYyx2DQpyZXRyaWV2aW5nIHJldmlzaW9uIDEuOQ0KZGlm
> ZiAtdSAtcjEuOSBtb3VudF9mZGVzYy5jDQotLS0gbW91bnRfZmRlc2MuYwkx
> OTk3LzA5LzE2IDEyOjI2OjU2CTEuOQ0KKysrIG1vdW50X2ZkZXNjLmMJMTk5
> OS8wNi8yNCAxNzo1MDoyMg0KQEAgLTk0LDcgKzk0LDcgQEANCiAJCXVzYWdl
> KCk7DQogDQogCWlmIChtb3VudChNT1VOVF9GREVTQywgYXJndlsxXSwgbW50
> ZmxhZ3MsIE5VTEwpKQ0KLQkJZXJyKDEsICIlcyIsICIiKTsNCisJCWVycigx
> LCAiZmRlc2Mgb24gJXMiLCBhcmd2WzFdKTsNCiAJZXhpdCgwKTsNCiB9DQog
> DQpJbmRleDogbW91bnRfZmlsZWNvcmUvbW91bnRfZmlsZWNvcmUuYw0KPT09
> PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09PT09PT09PT09PQ0KUkNTIGZpbGU6IC9jdnNyb290L2Jhc2Vz
> cmMvc2Jpbi9tb3VudF9maWxlY29yZS9tb3VudF9maWxlY29yZS5jLHYNCnJl
> dHJpZXZpbmcgcmV2aXNpb24gMS4xDQpkaWZmIC11IC1yMS4xIG1vdW50X2Zp
> bGVjb3JlLmMNCi0tLSBtb3VudF9maWxlY29yZS5jCTE5OTgvMDgvMTQgMDM6
> Mzg6NTEJMS4xDQorKysgbW91bnRfZmlsZWNvcmUuYwkxOTk5LzA2LzI0IDE3
> OjUwOjIzDQpAQCAtMTM4LDcgKzEzOCw3IEBADQogCWFyZ3MuZmxhZ3MgPSBv
> cHRzOw0KIA0KIAlpZiAobW91bnQoTU9VTlRfRklMRUNPUkUsIGRpciwgbW50
> ZmxhZ3MsICZhcmdzKSA8IDApDQotCQllcnIoMSwgIiVzIiwgIiIpOw0KKwkJ
> ZXJyKDEsICIlcyBvbiAlcyIsIGRldiwgZGlyKTsNCiAJZXhpdCgwKTsNCiB9
> DQogDQpJbmRleDogbW91bnRfa2VybmZzL21vdW50X2tlcm5mcy5jDQo9PT09
> PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09PT09PT09PT09DQpSQ1MgZmlsZTogL2N2c3Jvb3QvYmFzZXNy
> Yy9zYmluL21vdW50X2tlcm5mcy9tb3VudF9rZXJuZnMuYyx2DQpyZXRyaWV2
> aW5nIHJldmlzaW9uIDEuMTANCmRpZmYgLXUgLXIxLjEwIG1vdW50X2tlcm5m
> cy5jDQotLS0gbW91bnRfa2VybmZzLmMJMTk5Ny8wOS8xNiAxMjoyOToxNgkx
> LjEwDQorKysgbW91bnRfa2VybmZzLmMJMTk5OS8wNi8yNCAxNzo1MDoyMw0K
> QEAgLTk0LDcgKzk0LDcgQEANCiAJCXVzYWdlKCk7DQogDQogCWlmIChtb3Vu
> dChNT1VOVF9LRVJORlMsIGFyZ3ZbMV0sIG1udGZsYWdzLCBOVUxMKSkNCi0J
> CWVycigxLCAiJXMiLCAiIik7DQorCQllcnIoMSwgImtlcm5mcyBvbiAlcyIs
> IGFyZ3ZbMV0pOw0KIAlleGl0KDApOw0KIH0NCiANCkluZGV4OiBtb3VudF9s
> ZnMvbW91bnRfbGZzLmMNCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0NClJDUyBm
> aWxlOiAvY3Zzcm9vdC9iYXNlc3JjL3NiaW4vbW91bnRfbGZzL21vdW50X2xm
> cy5jLHYNCnJldHJpZXZpbmcgcmV2aXNpb24gMS43DQpkaWZmIC11IC1yMS43
> IG1vdW50X2xmcy5jDQotLS0gbW91bnRfbGZzLmMJMTk5OC8wMy8wMSAwMjoy
> MDo0MQkxLjcNCisrKyBtb3VudF9sZnMuYwkxOTk5LzA2LzI0IDE3OjUwOjI0
> DQpAQCAtNTMsNiArNTMsNyBAQA0KICNpbmNsdWRlIDx1ZnMvdWZzL3Vmc21v
> dW50Lmg+DQogDQogI2luY2x1ZGUgPGVyci5oPg0KKyNpbmNsdWRlIDxlcnJu
> by5oPg0KICNpbmNsdWRlIDxzdGRpby5oPg0KICNpbmNsdWRlIDxzdGRsaWIu
> aD4NCiAjaW5jbHVkZSA8c3RyaW5nLmg+DQpAQCAtODEsNiArODIsNyBAQA0K
> IAlzdHJ1Y3QgdWZzX2FyZ3MgYXJnczsNCiAJaW50IGNoLCBtbnRmbGFncywg
> bm9jbGVhbjsNCiAJY2hhciAqZnNfbmFtZSwgKm9wdGlvbnM7DQorCWNvbnN0
> IGNoYXIgKmVycmNhdXNlOw0KIA0KIAlvcHRpb25zID0gTlVMTDsNCiAJbW50
> ZmxhZ3MgPSBub2NsZWFuID0gMDsNCkBAIC0xMDgsNyArMTEwLDcgQEANCiAJ
> aWYgKGFyZ2MgIT0gMikNCiAJCXVzYWdlKCk7DQogDQotICAgICAgICBhcmdz
> LmZzcGVjID0gYXJndlswXTsJLyogdGhlIG5hbWUgb2YgdGhlIGRldmljZSBm
> aWxlICovDQorCWFyZ3MuZnNwZWMgPSBhcmd2WzBdOwkvKiB0aGUgbmFtZSBv
> ZiB0aGUgZGV2aWNlIGZpbGUgKi8NCiAJZnNfbmFtZSA9IGFyZ3ZbMV07CS8q
> IHRoZSBtb3VudCBwb2ludCAqLw0KIA0KICNkZWZpbmUgREVGQVVMVF9ST09U
> VUlECS0yDQpAQCAtMTE4LDggKzEyMCwyNCBAQA0KIAllbHNlDQogCQlhcmdz
> LmV4cG9ydC5leF9mbGFncyA9IDA7DQogDQotCWlmIChtb3VudChNT1VOVF9M
> RlMsIGZzX25hbWUsIG1udGZsYWdzLCAmYXJncykpDQotCQllcnIoMSwgIiVz
> IiwgIiIpOw0KKwlpZiAobW91bnQoTU9VTlRfTEZTLCBmc19uYW1lLCBtbnRm
> bGFncywgJmFyZ3MpKSB7DQorCQlzd2l0Y2ggKGVycm5vKSB7DQorCQljYXNl
> IEVNRklMRToNCisJCQllcnJjYXVzZSA9ICJtb3VudCB0YWJsZSBmdWxsIjsN
> CisJCQlicmVhazsNCisJCWNhc2UgRUlOVkFMOg0KKwkJCWlmIChtbnRmbGFn
> cyAmIE1OVF9VUERBVEUpDQorCQkJCWVycmNhdXNlID0NCisJCQkgICAgInNw
> ZWNpZmllZCBkZXZpY2UgZG9lcyBub3QgbWF0Y2ggbW91bnRlZCBkZXZpY2Ui
> Ow0KKwkJCWVsc2UNCisJCQkJZXJyY2F1c2UgPSAiaW5jb3JyZWN0IHN1cGVy
> IGJsb2NrIjsNCisJCQlicmVhazsNCisJCWRlZmF1bHQ6DQorCQkJZXJyY2F1
> c2UgPSBzdHJlcnJvcihlcnJubyk7DQorCQkJYnJlYWs7DQorCQl9DQorCQll
> cnJ4KDEsICIlcyBvbiAlczogJXMiLCBhcmdzLmZzcGVjLCBmc19uYW1lLCBl
> cnJjYXVzZSk7DQorCX0NCiANCiAJaWYgKCFub2NsZWFuKQ0KIAkJaW52b2tl
> X2NsZWFuZXIoZnNfbmFtZSk7DQpJbmRleDogbW91bnRfbXNkb3MvbW91bnRf
> bXNkb3MuYw0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQ0KUkNTIGZpbGU6IC9j
> dnNyb290L2Jhc2VzcmMvc2Jpbi9tb3VudF9tc2Rvcy9tb3VudF9tc2Rvcy5j
> LHYNCnJldHJpZXZpbmcgcmV2aXNpb24gMS4xOQ0KZGlmZiAtdSAtcjEuMTkg
> bW91bnRfbXNkb3MuYw0KLS0tIG1vdW50X21zZG9zLmMJMTk5OC8wMy8wMSAw
> MjoyMDo0NQkxLjE5DQorKysgbW91bnRfbXNkb3MuYwkxOTk5LzA2LzI0IDE3
> OjUwOjI0DQpAQCAtMTQ2LDcgKzE0Niw3IEBADQogCX0NCiANCiAJaWYgKG1v
> dW50KE1PVU5UX01TRE9TLCBkaXIsIG1udGZsYWdzLCAmYXJncykgPCAwKQ0K
> LQkJZXJyKDEsICJtb3VudCIpOw0KKwkJZXJyKDEsICIlcyBvbiAlcyIsIGRl
> diwgZGlyKTsNCiANCiAJZXhpdCAoMCk7DQogfQ0KSW5kZXg6IG1vdW50X251
> bGwvbW91bnRfbnVsbC5jDQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09DQpSQ1Mg
> ZmlsZTogL2N2c3Jvb3QvYmFzZXNyYy9zYmluL21vdW50X251bGwvbW91bnRf
> bnVsbC5jLHYNCnJldHJpZXZpbmcgcmV2aXNpb24gMS41DQpkaWZmIC11IC1y
> MS41IG1vdW50X251bGwuYw0KLS0tIG1vdW50X251bGwuYwkxOTk3LzA5LzE2
> IDEyOjMxOjAyCTEuNQ0KKysrIG1vdW50X251bGwuYwkxOTk5LzA2LzI0IDE3
> OjUwOjI2DQpAQCAtMTA2LDcgKzEwNiw3IEBADQogCWFyZ3MudGFyZ2V0ID0g
> dGFyZ2V0Ow0KIA0KIAlpZiAobW91bnQoTU9VTlRfTlVMTCwgYXJndlsxXSwg
> bW50ZmxhZ3MsICZhcmdzKSkNCi0JCWVycigxLCAiJXMiLCAiIik7DQorCQll
> cnIoMSwgIiVzIG9uICVzIiwgdGFyZ2V0LCBhcmd2WzFdKTsNCiAJZXhpdCgw
> KTsNCiB9DQogDQpJbmRleDogbW91bnRfcHJvY2ZzL21vdW50X3Byb2Nmcy5j
> DQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09PT09PT09PT09PT09PT09DQpSQ1MgZmlsZTogL2N2c3Jvb3Qv
> YmFzZXNyYy9zYmluL21vdW50X3Byb2Nmcy9tb3VudF9wcm9jZnMuYyx2DQpy
> ZXRyaWV2aW5nIHJldmlzaW9uIDEuOQ0KZGlmZiAtdSAtcjEuOSBtb3VudF9w
> cm9jZnMuYw0KLS0tIG1vdW50X3Byb2Nmcy5jCTE5OTcvMDkvMTYgMTI6MzI6
> MDQJMS45DQorKysgbW91bnRfcHJvY2ZzLmMJMTk5OS8wNi8yNCAxNzo1MDoy
> Ng0KQEAgLTk0LDcgKzk0LDcgQEANCiAJCXVzYWdlKCk7DQogDQogCWlmICht
> b3VudChNT1VOVF9QUk9DRlMsIGFyZ3ZbMV0sIG1udGZsYWdzLCBOVUxMKSkN
> Ci0JCWVycigxLCAiJXMiLCAiIik7DQorCQllcnIoMSwgInByb2NmcyBvbiAl
> cyIsIGFyZ3ZbMV0pOw0KIAlleGl0KDApOw0KIH0NCiANCkluZGV4OiBtb3Vu
> dF91bWFwL21vdW50X3VtYXAuYw0KPT09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQ0K
> UkNTIGZpbGU6IC9jdnNyb290L2Jhc2VzcmMvc2Jpbi9tb3VudF91bWFwL21v
> dW50X3VtYXAuYyx2DQpyZXRyaWV2aW5nIHJldmlzaW9uIDEuOA0KZGlmZiAt
> dSAtcjEuOCBtb3VudF91bWFwLmMNCi0tLSBtb3VudF91bWFwLmMJMTk5OS8w
> Ni8wMSAxOToxNjo0OQkxLjgNCisrKyBtb3VudF91bWFwLmMJMTk5OS8wNi8y
> NCAxNzo1MDozMQ0KQEAgLTIzMCw3ICsyMzAsNyBAQA0KIAlhcmdzLmdtYXBk
> YXRhID0gZ21hcGRhdGE7DQogDQogCWlmIChtb3VudChNT1VOVF9VTUFQLCBh
> cmd2WzFdLCBtbnRmbGFncywgJmFyZ3MpKQ0KLQkJZXJyKDEsICIlcyIsICIi
> KTsNCisJCWVycigxLCAiJXMgb24gJXMiLCBzb3VyY2UsIGFyZ3ZbMV0pOw0K
> IAlleGl0KDApOw0KIH0NCiANCkluZGV4OiBtb3VudF91bmlvbi9tb3VudF91
> bmlvbi5jDQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
> PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09DQpSQ1MgZmlsZTogL2N2
> c3Jvb3QvYmFzZXNyYy9zYmluL21vdW50X3VuaW9uL21vdW50X3VuaW9uLmMs
> dg0KcmV0cmlldmluZyByZXZpc2lvbiAxLjUNCmRpZmYgLXUgLXIxLjUgbW91
> bnRfdW5pb24uYw0KLS0tIG1vdW50X3VuaW9uLmMJMTk5Ny8wOS8xNiAxMjoz
> Mjo1NAkxLjUNCisrKyBtb3VudF91bmlvbi5jCTE5OTkvMDYvMjQgMTc6NTA6
> MzENCkBAIC0xMTcsNyArMTE3LDcgQEANCiAJYXJncy50YXJnZXQgPSB0YXJn
> ZXQ7DQogDQogCWlmIChtb3VudChNT1VOVF9VTklPTiwgYXJndlsxXSwgbW50
> ZmxhZ3MsICZhcmdzKSkNCi0JCWVycigxLCAiJXMiLCAiIik7DQorCQllcnIo
> MSwgIiVzIG9uICVzIiwgdGFyZ2V0LCBhcmd2WzFdKTsNCiAJZXhpdCgwKTsN
> CiB9DQogDQo=
> --0-1387222599-930244416=:12056--
-- Jason R. Thorpe <thorpej@nas.nasa.gov>