Commit f8a101ff authored by Matthew Wilcox (Oracle)'s avatar Matthew Wilcox (Oracle) Committed by Andrew Morton
Browse files

i915: convert i915_gpu_error to use a folio_batch 

Remove one of the last remaining users of pagevec.

Link: https://lkml.kernel.org/r/20230621164557.3510324-9-willy@infradead.org


Signed-off-by: default avatarMatthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent ce064428

drivers/gpu/drm/i915/i915_gpu_error.c

+25 −25
Original line number Original line Diff line number Diff line
@@ -187,64 +187,64 @@ i915_error_printer(struct drm_i915_error_state_buf *e) 
}

}





/* single threaded page allocator with a reserved stash for emergencies */

/* single threaded page allocator with a reserved stash for emergencies */

static void pool_fini(struct pagevec *pv)

static void pool_fini(struct folio_batch *fbatch)

{

{

	pagevec_release(pv);

	folio_batch_release(fbatch);

}

}





static int pool_refill(struct pagevec *pv, gfp_t gfp)

static int pool_refill(struct folio_batch *fbatch, gfp_t gfp)

{

{

	while (pagevec_space(pv)) {

	while (folio_batch_space(fbatch)) {

		struct page *p;

		struct folio *folio;





		p = alloc_page(gfp);

		folio = folio_alloc(gfp, 0);

		if (!p)

		if (!folio)

			return -ENOMEM;

			return -ENOMEM;





		pagevec_add(pv, p);

		folio_batch_add(fbatch, folio);

	}

	}





	return 0;

	return 0;

}

}





static int pool_init(struct pagevec *pv, gfp_t gfp)

static int pool_init(struct folio_batch *fbatch, gfp_t gfp)

{

{

	int err;

	int err;





	pagevec_init(pv);

	folio_batch_init(fbatch);





	err = pool_refill(pv, gfp);

	err = pool_refill(fbatch, gfp);

	if (err)

	if (err)

		pool_fini(pv);

		pool_fini(fbatch);





	return err;

	return err;

}

}





static void *pool_alloc(struct pagevec *pv, gfp_t gfp)

static void *pool_alloc(struct folio_batch *fbatch, gfp_t gfp)

{

{

	struct page *p;

	struct folio *folio;





	p = alloc_page(gfp);

	folio = folio_alloc(gfp, 0);

	if (!p && pagevec_count(pv))

	if (!folio && folio_batch_count(fbatch))

		p = pv->pages[--pv->nr];

		folio = fbatch->folios[--fbatch->nr];





	return p ? page_address(p) : NULL;

	return folio ? folio_address(folio) : NULL;

}

}





static void pool_free(struct pagevec *pv, void *addr)

static void pool_free(struct folio_batch *fbatch, void *addr)

{

{

	struct page *p = virt_to_page(addr);

	struct folio *folio = virt_to_folio(addr);





	if (pagevec_space(pv))

	if (folio_batch_space(fbatch))

		pagevec_add(pv, p);

		folio_batch_add(fbatch, folio);

	else

	else

		__free_page(p);

		folio_put(folio);

}

}





#ifdef CONFIG_DRM_I915_COMPRESS_ERROR

#ifdef CONFIG_DRM_I915_COMPRESS_ERROR





struct i915_vma_compress {

struct i915_vma_compress {

	struct pagevec pool;

	struct folio_batch pool;

	struct z_stream_s zstream;

	struct z_stream_s zstream;

	void *tmp;

	void *tmp;

};

};
@@ -381,7 +381,7 @@ static void err_compression_marker(struct drm_i915_error_state_buf *m) 
#else

#else





struct i915_vma_compress {

struct i915_vma_compress {

	struct pagevec pool;

	struct folio_batch pool;

};

};





static bool compress_init(struct i915_vma_compress *c)

static bool compress_init(struct i915_vma_compress *c)