Re: [PATCH v5 0/5] block: widen and check consistency of size-related BlockConf properties

2020-05-26 Thread Roman Kagan
Please ignore the series, I forgot xen-block device. Roman. On Tue, May 26, 2020 at 11:17:35AM +0300, Roman Kagan wrote: > BlockConf includes several properties counted in bytes. > > Enhance their handling in a some aspects, specifically > > - accept common size suffixes (k, m) > - perform

Re: [PATCH v5 0/5] block: widen and check consistency of size-related BlockConf properties

2020-05-26 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200526081740.256236-1-rvka...@yandex-team.ru/ Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20200526081740.256236-1-rvka...@yandex-team.ru Subject: [PATCH v5 0/5] block: widen and check

Re: [PATCH v5 0/5] block: widen and check consistency of size-related BlockConf properties

2020-05-26 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200526081740.256236-1-rvka...@yandex-team.ru/ Hi, This series failed the asan build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN ===

Re: [PATCH v5 0/5] block: widen and check consistency of size-related BlockConf properties

2020-05-26 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200526081740.256236-1-rvka...@yandex-team.ru/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT

[PATCH v5 0/5] block: widen and check consistency of size-related BlockConf properties

2020-05-26 Thread Roman Kagan
BlockConf includes several properties counted in bytes. Enhance their handling in a some aspects, specifically - accept common size suffixes (k, m) - perform consistency checks on the values, to prevent their silent truncation and rounding - lift the upper limit on physical_block_size and