bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: mountlist: libmount support on Linux


From: Pádraig Brady
Subject: Re: mountlist: libmount support on Linux
Date: Fri, 29 Aug 2014 16:05:02 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2

On 08/29/2014 03:46 PM, Fridolin Pokorny wrote:
> On Fri, 2014-08-29 at 16:02 +0200, Fridolin Pokorny wrote:
>> On Wed, 2014-08-27 at 13:06 -0600, Eric Blake wrote:
>>> On 08/27/2014 10:15 AM, Pádraig Brady wrote:
>>>> On 08/27/2014 03:08 PM, Fridolin Pokorny wrote:
>>>>> diff --git a/lib/mountlist.c b/lib/mountlist.c
>>>>
>>>> Cool, this fits well.
>>>> It would be good to mention the functionality and
>>>> performance benefits in the changelog.
>>>>
>>>>> +#ifdef MOUNTED_PROC_MOUNTINFO
>>>>> +static const char *
>>>>> +mountinfo_path (void) {
>>>>> +  static char filename[sizeof ("/proc//mountinfo") + 13]; /* 13 to hold 
>>>>> a PID */
>>>
>>> The magic number 13 is gross, compared to using <intprops.h> from gnulib
>>> and using INT_STRLEN_BOUND(pid_t) (which should evaluate to 13) instead.
>>
>> The magic 13 is from the kernel, I didn't know about
>> INT_STRLEN_BOUND(), thanks for suggestion. It is far better solution.
>>
>> Anyway, it might be better to use /proc/self instead of /proc/<PID> as
>> suggested. Modified patch attached (also ChangeLog was modified).
> 
> Fixed typos, sorry for that.
> s/enviroment/environment/
> s/determinated/determined/

Looking good. I'll probably add in the m4:

if test -f /proc/self/mountinfo; then
  AC_MSG_WARN([This system supports /proc/self/mountinfo but libmount is 
missing.])
fi

thanks!
Pádraig.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]