Re: bin/43587: stupid error message during boot from sysctl if no COMPAT40 in kernel

[Wolfgang Stukenbrock <wolfgang.stukenbrock%nagler-company.com@localhost>]

The following reply was made to PR bin/43587; it has been noted by GNATS.

From: Wolfgang Stukenbrock <wolfgang.stukenbrock%nagler-company.com@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: gnats-admin%NetBSD.org@localhost, netbsd-bugs%NetBSD.org@localhost
Subject: Re: bin/43587: stupid error message during boot from sysctl if no 
COMPAT40 in kernel
Date: Tue, 28 May 2013 10:56:26 +0200

 Hi - it took some time to get here again for me ...
 
 I'm just preparing the upgrade to 6.1 and had a look at this again.
 
 Your patch to kill the error output for optional settings sounds like 
 the better aproach as reading and check, if already set.
 There will be less side effects now and in the future.
 
 But I think only the EPERM case printing should be avoided.
 (Accedently EPERM can happen in three cases - according to the manual, 
 but this is still better than suppressing all other kind of errors too.)
 
 So useing
    if (!opt || errno != EPERM)
 should be better than just
    if (!opt)
 in both cases.
 
 
 
 Best regards.
 
 W. Stukenbrock