[PATCH] accel/qaic: Update MAX_ORDER use to be inclusive

Jacek Lawrynowicz jacek.lawrynowicz at linux.intel.com
Mon Nov 6 07:20:01 UTC 2023


On 03.11.2023 16:33, Jeffrey Hugo wrote:
> MAX_ORDER was redefined so that valid allocations to the page allocator
> are in the range of 0..MAX_ORDER, inclusive in the commit
> 23baf831a32c ("mm, treewide: redefine MAX_ORDER sanely").
> 
> We are treating MAX_ORDER as an exclusive value, and thus could be
> requesting larger allocations.  Update our use to match the redefinition
> of MAX_ORDER.
> 
> Signed-off-by: Jeffrey Hugo <quic_jhugo at quicinc.com>
> Reviewed-by: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy at quicinc.com>
> Reviewed-by: Carl Vanderlip <quic_carlv at quicinc.com>
> ---
>  drivers/accel/qaic/qaic_data.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/accel/qaic/qaic_data.c b/drivers/accel/qaic/qaic_data.c
> index 8da81768f2ab..8998c28e566e 100644
> --- a/drivers/accel/qaic/qaic_data.c
> +++ b/drivers/accel/qaic/qaic_data.c
> @@ -452,7 +452,7 @@ static int create_sgt(struct qaic_device *qdev, struct sg_table **sgt_out, u64 s
>  		 * later
>  		 */
>  		buf_extra = (PAGE_SIZE - size % PAGE_SIZE) % PAGE_SIZE;
> -		max_order = min(MAX_ORDER - 1, get_order(size));
> +		max_order = min(MAX_ORDER, get_order(size));
>  	} else {
>  		/* allocate a single page for book keeping */
>  		nr_pages = 1;

Reviewed-by: Jacek Lawrynowicz <jacek.lawrynowicz at linux.intel.com>


More information about the dri-devel mailing list