lkml.org 
[lkml]   [2006]   [May]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [patch 4/5] vfs: per superblock dentry stats
On Mon, May 29, David Chinner wrote:

> > -struct dentry_stat_t {
> > +struct dentry_stat {
> > int nr_dentry;
> > int nr_unused;
> > - int age_limit; /* age in seconds */
> > - int want_pages; /* pages requested by system */
> > - int dummy[2];
> > + int age_limit; /* age in seconds */
> > };
>
> that changes the size of the structure from 6*sizeof(int) to
> 3*sizeof(int)....
>
> > Index: work-2.6/kernel/sysctl.c
> > ===================================================================
> > --- work-2.6.orig/kernel/sysctl.c
> > +++ work-2.6/kernel/sysctl.c
> > @@ -958,7 +958,7 @@ static ctl_table fs_table[] = {
> > {
> > .ctl_name = FS_DENTRY,
> > .procname = "dentry-state",
> > - .data = &dentry_stat,
> > + .data = &global_dentry_stat,
> > .maxlen = 6*sizeof(int),
>
> With the above change, maxlen = 3*sizeof(int).
>
> Given that's a userspace visible change, should the above structure
> change use a "int dummy[3];" to keep the global structure and userspace
> export the same?

Hmm, probably you are right. I'll change this.

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

\
 
 \ /
  Last update: 2006-05-29 11:45    [W:0.049 / U:0.204 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site