2023-11-15 15:31:46

by Mariusz Tkaczyk

[permalink] [raw]
Subject: Re: [PATCH] kernfs: support kernfs notify in memory recliam context

On Tue, 14 Nov 2023 15:53:47 -0800
[email protected] wrote:

> Understood, thanks. Sound like depending on Userspace on memory reclaim
> path is really bad idea and the only option for fixing it is to remove
> that dependency, but i am not sure that is possible without breaking the
> consistency of metadata.
>
> Thanks,
>
> Junxiao.

Indeed the project of external metadata management if fragile. You cares about
IMSM here (same as me) so ideally we should implement metadata management in
kernel- I think that IMSM deserved that after 10 years on the market. There is
no better option, other options are just "workarounds" for the lack of metadata
management in kernel.

Song, any comments here?

From the second hand, there is native raid which should just work, so maybe you
can switch to the native raid?

Thanks,
Mariusz


2023-11-16 17:05:18

by Junxiao Bi

[permalink] [raw]
Subject: Re: [PATCH] kernfs: support kernfs notify in memory recliam context

On 11/15/23 7:30 AM, Mariusz Tkaczyk wrote:

> On Tue, 14 Nov 2023 15:53:47 -0800
> [email protected] wrote:
>
>> Understood, thanks. Sound like depending on Userspace on memory reclaim
>> path is really bad idea and the only option for fixing it is to remove
>> that dependency, but i am not sure that is possible without breaking the
>> consistency of metadata.
>>
>> Thanks,
>>
>> Junxiao.
> Indeed the project of external metadata management if fragile. You cares about
> IMSM here (same as me) so ideally we should implement metadata management in
> kernel- I think that IMSM deserved that after 10 years on the market. There is
> no better option, other options are just "workarounds" for the lack of metadata
> management in kernel.
Agree, sound like that's the way to proceed.
>
> Song, any comments here?
>
> From the second hand, there is native raid which should just work, so maybe you
> can switch to the native raid?

Unfortunately that's is not possible, it's a production setup to use
imsm raid.

Thanks,

Junxiao.

>
> Thanks,
> Mariusz

2023-11-17 08:37:06

by Mariusz Tkaczyk

[permalink] [raw]
Subject: Re: [PATCH] kernfs: support kernfs notify in memory recliam context

On Thu, 16 Nov 2023 09:04:34 -0800
[email protected] wrote:

> On 11/15/23 7:30 AM, Mariusz Tkaczyk wrote:
>
> > On Tue, 14 Nov 2023 15:53:47 -0800
> > [email protected] wrote:
> >
> >> Understood, thanks. Sound like depending on Userspace on memory reclaim
> >> path is really bad idea and the only option for fixing it is to remove
> >> that dependency, but i am not sure that is possible without breaking the
> >> consistency of metadata.
> >>
> >> Thanks,
> >>
> >> Junxiao.
> > Indeed the project of external metadata management if fragile. You cares
> > about IMSM here (same as me) so ideally we should implement metadata
> > management in kernel- I think that IMSM deserved that after 10 years on the
> > market. There is no better option, other options are just "workarounds" for
> > the lack of metadata management in kernel.
> Agree, sound like that's the way to proceed.
> >
> > Song, any comments here?
> >
> > From the second hand, there is native raid which should just work, so
> > maybe you can switch to the native raid?
>
> Unfortunately that's is not possible, it's a production setup to use
> imsm raid.

Implementing IMSM in kernel is a goal for months/ years so I don't
see it in a timeline you would need that on to fix your production setup. It
will be a big feature, but let's wait for Song voice first.

Thanks,
Mariusz