Re: [PATCH v2 for 5.2 0/3] block: add logging facility for long standing IO requests
On 8/12/20 5:00 PM, Stefan Hajnoczi wrote: > On Mon, Aug 10, 2020 at 01:14:44PM +0300, Denis V. Lunev wrote: >> There are severe delays with IO requests processing if QEMU is running in >> virtual machine or over software defined storage. Such delays potentially >> results in unpredictable guest behavior. For example, guests over IDE or >> SATA drive could remount filesystem read-only if write is performed >> longer than 10 seconds. >> >> Such reports are very complex to process. Some good starting point for this >> seems quite reasonable. This patch provides one. It adds logging of such >> potentially dangerous long IO operations. >> >> Changed from v2: >> - removed accidentally added slirp subproject ID >> - added comment describing timeout selection to patch 3 >> >> Changes from v1: >> - fixed conversions using macros suggested by Stefan >> - fixed option declaration >> - enabled by default with patch 3 >> >> Signed-off-by: Denis V. Lunev >> CC: Vladimir Sementsov-Ogievskiy >> CC: Kevin Wolf >> CC: Max Reitz >> CC: Stefan Hajnoczi > Reviewed-by: Stefan Hajnoczi ping
Re: [PATCH v2 for 5.2 0/3] block: add logging facility for long standing IO requests
On Mon, Aug 10, 2020 at 01:14:44PM +0300, Denis V. Lunev wrote: > There are severe delays with IO requests processing if QEMU is running in > virtual machine or over software defined storage. Such delays potentially > results in unpredictable guest behavior. For example, guests over IDE or > SATA drive could remount filesystem read-only if write is performed > longer than 10 seconds. > > Such reports are very complex to process. Some good starting point for this > seems quite reasonable. This patch provides one. It adds logging of such > potentially dangerous long IO operations. > > Changed from v2: > - removed accidentally added slirp subproject ID > - added comment describing timeout selection to patch 3 > > Changes from v1: > - fixed conversions using macros suggested by Stefan > - fixed option declaration > - enabled by default with patch 3 > > Signed-off-by: Denis V. Lunev > CC: Vladimir Sementsov-Ogievskiy > CC: Kevin Wolf > CC: Max Reitz > CC: Stefan Hajnoczi Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature