Re: [PATCH 1/2] tools/lib/api/fs: Add procfs int read/write helpers
From: Borislav Petkov
Date: Tue Feb 07 2017 - 05:30:13 EST
On Mon, Feb 06, 2017 at 10:43:56PM -0300, Arnaldo Carvalho de Melo wrote:
> > int sysctl__read_int(const char *sysctl, int *value);
>
> Isn't sysctl__read_int() what you want?
Right, so looking at this: don't you think that having both sysctl__*
and procfs__* is a little redundant?
The sysctl* things are doing the accesses over proc so shouldn't it all
be procfs__* interfaces and no sysctl__* ones at all
or
at least the sysctl__* ones should call the procfs__* ones?
--
Regards/Gruss,
Boris.
Good mailing practices for 400: avoid top-posting and trim the reply.