From: kbuild test robot <lkp@intel.com>
To: john.hubbard@gmail.com
Cc: kbuild-all@01.org, Andrew Morton <akpm@linux-foundation.org>,
Christoph Hellwig <hch@infradead.org>,
Ira Weiny <ira.weiny@intel.com>, Jan Kara <jack@suse.cz>,
Jason Gunthorpe <jgg@ziepe.ca>,
Jerome Glisse <jglisse@redhat.com>,
LKML <linux-kernel@vger.kernel.org>,
linux-mm@kvack.org, linux-fsdevel@vger.kernel.org,
John Hubbard <jhubbard@nvidia.com>,
Benjamin Herrenschmidt <benh@kernel.crashing.org>,
Christoph Hellwig <hch@lst.de>,
Michael Ellerman <mpe@ellerman.id.au>,
linuxppc-dev@lists.ozlabs.org
Subject: Re: [PATCH] powerpc: convert put_page() to put_user_page*()
Date: Thu, 8 Aug 2019 07:24:38 +0800 [thread overview]
Message-ID: <201908080609.5QdIClpX%lkp@intel.com> (raw)
In-Reply-To: <20190805023819.11001-1-jhubbard@nvidia.com>
[-- Attachment #1: Type: text/plain, Size: 6865 bytes --]
Hi,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master]
[cannot apply to v5.3-rc3 next-20190807]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/john-hubbard-gmail-com/powerpc-convert-put_page-to-put_user_page/20190805-132131
config: powerpc-allmodconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 7.4.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
GCC_VERSION=7.4.0 make.cross ARCH=powerpc
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
arch/powerpc/kvm/book3s_64_mmu_radix.c: In function 'kvmppc_book3s_instantiate_page':
>> arch/powerpc/kvm/book3s_64_mmu_radix.c:879:4: error: too many arguments to function 'put_user_pages_dirty_lock'
put_user_pages_dirty_lock(&page, 1, dirty);
^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from arch/powerpc/include/asm/io.h:29:0,
from include/linux/io.h:13,
from include/linux/irq.h:20,
from arch/powerpc/include/asm/hardirq.h:6,
from include/linux/hardirq.h:9,
from include/linux/kvm_host.h:7,
from arch/powerpc/kvm/book3s_64_mmu_radix.c:10:
include/linux/mm.h:1061:6: note: declared here
void put_user_pages_dirty_lock(struct page **pages, unsigned long npages);
^~~~~~~~~~~~~~~~~~~~~~~~~
--
arch/powerpc/mm/book3s64/iommu_api.c: In function 'mm_iommu_unpin':
>> arch/powerpc/mm/book3s64/iommu_api.c:220:3: error: too many arguments to function 'put_user_pages_dirty_lock'
put_user_pages_dirty_lock(&page, 1, dirty);
^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/migrate.h:5:0,
from arch/powerpc/mm/book3s64/iommu_api.c:13:
include/linux/mm.h:1061:6: note: declared here
void put_user_pages_dirty_lock(struct page **pages, unsigned long npages);
^~~~~~~~~~~~~~~~~~~~~~~~~
vim +/put_user_pages_dirty_lock +879 arch/powerpc/kvm/book3s_64_mmu_radix.c
765
766 int kvmppc_book3s_instantiate_page(struct kvm_vcpu *vcpu,
767 unsigned long gpa,
768 struct kvm_memory_slot *memslot,
769 bool writing, bool kvm_ro,
770 pte_t *inserted_pte, unsigned int *levelp)
771 {
772 struct kvm *kvm = vcpu->kvm;
773 struct page *page = NULL;
774 unsigned long mmu_seq;
775 unsigned long hva, gfn = gpa >> PAGE_SHIFT;
776 bool upgrade_write = false;
777 bool *upgrade_p = &upgrade_write;
778 pte_t pte, *ptep;
779 unsigned int shift, level;
780 int ret;
781 bool large_enable;
782
783 /* used to check for invalidations in progress */
784 mmu_seq = kvm->mmu_notifier_seq;
785 smp_rmb();
786
787 /*
788 * Do a fast check first, since __gfn_to_pfn_memslot doesn't
789 * do it with !atomic && !async, which is how we call it.
790 * We always ask for write permission since the common case
791 * is that the page is writable.
792 */
793 hva = gfn_to_hva_memslot(memslot, gfn);
794 if (!kvm_ro && __get_user_pages_fast(hva, 1, 1, &page) == 1) {
795 upgrade_write = true;
796 } else {
797 unsigned long pfn;
798
799 /* Call KVM generic code to do the slow-path check */
800 pfn = __gfn_to_pfn_memslot(memslot, gfn, false, NULL,
801 writing, upgrade_p);
802 if (is_error_noslot_pfn(pfn))
803 return -EFAULT;
804 page = NULL;
805 if (pfn_valid(pfn)) {
806 page = pfn_to_page(pfn);
807 if (PageReserved(page))
808 page = NULL;
809 }
810 }
811
812 /*
813 * Read the PTE from the process' radix tree and use that
814 * so we get the shift and attribute bits.
815 */
816 local_irq_disable();
817 ptep = __find_linux_pte(vcpu->arch.pgdir, hva, NULL, &shift);
818 /*
819 * If the PTE disappeared temporarily due to a THP
820 * collapse, just return and let the guest try again.
821 */
822 if (!ptep) {
823 local_irq_enable();
824 if (page) {
825 if (upgrade_write)
826 put_user_page(page);
827 else
828 put_page(page);
829 }
830 return RESUME_GUEST;
831 }
832 pte = *ptep;
833 local_irq_enable();
834
835 /* If we're logging dirty pages, always map single pages */
836 large_enable = !(memslot->flags & KVM_MEM_LOG_DIRTY_PAGES);
837
838 /* Get pte level from shift/size */
839 if (large_enable && shift == PUD_SHIFT &&
840 (gpa & (PUD_SIZE - PAGE_SIZE)) ==
841 (hva & (PUD_SIZE - PAGE_SIZE))) {
842 level = 2;
843 } else if (large_enable && shift == PMD_SHIFT &&
844 (gpa & (PMD_SIZE - PAGE_SIZE)) ==
845 (hva & (PMD_SIZE - PAGE_SIZE))) {
846 level = 1;
847 } else {
848 level = 0;
849 if (shift > PAGE_SHIFT) {
850 /*
851 * If the pte maps more than one page, bring over
852 * bits from the virtual address to get the real
853 * address of the specific single page we want.
854 */
855 unsigned long rpnmask = (1ul << shift) - PAGE_SIZE;
856 pte = __pte(pte_val(pte) | (hva & rpnmask));
857 }
858 }
859
860 pte = __pte(pte_val(pte) | _PAGE_EXEC | _PAGE_ACCESSED);
861 if (writing || upgrade_write) {
862 if (pte_val(pte) & _PAGE_WRITE)
863 pte = __pte(pte_val(pte) | _PAGE_DIRTY);
864 } else {
865 pte = __pte(pte_val(pte) & ~(_PAGE_WRITE | _PAGE_DIRTY));
866 }
867
868 /* Allocate space in the tree and write the PTE */
869 ret = kvmppc_create_pte(kvm, kvm->arch.pgtable, pte, gpa, level,
870 mmu_seq, kvm->arch.lpid, NULL, NULL);
871 if (inserted_pte)
872 *inserted_pte = pte;
873 if (levelp)
874 *levelp = level;
875
876 if (page) {
877 bool dirty = !ret && (pte_val(pte) & _PAGE_WRITE);
878 if (upgrade_write)
> 879 put_user_pages_dirty_lock(&page, 1, dirty);
880 else {
881 if (dirty)
882 set_page_dirty_lock(page);
883 put_page(page);
884 }
885 }
886
887 /* Increment number of large pages if we (successfully) inserted one */
888 if (!ret) {
889 if (level == 1)
890 kvm->stat.num_2M_pages++;
891 else if (level == 2)
892 kvm->stat.num_1G_pages++;
893 }
894
895 return ret;
896 }
897
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 62277 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com>
To: john.hubbard@gmail.com
Cc: Jan Kara <jack@suse.cz>,
linux-mm@kvack.org, John Hubbard <jhubbard@nvidia.com>,
linuxppc-dev@lists.ozlabs.org,
LKML <linux-kernel@vger.kernel.org>,
Christoph Hellwig <hch@infradead.org>,
Jason Gunthorpe <jgg@ziepe.ca>,
Jerome Glisse <jglisse@redhat.com>,
kbuild-all@01.org, linux-fsdevel@vger.kernel.org,
Andrew Morton <akpm@linux-foundation.org>,
Ira Weiny <ira.weiny@intel.com>, Christoph Hellwig <hch@lst.de>
Subject: Re: [PATCH] powerpc: convert put_page() to put_user_page*()
Date: Thu, 8 Aug 2019 07:24:38 +0800 [thread overview]
Message-ID: <201908080609.5QdIClpX%lkp@intel.com> (raw)
In-Reply-To: <20190805023819.11001-1-jhubbard@nvidia.com>
[-- Attachment #1: Type: text/plain, Size: 6865 bytes --]
Hi,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on linus/master]
[cannot apply to v5.3-rc3 next-20190807]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/john-hubbard-gmail-com/powerpc-convert-put_page-to-put_user_page/20190805-132131
config: powerpc-allmodconfig (attached as .config)
compiler: powerpc64-linux-gcc (GCC) 7.4.0
reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
GCC_VERSION=7.4.0 make.cross ARCH=powerpc
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All errors (new ones prefixed by >>):
arch/powerpc/kvm/book3s_64_mmu_radix.c: In function 'kvmppc_book3s_instantiate_page':
>> arch/powerpc/kvm/book3s_64_mmu_radix.c:879:4: error: too many arguments to function 'put_user_pages_dirty_lock'
put_user_pages_dirty_lock(&page, 1, dirty);
^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from arch/powerpc/include/asm/io.h:29:0,
from include/linux/io.h:13,
from include/linux/irq.h:20,
from arch/powerpc/include/asm/hardirq.h:6,
from include/linux/hardirq.h:9,
from include/linux/kvm_host.h:7,
from arch/powerpc/kvm/book3s_64_mmu_radix.c:10:
include/linux/mm.h:1061:6: note: declared here
void put_user_pages_dirty_lock(struct page **pages, unsigned long npages);
^~~~~~~~~~~~~~~~~~~~~~~~~
--
arch/powerpc/mm/book3s64/iommu_api.c: In function 'mm_iommu_unpin':
>> arch/powerpc/mm/book3s64/iommu_api.c:220:3: error: too many arguments to function 'put_user_pages_dirty_lock'
put_user_pages_dirty_lock(&page, 1, dirty);
^~~~~~~~~~~~~~~~~~~~~~~~~
In file included from include/linux/migrate.h:5:0,
from arch/powerpc/mm/book3s64/iommu_api.c:13:
include/linux/mm.h:1061:6: note: declared here
void put_user_pages_dirty_lock(struct page **pages, unsigned long npages);
^~~~~~~~~~~~~~~~~~~~~~~~~
vim +/put_user_pages_dirty_lock +879 arch/powerpc/kvm/book3s_64_mmu_radix.c
765
766 int kvmppc_book3s_instantiate_page(struct kvm_vcpu *vcpu,
767 unsigned long gpa,
768 struct kvm_memory_slot *memslot,
769 bool writing, bool kvm_ro,
770 pte_t *inserted_pte, unsigned int *levelp)
771 {
772 struct kvm *kvm = vcpu->kvm;
773 struct page *page = NULL;
774 unsigned long mmu_seq;
775 unsigned long hva, gfn = gpa >> PAGE_SHIFT;
776 bool upgrade_write = false;
777 bool *upgrade_p = &upgrade_write;
778 pte_t pte, *ptep;
779 unsigned int shift, level;
780 int ret;
781 bool large_enable;
782
783 /* used to check for invalidations in progress */
784 mmu_seq = kvm->mmu_notifier_seq;
785 smp_rmb();
786
787 /*
788 * Do a fast check first, since __gfn_to_pfn_memslot doesn't
789 * do it with !atomic && !async, which is how we call it.
790 * We always ask for write permission since the common case
791 * is that the page is writable.
792 */
793 hva = gfn_to_hva_memslot(memslot, gfn);
794 if (!kvm_ro && __get_user_pages_fast(hva, 1, 1, &page) == 1) {
795 upgrade_write = true;
796 } else {
797 unsigned long pfn;
798
799 /* Call KVM generic code to do the slow-path check */
800 pfn = __gfn_to_pfn_memslot(memslot, gfn, false, NULL,
801 writing, upgrade_p);
802 if (is_error_noslot_pfn(pfn))
803 return -EFAULT;
804 page = NULL;
805 if (pfn_valid(pfn)) {
806 page = pfn_to_page(pfn);
807 if (PageReserved(page))
808 page = NULL;
809 }
810 }
811
812 /*
813 * Read the PTE from the process' radix tree and use that
814 * so we get the shift and attribute bits.
815 */
816 local_irq_disable();
817 ptep = __find_linux_pte(vcpu->arch.pgdir, hva, NULL, &shift);
818 /*
819 * If the PTE disappeared temporarily due to a THP
820 * collapse, just return and let the guest try again.
821 */
822 if (!ptep) {
823 local_irq_enable();
824 if (page) {
825 if (upgrade_write)
826 put_user_page(page);
827 else
828 put_page(page);
829 }
830 return RESUME_GUEST;
831 }
832 pte = *ptep;
833 local_irq_enable();
834
835 /* If we're logging dirty pages, always map single pages */
836 large_enable = !(memslot->flags & KVM_MEM_LOG_DIRTY_PAGES);
837
838 /* Get pte level from shift/size */
839 if (large_enable && shift == PUD_SHIFT &&
840 (gpa & (PUD_SIZE - PAGE_SIZE)) ==
841 (hva & (PUD_SIZE - PAGE_SIZE))) {
842 level = 2;
843 } else if (large_enable && shift == PMD_SHIFT &&
844 (gpa & (PMD_SIZE - PAGE_SIZE)) ==
845 (hva & (PMD_SIZE - PAGE_SIZE))) {
846 level = 1;
847 } else {
848 level = 0;
849 if (shift > PAGE_SHIFT) {
850 /*
851 * If the pte maps more than one page, bring over
852 * bits from the virtual address to get the real
853 * address of the specific single page we want.
854 */
855 unsigned long rpnmask = (1ul << shift) - PAGE_SIZE;
856 pte = __pte(pte_val(pte) | (hva & rpnmask));
857 }
858 }
859
860 pte = __pte(pte_val(pte) | _PAGE_EXEC | _PAGE_ACCESSED);
861 if (writing || upgrade_write) {
862 if (pte_val(pte) & _PAGE_WRITE)
863 pte = __pte(pte_val(pte) | _PAGE_DIRTY);
864 } else {
865 pte = __pte(pte_val(pte) & ~(_PAGE_WRITE | _PAGE_DIRTY));
866 }
867
868 /* Allocate space in the tree and write the PTE */
869 ret = kvmppc_create_pte(kvm, kvm->arch.pgtable, pte, gpa, level,
870 mmu_seq, kvm->arch.lpid, NULL, NULL);
871 if (inserted_pte)
872 *inserted_pte = pte;
873 if (levelp)
874 *levelp = level;
875
876 if (page) {
877 bool dirty = !ret && (pte_val(pte) & _PAGE_WRITE);
878 if (upgrade_write)
> 879 put_user_pages_dirty_lock(&page, 1, dirty);
880 else {
881 if (dirty)
882 set_page_dirty_lock(page);
883 put_page(page);
884 }
885 }
886
887 /* Increment number of large pages if we (successfully) inserted one */
888 if (!ret) {
889 if (level == 1)
890 kvm->stat.num_2M_pages++;
891 else if (level == 2)
892 kvm->stat.num_1G_pages++;
893 }
894
895 return ret;
896 }
897
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 62277 bytes --]
next prev parent reply other threads:[~2019-08-07 23:25 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-05 2:38 [PATCH] powerpc: convert put_page() to put_user_page*() john.hubbard
2019-08-05 2:38 ` john.hubbard
2019-08-07 23:24 ` kbuild test robot [this message]
2019-08-07 23:24 ` kbuild test robot
2019-08-07 23:34 ` John Hubbard
2019-08-07 23:34 ` John Hubbard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201908080609.5QdIClpX%lkp@intel.com \
--to=lkp@intel.com \
--cc=akpm@linux-foundation.org \
--cc=benh@kernel.crashing.org \
--cc=hch@infradead.org \
--cc=hch@lst.de \
--cc=ira.weiny@intel.com \
--cc=jack@suse.cz \
--cc=jgg@ziepe.ca \
--cc=jglisse@redhat.com \
--cc=jhubbard@nvidia.com \
--cc=john.hubbard@gmail.com \
--cc=kbuild-all@01.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=mpe@ellerman.id.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.