| Message ID | 20210127201031.98544-2-alobakin@pm.me (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | net: consolidate page_is_pfmemalloc() usage | expand |
| Context | Check | Description |
|---|---|---|
| netdev/cover_letter | success | Link |
| netdev/fixes_present | success | Link |
| netdev/patch_count | success | Link |
| netdev/tree_selection | success | Clearly marked for net-next |
| netdev/subject_prefix | success | Link |
| netdev/cc_maintainers | success | CCed 2 of 2 maintainers |
| netdev/source_inline | success | Was 0 now: 0 |
| netdev/verify_signedoff | success | Link |
| netdev/module_param | success | Was 0 now: 0 |
| netdev/build_32bit | success | Errors and warnings before: 22615 this patch: 22615 |
| netdev/kdoc | success | Errors and warnings before: 12 this patch: 12 |
| netdev/verify_fixes | success | Link |
| netdev/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 8 lines checked |
| netdev/build_allmodconfig_warn | success | Errors and warnings before: 22031 this patch: 22031 |
| netdev/header_inline | success | Link |
| netdev/stable | success | Stable not CCed |
On Wed, 27 Jan 2021, Alexander Lobakin wrote: > The function only tests for page->index, so its argument should be > const. > > Signed-off-by: Alexander Lobakin <alobakin@pm.me> Acked-by: David Rientjes <rientjes@google.com>
On Wed, 27 Jan 2021 20:11:01 +0000 Alexander Lobakin wrote: > The function only tests for page->index, so its argument should be > const. > > Signed-off-by: Alexander Lobakin <alobakin@pm.me> > --- > include/linux/mm.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/include/linux/mm.h b/include/linux/mm.h > index ecdf8a8cd6ae..078633d43af9 100644 > --- a/include/linux/mm.h > +++ b/include/linux/mm.h > @@ -1584,7 +1584,7 @@ struct address_space *page_mapping_file(struct page *page); > * ALLOC_NO_WATERMARKS and the low watermark was not > * met implying that the system is under some pressure. > */ > -static inline bool page_is_pfmemalloc(struct page *page) > +static inline bool page_is_pfmemalloc(const struct page *page) > { > /* > * Page index cannot be this large so this must be No objections for this going via net-next?
diff --git a/include/linux/mm.h b/include/linux/mm.h index ecdf8a8cd6ae..078633d43af9 100644 --- a/include/linux/mm.h +++ b/include/linux/mm.h @@ -1584,7 +1584,7 @@ struct address_space *page_mapping_file(struct page *page); * ALLOC_NO_WATERMARKS and the low watermark was not * met implying that the system is under some pressure. */ -static inline bool page_is_pfmemalloc(struct page *page) +static inline bool page_is_pfmemalloc(const struct page *page) { /* * Page index cannot be this large so this must be
The function only tests for page->index, so its argument should be const. Signed-off-by: Alexander Lobakin <alobakin@pm.me> --- include/linux/mm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)