2017-11-10 10:33:49

by Djalal Harouni

[permalink] [raw]
Subject: Re: [PATCH RFC v3 1/7] proc: add proc_fs_info struct to store proc information

On Fri, Nov 10, 2017 at 11:26 AM, Alexey Dobriyan <[email protected]> wrote:
> On 11/9/17, Djalal Harouni <[email protected]> wrote:
>
>> +struct proc_fs_info {
>> + struct pid_namespace *pid_ns;
>> +};
>
>> +static inline struct proc_fs_info *proc_sb(struct super_block *sb)
>> +{
>> + return sb->s_fs_info;
>> +}
>
> Can you rename this to "struct proc_super_block *" then?
> That "info" suffix all over filesystems doesn't add much info itself
> just more typing.
> Ditto for "fs_info" identifiers.

Ok, will do.

>> +extern inline struct proc_fs_info *proc_sb(struct super_block *sb)
>> { return NULL;}
>
> extern inline?

Oups, sorry will fix it and try to compile without proc.

Thank you!

--
tixxdz

From 1583674520550827819@xxx Fri Nov 10 10:28:35 +0000 2017
X-GM-THRID: 1583606009222345701
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread