Commit b9d83821 authored by Wojciech Malikowski's avatar Wojciech Malikowski Committed by Ben Walker
Browse files

lib/ftl: Prevent from sending writes on resetting zone 



When reset operation is ongoing mark zone as busy
to avoid write operation on it.

This path fixes issue #1171.

Change-Id: I30097e1db1bfbbabebd7e6adeab6e8810dc8b2b1
Signed-off-by: default avatarWojciech Malikowski <wojciech.malikowski@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/483264


Community-CI: SPDK CI Jenkins <sys_sgci@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: default avatarMateusz Kozlowski <mateusz.kozlowski@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
parent e1d1695c

lib/ftl/ftl_band.c

+6 −1
Original line number Diff line number Diff line
@@ -1043,11 +1043,14 @@ ftl_band_erase_cb(struct ftl_io *io, void *ctx, int status) 
{

	struct ftl_zone *zone;



	zone = ftl_band_zone_from_addr(io->band, io->addr);

	zone->busy = false;



	if (spdk_unlikely(status)) {

		ftl_erase_fail(io, status);

		return;

	}

	zone = ftl_band_zone_from_addr(io->band, io->addr);



	zone->info.state = SPDK_BDEV_ZONE_STATE_EMPTY;

	zone->info.write_pointer = zone->info.zone_id;

}
@@ -1075,9 +1078,11 @@ ftl_band_erase(struct ftl_band *band) 
			break;

		}



		zone->busy = true;

		io->addr.offset = zone->info.zone_id;

		rc = ftl_io_erase(io);

		if (rc) {

			zone->busy = false;

			assert(0);

			/* TODO: change band's state back to close? */

			break;