On Fri, Mar 13, 2026 at 11:23:50AM +0800, Li Wang wrote:
> On Thu, Mar 12, 2026 at 07:58:28PM -0700, Yosry Ahmed wrote:
> > > Yes, this looks more tidy than open it. I'll go this helper in v3.
> > >
> > > >
> > > > static void check_zswap_enabled(void)
> > > > {
> > > > char value;
> > >
> > > read_text() requires string but not char.
> >
> > I was just illustrating. This should be passed in as &value. That should
> > work?
>
> Yes, pass it as &value works well.
And use sizeof(&value) just get the pointer's size but not
the buffer size, so I would recommend: char value[2].
--
Regards,
Li Wang
