MDEV-24765 fseg_free_extent fails to call buf_page_free() for the whole segment

Thirunarayanan · Thirunarayanan · commit b76e5c66107d · 2021-02-02T18:17:44.000+05:30
This is caused by commit c92f7e2(MDEV-8139). InnoDB fails to set the page status as FREED in buffer pool while freeing the extent.
diff --git a/storage/innobase/fsp/fsp0fsp.cc b/storage/innobase/fsp/fsp0fsp.cc
@@ -2635,9 +2635,9 @@ fseg_free_extent(
 
 	fsp_free_extent(space, page, mtr);
 
-	for (ulint i = 0; i < FSP_EXTENT_SIZE; i++) {
+	for (uint32_t i = 0; i < FSP_EXTENT_SIZE; i++) {
 		if (!xdes_is_free(descr, i)) {
-			buf_page_free(space, first_page_in_extent + 1, mtr,
+			buf_page_free(space, first_page_in_extent + i, mtr,
 				      __FILE__, __LINE__);
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -2635,9 +2635,9 @@ fseg_free_extent(`
`2635`	`2635`
`2636`	`2636`	`fsp_free_extent(space, page, mtr);`
`2637`	`2637`
`2638`		`- for (ulint i = 0; i < FSP_EXTENT_SIZE; i++) {`
	`2638`	`+ for (uint32_t i = 0; i < FSP_EXTENT_SIZE; i++) {`
`2639`	`2639`	`if (!xdes_is_free(descr, i)) {`
`2640`		`- buf_page_free(space, first_page_in_extent + 1, mtr,`
	`2640`	`+ buf_page_free(space, first_page_in_extent + i, mtr,`
`2641`	`2641`	`__FILE__, __LINE__);`
`2642`	`2642`	`}`
`2643`	`2643`	`}`