From 3f4fc2ccc73a2225d36b2a28118b434b5ca96a7c Mon Sep 17 00:00:00 2001 From: Shivank Garg Date: Wed, 30 Apr 2025 11:29:59 +0000 Subject: [PATCH] x86/mm: pgtable: Fix W=1 build kernel-doc warnings Sample warnings generated with 'make W=1': arch/x86/mm/pgtable.c:623: warning: Function parameter or struct member 'reserve' not described in 'reserve_top_address' arch/x86/mm/pgtable.c:672: warning: Function parameter or struct member 'p4d' not described in 'p4d_set_huge' arch/x86/mm/pgtable.c:672: warning: Function parameter or struct member 'addr' not described in 'p4d_set_huge' ... so on Add missing parameter documentation in page table functions to fix kernel-doc warnings. Signed-off-by: Shivank Garg --- arch/x86/mm/pgtable.c | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/arch/x86/mm/pgtable.c b/arch/x86/mm/pgtable.c index f7ae44d3dd9e..e92bb61ec9d7 100644 --- a/arch/x86/mm/pgtable.c +++ b/arch/x86/mm/pgtable.c @@ -614,7 +614,7 @@ pud_t pudp_invalidate(struct vm_area_struct *vma, unsigned long address, /** * reserve_top_address - reserves a hole in the top of kernel address space - * @reserve - size of hole to reserve + * @reserve: size of hole to reserve. * * Can be used to relocate the fixmap area and poke a hole in the top * of kernel address space to make room for a hypervisor. @@ -665,6 +665,9 @@ void native_set_fixmap(unsigned /* enum fixed_addresses */ idx, #ifdef CONFIG_X86_5LEVEL /** * p4d_set_huge - setup kernel P4D mapping + * @p4d: pointer to a P4D entry. + * @addr: virtual Address associated with p4d. + * @prot: protection bits to use. * * No 512GB pages yet -- always return 0 */ @@ -675,8 +678,9 @@ int p4d_set_huge(p4d_t *p4d, phys_addr_t addr, pgprot_t prot) /** * p4d_clear_huge - clear kernel P4D mapping when it is set + * @p4d: pointer to the P4D entry to clear. * - * No 512GB pages yet -- always return 0 + * No 512GB pages yet -- do nothing */ void p4d_clear_huge(p4d_t *p4d) { @@ -685,6 +689,9 @@ void p4d_clear_huge(p4d_t *p4d) /** * pud_set_huge - setup kernel PUD mapping + * @pud: pointer to a PUD entry. + * @addr: virtual Address associated with pud. + * @prot: protection bits to use. * * MTRRs can override PAT memory types with 4KiB granularity. Therefore, this * function sets up a huge page only if the complete range has the same MTRR @@ -716,6 +723,9 @@ int pud_set_huge(pud_t *pud, phys_addr_t addr, pgprot_t prot) /** * pmd_set_huge - setup kernel PMD mapping + * @pmd: pointer to a PMD entry. + * @addr: virtual Address associated with pmd. + * @prot: protection bits to use. * * See text over pud_set_huge() above. * @@ -745,6 +755,7 @@ int pmd_set_huge(pmd_t *pmd, phys_addr_t addr, pgprot_t prot) /** * pud_clear_huge - clear kernel PUD mapping when it is set + * @pud: pointer to the PUD entry to clear. * * Returns 1 on success and 0 on failure (no PUD map is found). */ @@ -760,6 +771,7 @@ int pud_clear_huge(pud_t *pud) /** * pmd_clear_huge - clear kernel PMD mapping when it is set + * @pmd: pointer to the PMD entry to clear. * * Returns 1 on success and 0 on failure (no PMD map is found). */ -- 2.34.1