Re: [PATCH v6 1/4] sgl_alloc_order: remove 4 GiB limit, sgl_free() warning

From: Jason Gunthorpe
Date: Tue Jan 19 2021 - 08:11:23 EST


On Mon, Jan 18, 2021 at 08:27:09PM -0500, Douglas Gilbert wrote:

> To protect against the "unsigned long long" length being too big why
> not pick a large power of two and if someone can justify a larger
> value, they can send a patch.
>
> if (length > 64ULL * 1024 * 1024 * 1024)
> return NULL;

That is not how we protect against arithemetic overflows in the kernel

Jason