On 10/01/2016 05:05 AM, Artyom Tarasenko wrote:
+    uint64_t mask = 1ULL + ~(8192ULL << 3 * TTE_PGSIZE(tlb->tte));

Not the first time it has appeared in this patch series, but I find the "1 + ~" idiom to be strange. Surely plain old "-" is clearer.


r~

Reply via email to