From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: blk-throttle: Move three assignments for the variable "ret" in tg_set_max() To: Johannes Thumshirn , linux-block@vger.kernel.org References: <75495d45-cfc4-9740-39e4-bd4c3e71232b@users.sourceforge.net> <20170123091510.GA3517@linux-x5ow.site> <130ce081-e9a2-4744-c74b-b73bea5ae98c@users.sourceforge.net> <20170123114609.GA8200@linux-x5ow.site> Cc: Jens Axboe , LKML , kernel-janitors@vger.kernel.org From: SF Markus Elfring Message-ID: <849a1dbc-f4c0-caf0-ad7e-706c08127c68@users.sourceforge.net> Date: Mon, 23 Jan 2017 13:06:44 +0100 MIME-Version: 1.0 In-Reply-To: <20170123114609.GA8200@linux-x5ow.site> Content-Type: text/plain; charset=windows-1252 List-ID: >> We have got different preferences for the placement of error code settings. > Yes we do, so what's the point? Both are OK. Can a function implementation be executed a bit faster in the case that error codes will usually only matter if they would be set after a concrete software failure > Please don't go down that road it opens so much potential for needless bikeshedding > and waste all of our (including your) time. I would appreciate to clarify involved run time consequences a bit more. Regards, Markus