On Monday 22 May 2006 13:15, you wrote:
> On Mon, 22 May 2006 02:27:09 PDT, Andrew Morton said:
> > ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.17-rc4/2.6.17-rc4-mm3/
>
> Mostly works, am chasing down 3 small things (all 3 were new as of -mm2 - I was
> busy chasing them when -mm3 showed up).
>
> 1) Something has gone astray in the hardware RNG rework. /sbin/rngd was
> quite happy dealing with the i810 RNG in my laptop under -rc4-mm1, but under
> -mm2 and -mm3, I get this (from strace /bin/rngd):
>
> open("/dev/hw_random", O_RDONLY) = 3
> read(3, "q\252cg", 4) = 4
> read(3, 0xbfaf56ac, 4) = -1 EAGAIN (Resource temporarily unavai lable)
>
> It works for some number of reads, but eventually pulls an EAGAIN. When
> stracing on a console that was slow-to-scroll, it did several dozen before
> failing - so it's apparently a timing thing. I stuck a debugging printk
> in just before the test that returns EAGAIN, and got this:
>
> [ 68.361000] rng_read data_present=1 i=0 bytes_read=1
> [ 68.361000] rng_read data_present=1 i=0 bytes_read=1
> [ 68.361000] rng_read data_present=1 i=0 bytes_read=1
> [ 68.361000] rng_read data_present=1 i=0 bytes_read=1
> [ 68.361000] rng_read data_present=0 i=20 bytes_read=0
>
> It looks to me line the old code stayed in a while() loop in rng_dev_read
> until it had fulfilled the read request (including possibly multiple
> calls to need_resched() and friends). The new code will bail on an -EAGAIN
> as soon as the *first* poll fails, rather than waiting until something
> is available - even if it is NOT flagged O_NONBLOCK.
Yeah. That is how it works. I am wondering why userspace doesn't
simply retry, if it receives an EAGAIN.
Should we return ERESTARTSYS or something like that instead?
-
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]
