* [PATCH 1/1] crypto:chelsio: Update ntx queue received from cxgb4
[not found] <cover.1539343454.git.harsh@chelsio.com>
@ 2018-10-12 11:46 ` Harsh Jain
2018-10-13 0:16 ` Lino Sanfilippo
0 siblings, 1 reply; 3+ messages in thread
From: Harsh Jain @ 2018-10-12 11:46 UTC (permalink / raw)
To: herbert, atul.gupta, indranil, swise, varun, ganeshgr, netdev,
linux-crypto
Cc: Harsh Jain
Update cxgb4 to send No. of Tx Queue created in lldinfo struct
and use the same ntxq in chcr driver.
This patch depends on following commit
commit add92a817e60e308a419693413a38d9d1e663aff
"Fix memory corruption in DMA Mapped buffers"
Signed-off-by: Harsh Jain <harsh@chelsio.com>
---
drivers/crypto/chelsio/chcr_algo.c | 3 +--
drivers/crypto/chelsio/chcr_core.c | 2 +-
drivers/net/ethernet/chelsio/cxgb4/cxgb4_uld.c | 19 +++++++++++++++----
3 files changed, 17 insertions(+), 7 deletions(-)
diff --git a/drivers/crypto/chelsio/chcr_algo.c b/drivers/crypto/chelsio/chcr_algo.c
index 010bbf6..9b937cb 100644
--- a/drivers/crypto/chelsio/chcr_algo.c
+++ b/drivers/crypto/chelsio/chcr_algo.c
@@ -1337,8 +1337,7 @@ static int chcr_device_init(struct chcr_context *ctx)
}
ctx->dev = u_ctx->dev;
adap = padap(ctx->dev);
- ntxq = min_not_zero((unsigned int)u_ctx->lldi.nrxq,
- adap->vres.ncrypto_fc);
+ ntxq = u_ctx->lldi.ntxq;
rxq_perchan = u_ctx->lldi.nrxq / u_ctx->lldi.nchan;
txq_perchan = ntxq / u_ctx->lldi.nchan;
spin_lock(&ctx->dev->lock_chcr_dev);
diff --git a/drivers/crypto/chelsio/chcr_core.c b/drivers/crypto/chelsio/chcr_core.c
index 04f277c..2399ce3 100644
--- a/drivers/crypto/chelsio/chcr_core.c
+++ b/drivers/crypto/chelsio/chcr_core.c
@@ -43,7 +43,7 @@ static chcr_handler_func work_handlers[NUM_CPL_CMDS] = {
static struct cxgb4_uld_info chcr_uld_info = {
.name = DRV_MODULE_NAME,
.nrxq = MAX_ULD_QSETS,
- .ntxq = MAX_ULD_QSETS,
+ /* Max ntxq will be derived from fw config file*/
.rxq_size = 1024,
.add = chcr_uld_add,
.state_change = chcr_uld_state_change,
diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_uld.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_uld.c
index 4bc2110..ad4fa0d 100644
--- a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_uld.c
+++ b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_uld.c
@@ -520,10 +520,19 @@ setup_sge_txq_uld(struct adapter *adap, unsigned int uld_type,
txq_info = kzalloc(sizeof(*txq_info), GFP_KERNEL);
if (!txq_info)
return -ENOMEM;
+ if (uld_type == CXGB4_ULD_CRYPTO) {
+ i = min_t(int, adap->vres.ncrypto_fc,
+ num_online_cpus());
+ txq_info->ntxq = rounddown(i, adap->params.nports);
+ if (txq_info->ntxq <= 0) {
+ dev_warn(adap->pdev_dev, "Crypto Tx Queues can't be zero\n");
+ return -EINVAL;
+ }
- i = min_t(int, uld_info->ntxq, num_online_cpus());
- txq_info->ntxq = roundup(i, adap->params.nports);
-
+ } else {
+ i = min_t(int, uld_info->ntxq, num_online_cpus());
+ txq_info->ntxq = roundup(i, adap->params.nports);
+ }
txq_info->uldtxq = kcalloc(txq_info->ntxq, sizeof(struct sge_uld_txq),
GFP_KERNEL);
if (!txq_info->uldtxq) {
@@ -546,11 +555,14 @@ static void uld_queue_init(struct adapter *adap, unsigned int uld_type,
struct cxgb4_lld_info *lli)
{
struct sge_uld_rxq_info *rxq_info = adap->sge.uld_rxq_info[uld_type];
+ int tx_uld_type = TX_ULD(uld_type);
+ struct sge_uld_txq_info *txq_info = adap->sge.uld_txq_info[tx_uld_type];
lli->rxq_ids = rxq_info->rspq_id;
lli->nrxq = rxq_info->nrxq;
lli->ciq_ids = rxq_info->rspq_id + rxq_info->nrxq;
lli->nciq = rxq_info->nciq;
+ lli->ntxq = txq_info->ntxq;
}
int t4_uld_mem_alloc(struct adapter *adap)
@@ -634,7 +646,6 @@ static void uld_init(struct adapter *adap, struct cxgb4_lld_info *lld)
lld->ports = adap->port;
lld->vr = &adap->vres;
lld->mtus = adap->params.mtus;
- lld->ntxq = adap->sge.ofldqsets;
lld->nchan = adap->params.nports;
lld->nports = adap->params.nports;
lld->wr_cred = adap->params.ofldq_wr_cred;
--
2.1.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/1] crypto:chelsio: Update ntx queue received from cxgb4
2018-10-12 11:46 ` [PATCH 1/1] crypto:chelsio: Update ntx queue received from cxgb4 Harsh Jain
@ 2018-10-13 0:16 ` Lino Sanfilippo
2018-10-13 7:23 ` Harsh Jain
0 siblings, 1 reply; 3+ messages in thread
From: Lino Sanfilippo @ 2018-10-13 0:16 UTC (permalink / raw)
To: Harsh Jain, herbert, atul.gupta, indranil, swise, varun,
ganeshgr, netdev, linux-crypto
Hi,
> + if (uld_type == CXGB4_ULD_CRYPTO) {
> + i = min_t(int, adap->vres.ncrypto_fc,
> + num_online_cpus());
> + txq_info->ntxq = rounddown(i, adap->params.nports);
> + if (txq_info->ntxq <= 0) {
> + dev_warn(adap->pdev_dev, "Crypto Tx Queues can't be zero\n");
> + return -EINVAL;
> + }
Shouldn't we free txq_info in the error case?
Regards,
Lino
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 1/1] crypto:chelsio: Update ntx queue received from cxgb4
2018-10-13 0:16 ` Lino Sanfilippo
@ 2018-10-13 7:23 ` Harsh Jain
0 siblings, 0 replies; 3+ messages in thread
From: Harsh Jain @ 2018-10-13 7:23 UTC (permalink / raw)
To: Lino Sanfilippo, herbert, atul.gupta, indranil, swise, varun,
ganeshgr, netdev, linux-crypto
On 13-10-2018 05:46, Lino Sanfilippo wrote:
> Hi,
>
>> + if (uld_type == CXGB4_ULD_CRYPTO) {
>> + i = min_t(int, adap->vres.ncrypto_fc,
>> + num_online_cpus());
>> + txq_info->ntxq = rounddown(i, adap->params.nports);
>> + if (txq_info->ntxq <= 0) {
>> + dev_warn(adap->pdev_dev, "Crypto Tx Queues can't be zero\n");
>> + return -EINVAL;
>> + }
> Shouldn't we free txq_info in the error case?
Yes, Will fix this in V2. Thanks.
>
> Regards,
> Lino
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-10-13 7:23 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <cover.1539343454.git.harsh@chelsio.com>
2018-10-12 11:46 ` [PATCH 1/1] crypto:chelsio: Update ntx queue received from cxgb4 Harsh Jain
2018-10-13 0:16 ` Lino Sanfilippo
2018-10-13 7:23 ` Harsh Jain
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.