git: 5013fe43c85e - stable/13 - cxgbei: Explicitly clear the page pode reservation pointer after freeing it.
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Fri, 29 Oct 2021 23:58:12 UTC
The branch stable/13 has been updated by jhb:
URL: https://cgit.FreeBSD.org/src/commit/?id=5013fe43c85ec39cb7d6ce9386074125e24211c2
commit 5013fe43c85ec39cb7d6ce9386074125e24211c2
Author: John Baldwin <jhb@FreeBSD.org>
AuthorDate: 2021-05-14 19:20:51 +0000
Commit: John Baldwin <jhb@FreeBSD.org>
CommitDate: 2021-10-29 23:00:13 +0000
cxgbei: Explicitly clear the page pode reservation pointer after freeing it.
A single union ctl_io can be reused across multiple transfers (in
particular by the ramdisk backend). On a reuse, the reservation
pointer would retain its value from the previous transfer tripping an
assertion.
Reported by: Jithesh Arakkan @ Chelsio
Sponsored by: Chelsio Communications
Differential Revision: https://reviews.freebsd.org/D29939
(cherry picked from commit e894e3adb206815c2acff17a4011becb166c2f66)
---
sys/dev/cxgbe/cxgbei/icl_cxgbei.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sys/dev/cxgbe/cxgbei/icl_cxgbei.c b/sys/dev/cxgbe/cxgbei/icl_cxgbei.c
index e3dc05645732..7f638c96483a 100644
--- a/sys/dev/cxgbe/cxgbei/icl_cxgbei.c
+++ b/sys/dev/cxgbe/cxgbei/icl_cxgbei.c
@@ -1162,6 +1162,7 @@ icl_cxgbei_conn_transfer_done(struct icl_conn *ic, void *arg)
t4_free_page_pods(prsv);
uma_zfree(prsv_zone, prsv);
+ io_to_ppod_reservation(ctsio) = NULL;
}
}