On 5/14/21 8:42 AM, Jens Axboe wrote: > On 5/14/21 5:06 AM, Pavel Begunkov wrote: >> Since recent changes instead of storing a large array of struct >> io_mapped_ubuf, we store pointers to them, that is 4 times slimmer and >> we should not to so worry about restricting max number of registererd >> buffer slots, increase the limit 4 times. > > Is this going to fall within the max kmalloc size? Ah yes, it's just a u64 now, so should be fine. Might be worth considering using vmalloc() for this in any case in the future, to make the allocation more reliable. 128K of contig memory can sometimes be an issue in prod. -- Jens Axboe