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

2021-11-23 Thread Viresh Kumar
On 23-11-21, 10:52, Wolfram Sang wrote: > On Thu, Nov 11, 2021 at 05:04:11PM +0100, Vincent Whitchurch wrote: > > If a timeout is hit, it can result is incorrect data on the I2C bus > > and/or memory corruptions in the guest since the device can still be > > operating on the buffers it was given wh

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

2021-11-19 Thread Vincent Whitchurch
On Fri, Nov 12, 2021 at 03:35:29AM +0100, Viresh Kumar wrote: > On 11-11-21, 17:04, Vincent Whitchurch wrote: > > static int virtio_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg *msgs, > > @@ -141,7 +140,6 @@ static int virtio_i2c_xfer(struct i2c_adapter *adap, > > struct i2c_msg *msgs, > >

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

2021-11-11 Thread Viresh Kumar
On 11-11-21, 17:04, Vincent Whitchurch wrote: > If a timeout is hit, it can result is incorrect data on the I2C bus > and/or memory corruptions in the guest since the device can still be > operating on the buffers it was given while the guest has freed them. > > Here is, for example, the start of

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

2021-11-11 Thread Michael S. Tsirkin
On Thu, Nov 11, 2021 at 05:04:11PM +0100, Vincent Whitchurch wrote: > If a timeout is hit, it can result is incorrect data on the I2C bus > and/or memory corruptions in the guest since the device can still be > operating on the buffers it was given while the guest has freed them. > > Here is, for

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

2021-11-11 Thread Vincent Whitchurch
If a timeout is hit, it can result is incorrect data on the I2C bus and/or memory corruptions in the guest since the device can still be operating on the buffers it was given while the guest has freed them. Here is, for example, the start of a slub_debug splat which was triggered on the next trans