Commit d8a22322 authored by Artur Paszkiewicz's avatar Artur Paszkiewicz Committed by Tomasz Zawadzki
Browse files

module/raid: fix type of raid_bdev_io completion status 



The status in raid_bdev_io is stored as uint8_t and this causes
spdk_bdev_io_status passed in raid_bdev_io_complete_part() to be
truncated for the negative enum values. This truncated value gets later
passed to spdk_bdev_io_complete(). Fix it by using the correct type.

Change-Id: I99c1d2e8b14c51acf6d3e8f1087eb65f7e05b821
Signed-off-by: default avatarArtur Paszkiewicz <artur.paszkiewicz@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/18710


Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
parent 255d0786

module/bdev/raid/bdev_raid.h

+1 −1
Original line number Diff line number Diff line
@@ -112,7 +112,7 @@ struct raid_bdev_io { 
	/* Used for tracking progress on io requests sent to member disks. */

	uint64_t			base_bdev_io_remaining;

	uint8_t				base_bdev_io_submitted;

	uint8_t				base_bdev_io_status;

	enum spdk_bdev_io_status	base_bdev_io_status;



	/* Private data for the raid module */

	void				*module_private;