mirror of
https://github.com/raspberrypi/linux.git
synced 2025-12-08 02:49:48 +00:00
octeontx2-pf: Fix PFC TX scheduler free
During PFC TX schedulers free, flag TXSCHQ_FREE_ALL was being set
which caused free up all schedulers other than the PFC schedulers.
This patch fixes that to free only the PFC Tx schedulers.
Fixes: 99c969a83d ("octeontx2-pf: Add egress PFC support")
Signed-off-by: Suman Ghosh <sumang@marvell.com>
Reviewed-by: Simon Horman <horms@kernel.org>
Link: https://lore.kernel.org/r/20230824081032.436432-2-sumang@marvell.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
committed by
Jakub Kicinski
parent
931abcdcad
commit
a9ac2e1877
@@ -804,6 +804,7 @@ void otx2_txschq_free_one(struct otx2_nic *pfvf, u16 lvl, u16 schq)
|
|||||||
|
|
||||||
mutex_unlock(&pfvf->mbox.lock);
|
mutex_unlock(&pfvf->mbox.lock);
|
||||||
}
|
}
|
||||||
|
EXPORT_SYMBOL(otx2_txschq_free_one);
|
||||||
|
|
||||||
void otx2_txschq_stop(struct otx2_nic *pfvf)
|
void otx2_txschq_stop(struct otx2_nic *pfvf)
|
||||||
{
|
{
|
||||||
|
|||||||
@@ -125,19 +125,12 @@ int otx2_pfc_txschq_alloc(struct otx2_nic *pfvf)
|
|||||||
|
|
||||||
static int otx2_pfc_txschq_stop_one(struct otx2_nic *pfvf, u8 prio)
|
static int otx2_pfc_txschq_stop_one(struct otx2_nic *pfvf, u8 prio)
|
||||||
{
|
{
|
||||||
struct nix_txsch_free_req *free_req;
|
int lvl;
|
||||||
|
|
||||||
mutex_lock(&pfvf->mbox.lock);
|
|
||||||
/* free PFC TLx nodes */
|
/* free PFC TLx nodes */
|
||||||
free_req = otx2_mbox_alloc_msg_nix_txsch_free(&pfvf->mbox);
|
for (lvl = 0; lvl < pfvf->hw.txschq_link_cfg_lvl; lvl++)
|
||||||
if (!free_req) {
|
otx2_txschq_free_one(pfvf, lvl,
|
||||||
mutex_unlock(&pfvf->mbox.lock);
|
pfvf->pfc_schq_list[lvl][prio]);
|
||||||
return -ENOMEM;
|
|
||||||
}
|
|
||||||
|
|
||||||
free_req->flags = TXSCHQ_FREE_ALL;
|
|
||||||
otx2_sync_mbox_msg(&pfvf->mbox);
|
|
||||||
mutex_unlock(&pfvf->mbox.lock);
|
|
||||||
|
|
||||||
pfvf->pfc_alloc_status[prio] = false;
|
pfvf->pfc_alloc_status[prio] = false;
|
||||||
return 0;
|
return 0;
|
||||||
|
|||||||
Reference in New Issue
Block a user