Matthew Wilcox <willy@xxxxxxxxxxxxx> writes:
CAUTION: This email originated from outside of the
organization. Do not click links or open attachments unless you
can confirm the sender and know the content is safe.
On Sat, Jan 14, 2023 at 02:28:50PM +0200, Shay Agroskin wrote:
> memset(&pool->alloc.cache, 0, sizeof(void *) * bulk);
> nr_pages = alloc_pages_bulk_array_node(gfp, pool->p.nid,
> bulk,
> -
> pool->alloc.cache);
> + (struct page
> **)pool->alloc.cache);
Can you fix the alignment here (so that the '(struct page **)'
would align
the the 'gfp' argument one line above) ?
No, that makes the line too long.
Couldn't find any word about inlining in the coding style. AFAIK
82 characters line is considered ok (and the file you're editing
has even longer lines than that). To me it'd look better aligned,
but suit yourself