Commit 4ea3e632 authored by Ben Walker's avatar Ben Walker Committed by Jim Harris
Browse files

bdev/aio: Remove list of channels on channel group 



This was only temporarily required for polling. With
a per-group aio ctx, it isn't needed anymore.

Change-Id: Ie59b50a4700f0f99dea470f857d187ac656dd229
Signed-off-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-on: https://review.gerrithub.io/c/443467


Reviewed-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 6bb59dff

lib/bdev/aio/bdev_aio.c

+0 −9
Original line number Diff line number Diff line
@@ -51,15 +51,11 @@ 
struct bdev_aio_io_channel {

	uint64_t				io_inflight;

	struct bdev_aio_group_channel		*group_ch;

	TAILQ_ENTRY(bdev_aio_io_channel)	link;

};



struct bdev_aio_group_channel {

	struct spdk_poller			*poller;



	io_context_t				io_ctx;



	TAILQ_HEAD(, bdev_aio_io_channel)	channels;

};



struct bdev_aio_task {
@@ -404,8 +400,6 @@ bdev_aio_create_cb(void *io_device, void *ctx_buf) 


	ch->group_ch = spdk_io_channel_get_ctx(spdk_get_io_channel(&aio_if));



	TAILQ_INSERT_TAIL(&ch->group_ch->channels, ch, link);



	return 0;

}
@@ -414,7 +408,6 @@ bdev_aio_destroy_cb(void *io_device, void *ctx_buf) 
{

	struct bdev_aio_io_channel *ch = ctx_buf;



	TAILQ_REMOVE(&ch->group_ch->channels, ch, link);

	spdk_put_io_channel(spdk_io_channel_from_ctx(ch->group_ch));

}
@@ -485,8 +478,6 @@ bdev_aio_group_create_cb(void *io_device, void *ctx_buf) 
{

	struct bdev_aio_group_channel *ch = ctx_buf;



	TAILQ_INIT(&ch->channels);



	if (io_setup(SPDK_AIO_QUEUE_DEPTH, &ch->io_ctx) < 0) {

		SPDK_ERRLOG("async I/O context setup failure\n");

		return -1;