From mboxrd@z Thu Jan 1 00:00:00 1970 From: walter harms Date: Wed, 15 Dec 2004 17:27:08 +0000 Subject: [KJ] Re: kernel-2.6.9: make sock_init() notice errors Message-Id: <41C073EC.5020202@bfs.de> MIME-Version: 1 Content-Type: multipart/mixed; boundary="===============20043682289829912==" List-Id: References: <20041215160106.GA5321@nd47.coderock.org> In-Reply-To: <20041215160106.GA5321@nd47.coderock.org> To: kernel-janitors@vger.kernel.org --===============20043682289829912== Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit Hi domen, hi list, its true that the user can not do much about missing memmory but he should know WHAT error occured. NTL not checking returncodes is an error. Continuing when one earlier function did not work is also an error and may cause problems in unexspected places. Since this is an init code its no performance consideration to be made. re, walter Domen Puncer wrote: > On 27/11/04 17:26 +0100, walter harms wrote: > >>Hi list, >>by accident i found that that sock_init never checks the return code of >>called init functions. I have no idea why this is not done so i added >>only a printk() to inform the user about the problem. >>The function returns no error indicator. i left it this way. > > > I don't like this patch: it doesn't really do anything useful... > ie. if user runs out of memory on boot, he'll have other problems > than caring about sock_init failing somewhere. > > Comments? > > Also, there's still whitespace damage like: > >>+ ret=init_inodecache(); >>+ if ( ret < 0 ) { > > > --===============20043682289829912== Content-Type: text/plain; charset="iso-8859-1" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Disposition: inline _______________________________________________ Kernel-janitors mailing list Kernel-janitors@lists.osdl.org http://lists.osdl.org/mailman/listinfo/kernel-janitors --===============20043682289829912==--