Re: [PATCH 16/21] binder: refactor page range allocation

[Alice Ryhl]

Carlos Llamas <cmllamas@xxxxxxxxxx> writes:
> Instead of looping through the page range twice to first determine if
> the mmap lock is required, simply do it per-page as needed. Split out
> all this logic into a separate binder_get_user_page_remote() function.
>
> Signed-off-by: Carlos Llamas <cmllamas@xxxxxxxxxx>

One nit, otherwise looks okay to me.
Reviewed-by: Alice Ryhl <aliceryhl@xxxxxxxxxx>

Carlos Llamas <cmllamas@xxxxxxxxxx> writes:
> +		unsigned long index;
> -		size_t index;

You're changing the type of index to unsigned long here, but it seems to
me that size_t is the correct type.

Alice