Re: [PATCH 1/2] i2c: virtio: disable timeout handling

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

 



On 2021/10/20 13:36, Greg KH wrote:

On Wed, Oct 20, 2021 at 12:20:13PM +0800, Jie Deng wrote:
On 2021/10/20 2:14, Wolfram Sang wrote:
I think it is set to HZ currently, though I haven't tried big
transfers but I still get into some issues with Qemu based stuff.
Maybe we can bump it up to few seconds :)
If you use adapter->timeout, this can even be set at runtime using a
ioctl. So, it can adapt to use cases. Of course, the driver should
initialize it to a sane default if the automatic default (HZ) is not
suitable.

I think a big value may solve most cases. but the driver never know how big
is enough by static configuration.

Can we make this value to be configurable, just let the other side provide
this value ?
If an ioctl can change it, that would mean it is configurable, right?


Yes, but we need to know what's the best value to be configured for a specific "other side".

I think the "other side" should be more aware of what value is reasonable to be used.






[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux