On Tue, May 08, 2007 at 05:54:41PM +0200, Thomas Gleixner (tglx@xxxxxxxxxxxxx) wrote: > On Tue, 2007-05-08 at 14:46 +0200, Jan Engelhardt wrote: > > >> +static int __logfs_readdir(struct file *file, void *buf, filldir_t filldir) > > >> +{ > > >> + err = read_dir(dir, &dd, pos); > > >> + if (err == -EOF) > > >> + break; > > > > > > -EOF results in a return code 0 ? > > > > Results in a return code -256. > > Really ? It breaks out of the loop and returns 0 ! Likely it was done with intention - readdir returns 0 on EOF and NULL direntry, in Jörn's code subsequent readdir call will return EOF again and filldir callback will not be called, so NULL will be returned to userspace. -- Evgeniy Polyakov - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html