Re: [PATCH] target/iblock: split T10 PI SGL across command bios

2018-08-22 Thread Greg Edwards
On Tue, Aug 21, 2018 at 04:13:57PM -0600, Greg Edwards wrote: > On Tue, Aug 21, 2018 at 04:07:08PM -0500, Mike Christie wrote: >> On 08/08/2018 02:31 PM, Greg Edwards wrote: >>> When T10 PI is enabled on a backing device for the iblock backstore, the >>> PI SGL for the entire command is attached

Re: [PATCH] target/iblock: split T10 PI SGL across command bios

2018-08-21 Thread Greg Edwards
On Tue, Aug 21, 2018 at 04:07:08PM -0500, Mike Christie wrote: > On 08/08/2018 02:31 PM, Greg Edwards wrote: >> When T10 PI is enabled on a backing device for the iblock backstore, the >> PI SGL for the entire command is attached to the first bio only. This >> works fine if the command is covered

Re: [PATCH] target/iblock: split T10 PI SGL across command bios

2018-08-21 Thread Mike Christie
On 08/08/2018 02:31 PM, Greg Edwards wrote: > When T10 PI is enabled on a backing device for the iblock backstore, the > PI SGL for the entire command is attached to the first bio only. This > works fine if the command is covered by a single bio, but results in > integrity verification errors for

[PATCH] target/iblock: split T10 PI SGL across command bios

2018-08-08 Thread Greg Edwards
When T10 PI is enabled on a backing device for the iblock backstore, the PI SGL for the entire command is attached to the first bio only. This works fine if the command is covered by a single bio, but results in integrity verification errors for the other bios in a multi-bio command. Split the