linux-kernel - Re: [RFC Aufs2 #2 21/28] aufs sysfs entries

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 17 Mar 2009 12:26:42 +0900
From:	hooanon05@...oo.co.jp
To:	Greg KH <greg@...ah.com>
Cc:	linux-kernel@...r.kernel.org, linux-fsdevel@...r.kernel.org
Subject: Re: [RFC Aufs2 #2 21/28] aufs sysfs entries 

Thank you very much for reviewing.

Greg KH:
> > +/* todo: file size may exceed PAGE_SIZE */
> > +ssize_t sysaufs_si_show(struct kobject *kobj, struct attribute *attr,
> > +			 char *buf)
> 
> NO IT CAN NOT!!!!
> 
> You are using sysfs wrong if you even think you are getting close to
> PAGE_SIZE.

The main reason I left a comment about PAGE_SIZE is that some of aufs
entry under sysfs contain the file path. And I don't think it is a good
idea to create a symlink since the path may be changed as a result of
users operation.
I left it as "todo" because such long path is rare.

If you have an idea to show users any file path which may be changed
dynamically, please let me know. At the beginning I have tried procfs,
but I was told that newer module should use sysfs instead of procfs. If
sysfs doesn't suit a path, what should I use? Debugfs?

> Please, use debugfs for this.  sysfs is a "one value per file" type
> filesystem.  You should never be using the seqfile interface for a sysfs
> file, that's a sure sign something is wrong in your design.
> 
> I think I'll stop reading now :(

I will change some entries which doesn't contain a file path (or make
them not to have have) and post again. So please don't stop reading.

J. R. Okajima
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/