| Message ID | 20230516225205.1429196-1-sidhartha.kumar@oracle.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | mm/hugetlb: remove hugetlb_page_subpool() | expand |
On 05/16/23 15:52, Sidhartha Kumar wrote: > All users of hugetlb_page_subpool() have been converted to use the folio > equivalent. This function can be safely removed. > > Signed-off-by: Sidhartha Kumar <sidhartha.kumar@oracle.com> > --- > include/linux/hugetlb.h | 13 ------------- > 1 file changed, 13 deletions(-) Thanks! Reviewed-by: Mike Kravetz <mike.kravetz@oracle.com>
On 17.05.23 00:52, Sidhartha Kumar wrote: > All users of hugetlb_page_subpool() have been converted to use the folio > equivalent. This function can be safely removed. > > Signed-off-by: Sidhartha Kumar <sidhartha.kumar@oracle.com> > --- > include/linux/hugetlb.h | 13 ------------- > 1 file changed, 13 deletions(-) > > diff --git a/include/linux/hugetlb.h b/include/linux/hugetlb.h > index b2b698f9a2ecb..21f942025fecd 100644 > --- a/include/linux/hugetlb.h > +++ b/include/linux/hugetlb.h > @@ -755,14 +755,6 @@ static inline struct hugepage_subpool *hugetlb_folio_subpool(struct folio *folio > return folio->_hugetlb_subpool; > } > > -/* > - * hugetlb page subpool pointer located in hpage[2].hugetlb_subpool > - */ > -static inline struct hugepage_subpool *hugetlb_page_subpool(struct page *hpage) > -{ > - return hugetlb_folio_subpool(page_folio(hpage)); > -} > - > static inline void hugetlb_set_folio_subpool(struct folio *folio, > struct hugepage_subpool *subpool) > { > @@ -1029,11 +1021,6 @@ static inline struct hugepage_subpool *hugetlb_folio_subpool(struct folio *folio > return NULL; > } > > -static inline struct hugepage_subpool *hugetlb_page_subpool(struct page *hpage) > -{ > - return NULL; > -} > - > static inline int isolate_or_dissolve_huge_page(struct page *page, > struct list_head *list) > { Reviewed-by: David Hildenbrand <david@redhat.com>
diff --git a/include/linux/hugetlb.h b/include/linux/hugetlb.h index b2b698f9a2ecb..21f942025fecd 100644 --- a/include/linux/hugetlb.h +++ b/include/linux/hugetlb.h @@ -755,14 +755,6 @@ static inline struct hugepage_subpool *hugetlb_folio_subpool(struct folio *folio return folio->_hugetlb_subpool; } -/* - * hugetlb page subpool pointer located in hpage[2].hugetlb_subpool - */ -static inline struct hugepage_subpool *hugetlb_page_subpool(struct page *hpage) -{ - return hugetlb_folio_subpool(page_folio(hpage)); -} - static inline void hugetlb_set_folio_subpool(struct folio *folio, struct hugepage_subpool *subpool) { @@ -1029,11 +1021,6 @@ static inline struct hugepage_subpool *hugetlb_folio_subpool(struct folio *folio return NULL; } -static inline struct hugepage_subpool *hugetlb_page_subpool(struct page *hpage) -{ - return NULL; -} - static inline int isolate_or_dissolve_huge_page(struct page *page, struct list_head *list) {
All users of hugetlb_page_subpool() have been converted to use the folio equivalent. This function can be safely removed. Signed-off-by: Sidhartha Kumar <sidhartha.kumar@oracle.com> --- include/linux/hugetlb.h | 13 ------------- 1 file changed, 13 deletions(-)