Re: [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files
From: Barret Rhoden <brho@xxxxxxxxxx>
Date: Mon, 16 Dec 2019 11:05:26 -0500
Cc: Paolo Bonzini <pbonzini@xxxxxxxxxx>, Dan Williams <dan.j.williams@xxxxxxxxx>, David Hildenbrand <david@xxxxxxxxxx>, Dave Jiang <dave.jiang@xxxxxxxxx>, Alexander Duyck <alexander.h.duyck@xxxxxxxxxxxxxxx>, linux-nvdimm@xxxxxxxxxxxx, x86@xxxxxxxxxx, kvm@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, jason.zeng@xxxxxxxxx
In-reply-to: <20191213171950.GA31552@linux.intel.com>
References: <20191212182238.46535-1-brho@google.com> <20191212182238.46535-3-brho@google.com> <06108004-1720-41EB-BCAB-BFA8FEBF4772@oracle.com> <ED482280-CB47-4AB6-9E7E-EEE7848E0F8B@oracle.com> <f8e948ff-6a2a-a6d6-9d8e-92b93003354a@google.com> <65FB6CC1-3AD2-4D6F-9481-500BD7037203@oracle.com> <20191213171950.GA31552@linux.intel.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0

On 12/13/19 12:19 PM, Sean Christopherson wrote:

Teaching thp_adjust() how to handle 1GB wouldn't be a bad thing.  It's
unlikely THP itself will support 1GB pages any time soon, but having the
logic there wouldn't hurt anything.


Cool.  This was my main concern - I didn't want to break THP.

I'll rework the series based on all of your comments.

Thanks,

Barret

Follow-Ups:
- Re: [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files
  - From: Sean Christopherson

References:
- [PATCH v5 0/2] kvm: Use huge pages for DAX-backed files
  - From: Barret Rhoden
- [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files
  - From: Barret Rhoden
- Re: [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files
  - From: Liran Alon
- Re: [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files
  - From: Liran Alon
- Re: [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files
  - From: Barret Rhoden
- Re: [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files
  - From: Liran Alon
- Re: [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files
  - From: Sean Christopherson

Prev by Date: Re: [PATCH 12/12] hw/i386/pc: Move PC-machine specific declarations to 'pc_internal.h'
Next by Date: Re: [PATCH v15 3/7] mm: Add function __putback_isolated_page
Previous by thread: Re: [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files
Next by thread: Re: [PATCH v5 2/2] kvm: Use huge pages for DAX-backed files
Index(es):
- Date
- Thread

[Index of Archives] [KVM ARM] [KVM ia64] [KVM ppc] [Virtualization Tools] [Spice Development] [Libvirt] [Libvirt Users] [Linux USB Devel] [Linux Audio Users] [Yosemite Questions] [Linux Kernel] [Linux SCSI] [XFree86]