On Tue, Oct 10, 2023 at 06:41:17PM +0800, Zhu Yanjun wrote: > 在 2023/10/9 12:35, Yi Zhang 写道: > > Hello > > > > blktests srp lead kernel panic[2] on aarch64 when the kernel enabled > > CONFIG_ARM64_64K_PAGES, bisect shows it was introduced from commit[1], > > pls help check it and let me know if you need any info/testing for it, thanks. > > > > [1] > > commit 325a7eb85199ec9c5b5a7af812f43ea16b735569 > > Author: Bob Pearson <rpearsonhpe@xxxxxxxxx> > > Date: Thu Jan 19 17:59:36 2023 -0600 > > > > RDMA/rxe: Cleanup page variables in rxe_mr.c > > > > Cleanup usage of mr->page_shift and mr->page_mask and introduce > > an extractor for mr->ibmr.page_size. Normal usage in the kernel > > has page_mask masking out offset in page rather than masking out > > the page number. The rxe driver had reversed that which was confusing. > > Implicitly there can be a per mr page_size which was not uniformly > > supported. > > > > Link: https://lore.kernel.org/r/20230119235936.19728-6-rpearsonhpe@xxxxxxxxx > > Signed-off-by: Bob Pearson <rpearsonhpe@xxxxxxxxx> > > Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxx> > > > > Hi, Yi > > I delved into the commit. And the commit can not be reverted cleanly. So I > made the following diff to try to revert this commit. After this commit is > applied, rping can work well. We can't keep reverting things for what are probably small bugs. Fix the issues please! Jason
