Re: [patch] inotify, improved.

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

 



On Fri, Jun 17, 2005 at 10:56:17AM -0700, Zach Brown wrote:
> John McCutchan wrote:
> 
> > I really don't like sending partial events. 
> 
> Partial reads, not partial events.  Just like the previous code, it
> returns to userspace after copying the events it found in the list
> before it went empty.  It doesn't go back to sleep to fill the rest of
> the buffer like a more classical blocking read() method would.
> 
> Where it differs is in what happens if you get errors copying events
> after having successfully copied some.  The previous code would return
> the error, that patch would return the bytes used by the good events.

My bad. Shouldn't we return the error code to user space though? We
shouldn't be hiding errors in the app. How does read() handle an error
part way through a read?

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

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux