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  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:   Thu, 20 Aug 2020 13:17:48 +0300
From:   Igor Russkikh <>
To:     Guenter Roeck <>,
        Sebastian Andrzej Siewior <>
CC:     <>, Jakub Kicinski <>,
        "Thomas Gleixner" <>,
        Dmitry Bogdanov <>
Subject: Re: [EXT] Re: [PATCH NET] net: atlantic: Use readx_poll_timeout() for
 large timeout

>> implemented a read callback with an udelay(10000U). This fails to
>> compile on ARM because the delay is >1ms. I doubt that it is needed to
>> spin for 10ms even if possible on x86.
>> >From looking at the code, the context appears to be preemptible so
> using
>> usleep() should work and avoid busy spinning.
>> Use readx_poll_timeout() in the poll loop.
>> Cc: Mark Starovoytov <>
>> Cc: Igor Russkikh <>
>> Signed-off-by: Sebastian Andrzej Siewior <>
> Fixes: 8dcf2ad39fdb2 ("net: atlantic: add hwmon getter for MAC
> temperature")
> Acked-by: Guenter Roeck <>
> As in: This patch does not cause any additional trouble and will fix the
> observed compile failure. However, the submitter of 8dcf2ad39fdb2 might
> consider adding a mutex either into hw_atl_b0_get_mac_temp() or into
> the calling code.

Hi Sebastian, Guenter, thanks for catching and taking care of this,
Looks good for me so far.

>> Could someone with hardware please verify it? It compiles, yes.

We'll verify this on our side, sure.


Powered by blists - more mailing lists