Re: [PATCH v1 02/16] mm/memory: remove "zap_details" parameter from zap_page_range_single()

On 3/2/26 11:01, Alice Ryhl wrote:

On Mon, Mar 02, 2026 at 09:18:45AM +0100, David Hildenbrand (Arm) wrote:

quoted

On 2/28/26 13:38, Alice Ryhl wrote:

quoted

Please run rustfmt on Rust changes. Here, rustfmt leads to this being
formatted on a single line:

Having to run tooling I don't even have installed when removing a single
function parameter; did not expect that :)

Well, rustfmt comes with the compiler, and it would be ideal to build
test changes before sending them :)

At least on Ubuntu on my notebook where I do most of the coding+patch
submissions it's a separate package?

I do all my builds on a different (more powerful) machine where the
whole rust machinery's in place. Further, build bots that run on my
private branches did not report any issues.

But no worries, I took care of testing it. Thanks for taking the time to
update the Rust code as well.

I just did an allyesconfig and it does not report any warnings.

So apparently, rustfmt problems not result in the compiler complaining?

Or something else is off here that rust/kernel/mm/virt.rs won't get
compiled on my machine, even with allyesconfig. I can definitely see
some RUSTC stuff happening in the logs, like

	RUSTC L rust/kernel.o

Thanks for the review and for pointing out rustfmt!

Documentation/rust/quick-start.rst has details for most distros.

Will read that the next time I feel brave enough to touch rust code.

-- 
Cheers,

David