Re: [PATCH v2 11/13] s390/qdio_setup: Use kzalloc() for the storage list

[Heiko Carstens]

On Thu, Sep 10, 2026 at 02:00:13PM +0300, Mike Rapoport (Microsoft) wrote:
> __qdio_allocate_qs() allocates the storage list information block of a
> queue.
> 
> This memory can be allocated with kmalloc() as there's nothing special
> about it to go directly to the page allocator.
...
> @@ -109,7 +109,7 @@ static int __qdio_allocate_qs(struct qdio_q **irq_ptr_qs, int nr_queues)
>  			return -ENOMEM;
>  		}
>  
> -		q->sl_page = (void *)__get_free_page(GFP_KERNEL);
> +		q->sl_page = kzalloc(PAGE_SIZE, GFP_KERNEL);
>  		if (!q->sl_page) {

Speaking about unnecessary changes: why is this a conversion to
kzalloc() instead of kmalloc()?