On 7/16/20 19:32, Keith Busch wrote: >> + const int last_prp = (NVME_CTRL_PAGE_SIZE >> ilog2(sizeof(__le64))) - 1; > This is an integer constant expression, so the result is already > evaluated at compile time. > Okay, we can ignore this patch then.