Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 88ba2aa5 authored by Dan Williams's avatar Dan Williams
Browse files

async_tx: kill ASYNC_TX_DEP_ACK flag 



In support of inter-channel chaining async_tx utilizes an ack flag to
gate whether a dependent operation can be chained to another.  While the
flag is not set the chain can be considered open for appending.  Setting
the ack flag closes the chain and flags the descriptor for garbage
collection.  The ASYNC_TX_DEP_ACK flag essentially means "close the
chain after adding this dependency".  Since each operation can only have
one child the api now implicitly sets the ack flag at dependency
submission time.  This removes an unnecessary management burden from
clients of the api.

[ Impact: clean up and enforce one dependency per operation ]

Reviewed-by: default avatarAndre Noll <maan@systemlinux.org>
Acked-by: default avatarMaciej Sosnowski <maciej.sosnowski@intel.com>
Signed-off-by: default avatarDan Williams <dan.j.williams@intel.com>
parent 099f53cb

Documentation/crypto/async-tx-api.txt

+4 −5
Original line number Diff line number Diff line
@@ -80,8 +80,8 @@ acknowledged by the application before the offload engine driver is allowed to 
recycle (or free) the descriptor.  A descriptor can be acked by one of the

following methods:

1/ setting the ASYNC_TX_ACK flag if no child operations are to be submitted

2/ setting the ASYNC_TX_DEP_ACK flag to acknowledge the parent

   descriptor of a new operation.

2/ submitting an unacknowledged descriptor as a dependency to another

   async_tx call will implicitly set the acknowledged state.

3/ calling async_tx_ack() on the descriptor.



3.4 When does the operation execute?
@@ -136,10 +136,9 @@ int run_xor_copy_xor(struct page **xor_srcs, 


	tx = async_xor(xor_dest, xor_srcs, 0, xor_src_cnt, xor_len,

		       ASYNC_TX_XOR_DROP_DST, NULL, NULL, NULL);

	tx = async_memcpy(copy_dest, copy_src, 0, 0, copy_len,

			  ASYNC_TX_DEP_ACK, tx, NULL, NULL);

	tx = async_memcpy(copy_dest, copy_src, 0, 0, copy_len, tx, NULL, NULL);

	tx = async_xor(xor_dest, xor_srcs, 0, xor_src_cnt, xor_len,

		       ASYNC_TX_XOR_DROP_DST | ASYNC_TX_DEP_ACK | ASYNC_TX_ACK,

		       ASYNC_TX_XOR_DROP_DST | ASYNC_TX_ACK,

		       tx, complete_xor_copy_xor, NULL);



	async_tx_issue_pending_all();

crypto/async_tx/async_memcpy.c

+1 −1
Original line number Diff line number Diff line
@@ -35,7 +35,7 @@ 
 * @src: src page

 * @offset: offset in pages to start transaction

 * @len: length in bytes

 * @flags: ASYNC_TX_ACK, ASYNC_TX_DEP_ACK,

 * @flags: ASYNC_TX_ACK

 * @depend_tx: memcpy depends on the result of this transaction

 * @cb_fn: function to call when the memcpy completes

 * @cb_param: parameter to pass to the callback routine

crypto/async_tx/async_memset.c

+1 −1
Original line number Diff line number Diff line
@@ -35,7 +35,7 @@ 
 * @val: fill value

 * @offset: offset in pages to start transaction

 * @len: length in bytes

 * @flags: ASYNC_TX_ACK, ASYNC_TX_DEP_ACK

 * @flags: ASYNC_TX_ACK

 * @depend_tx: memset depends on the result of this transaction

 * @cb_fn: function to call when the memcpy completes

 * @cb_param: parameter to pass to the callback routine

crypto/async_tx/async_tx.c

+2 −2
Original line number Diff line number Diff line
@@ -223,7 +223,7 @@ async_tx_submit(struct dma_chan *chan, struct dma_async_tx_descriptor *tx, 
	if (flags & ASYNC_TX_ACK)

		async_tx_ack(tx);



	if (depend_tx && (flags & ASYNC_TX_DEP_ACK))

	if (depend_tx)

		async_tx_ack(depend_tx);

}

EXPORT_SYMBOL_GPL(async_tx_submit);
@@ -231,7 +231,7 @@ EXPORT_SYMBOL_GPL(async_tx_submit); 
/**

 * async_trigger_callback - schedules the callback function to be run after

 * any dependent operations have been completed.

 * @flags: ASYNC_TX_ACK, ASYNC_TX_DEP_ACK

 * @flags: ASYNC_TX_ACK

 * @depend_tx: 'callback' requires the completion of this transaction

 * @cb_fn: function to call after depend_tx completes

 * @cb_param: parameter to pass to the callback routine

crypto/async_tx/async_xor.c

+2 −4
Original line number Diff line number Diff line
@@ -105,7 +105,6 @@ do_async_xor(struct dma_chan *chan, struct page *dest, struct page **src_list, 
				_cb_param);



		depend_tx = tx;

		flags |= ASYNC_TX_DEP_ACK;



		if (src_cnt > xor_src_cnt) {

			/* drop completed sources */
@@ -168,8 +167,7 @@ do_sync_xor(struct page *dest, struct page **src_list, unsigned int offset, 
 * @offset: offset in pages to start transaction

 * @src_cnt: number of source pages

 * @len: length in bytes

 * @flags: ASYNC_TX_XOR_ZERO_DST, ASYNC_TX_XOR_DROP_DEST,

 *	ASYNC_TX_ACK, ASYNC_TX_DEP_ACK

 * @flags: ASYNC_TX_XOR_ZERO_DST, ASYNC_TX_XOR_DROP_DEST, ASYNC_TX_ACK

 * @depend_tx: xor depends on the result of this transaction.

 * @cb_fn: function to call when the xor completes

 * @cb_param: parameter to pass to the callback routine
@@ -230,7 +228,7 @@ static int page_is_zero(struct page *p, unsigned int offset, size_t len) 
 * @src_cnt: number of source pages

 * @len: length in bytes

 * @result: 0 if sum == 0 else non-zero

 * @flags: ASYNC_TX_ACK, ASYNC_TX_DEP_ACK

 * @flags: ASYNC_TX_ACK

 * @depend_tx: xor depends on the result of this transaction.

 * @cb_fn: function to call when the xor completes

 * @cb_param: parameter to pass to the callback routine