From mboxrd@z Thu Jan 1 00:00:00 1970 From: SF Markus Elfring Date: Sat, 29 Apr 2017 08:30:29 +0000 Subject: Re: SATA: Fine-tuning for two function implementations Message-Id: List-Id: References: <20170428215334.GH22354@htj.duckdns.org> In-Reply-To: <20170428215334.GH22354@htj.duckdns.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: Tejun Heo Cc: linux-ide@vger.kernel.org, Hans de Goede , LKML , kernel-janitors@vger.kernel.org > Hmmm, allocs -> callocs. Are these actually beneficial? If so, why? > Because one multiplication is rolled into the call? Did the previous size calculations contain the general possibility for integer overflows? https://cwe.mitre.org/data/definitions/190.html * Will the computed values usually stay within the limits of the used data types so far? * How much do you care for corresponding checks and source code annotations by functions like “devm_kcalloc”? Regards, Markus