* Re: [PATCH v2] xfs_io: make MADV_SOFT_OFFLINE conditional
[not found] <20240522223748.9986-1-bage@debian.org>
@ 2024-05-24 8:18 ` Christoph Hellwig
2024-05-25 11:25 ` Bastian Germann
0 siblings, 1 reply; 2+ messages in thread
From: Christoph Hellwig @ 2024-05-24 8:18 UTC (permalink / raw)
To: Bastian Germann
Cc: linux-xfs, Darrick J . Wong, Thomas Bogendoerfer, linux-mips,
linux-api
> +#ifdef MADV_SOFT_OFFLINE
Can you add a comment here that this is missing on mips as of Linux
6.9?
Also adding the mips maintainer so he can add it, as missing a madvise
random feature on a random architecture looks like a bug that should be
fixed.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH v2] xfs_io: make MADV_SOFT_OFFLINE conditional
2024-05-24 8:18 ` [PATCH v2] xfs_io: make MADV_SOFT_OFFLINE conditional Christoph Hellwig
@ 2024-05-25 11:25 ` Bastian Germann
0 siblings, 0 replies; 2+ messages in thread
From: Bastian Germann @ 2024-05-25 11:25 UTC (permalink / raw)
To: Christoph Hellwig
Cc: linux-xfs, Darrick J . Wong, Thomas Bogendoerfer, linux-mips,
linux-api
Am 24.05.24 um 10:18 schrieb Christoph Hellwig:
>> +#ifdef MADV_SOFT_OFFLINE
>
> Can you add a comment here that this is missing on mips as of Linux
> 6.9?
Sure.
> Also adding the mips maintainer so he can add it, as missing a madvise
> random feature on a random architecture looks like a bug that should be
> fixed.
There are more of this class of bugs which f82b77462b ("tools include:
Add mman macros needed by perf for all arch") presents.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2024-05-25 11:25 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20240522223748.9986-1-bage@debian.org>
2024-05-24 8:18 ` [PATCH v2] xfs_io: make MADV_SOFT_OFFLINE conditional Christoph Hellwig
2024-05-25 11:25 ` Bastian Germann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).