I believe it's spelled "howmany()."

On Thu, Jan 7, 2016 at 8:27 AM, Ravi Pokala <[email protected]> wrote:
> -----Original Message-----
>
>
> From: <[email protected]> on behalf of Jim Harris 
> <[email protected]>
> Date: 2016-01-07, Thursday at 08:18
> To: <[email protected]>, <[email protected]>, 
> <[email protected]>
> Subject: svn commit: r293328 - head/sys/dev/nvme
>
>>Author: jimharris
>>Date: Thu Jan  7 16:18:32 2016
>>New Revision: 293328
>>URL: https://svnweb.freebsd.org/changeset/base/293328
>>
>>...
>>
>>Modified: head/sys/dev/nvme/nvme_ctrlr.c
>>==============================================================================
>>--- head/sys/dev/nvme/nvme_ctrlr.c     Thu Jan  7 16:12:42 2016        
>>(r293327)
>>+++ head/sys/dev/nvme/nvme_ctrlr.c     Thu Jan  7 16:18:32 2016        
>>(r293328)
>>@@ -42,6 +42,12 @@ __FBSDID("$FreeBSD$");
>>
>> #include "nvme_private.h"
>>
>>+/*
>>+ * Used for calculating number of CPUs to assign to each core and number of 
>>I/O
>>+ *  queues to allocate per controller.
>>+ */
>>+#define NVME_CEILING(num, div)        ((((num) - 1) / (div)) + 1)
>>+
>>
>>...
>
> I'm surprised that this isn't in <sys/param.h>, along with 
> roundup()/rounddown()/etc. Finding the ceiling like this is probably pretty 
> common, so shouldn't it be added to the common header so everyone can use it?
>
> -Ravi (rpokala@)
>
>
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "[email protected]"

Reply via email to