vm_lowmem: fix signature mismatch for lowmem event dispatcher

[aokblast]

Use wrapper instead of overwrite

[jhb]

The _1 and _2 parameter names are not typical in FreeBSD. I would suggest sticking with void *arg for the first argument and int flags for the second argument.

sys/ufs/ufs/ufs_dirhash.c

101 ↗

(On Diff #151386)

[glebius]

IMHO, better to use meaningful names for arguments even though they aren't used:

foo_lowmem(void *ctx __unused, int flags __unused)

[aokblast]

Fix naming convension for argument

[glebius]

Well, worse than argument arg is only variable var :) But since John suggested that, let it be so.

[jhb]

Heh, ctx would be ok as well. void *arg is a pretty common idiom in FreeBSD for callback arguments where the entity registering a callback gets to pass through an opaque argument to the callback (which is what the first arguments to event handlers are).

[aokblast]

I use arg because linuxkpi_vm_lowmem and ttm_pool_mm_shrink use arg.

Here is the commit correspond to this differential revision. Please help me on merging. Thanks:)

https://github.com/aokblast/freebsd-src/commit/5137053014715094a6a8e871ddeb1e4a1fda3546

[jhb]

FYI, I split out the geli fix into a separate commit from the vm_lowmem fixes and tweaked the commit message slightly.