KVM: PPC: Book3S HV: Cleanups - constify memslots, fix comments
This adds 'const' to the declarations for the struct kvm_memory_slot pointer parameters of some functions, which will make it possible to call those functions from kvmppc_core_commit_memory_region_hv() in the next patch. This also fixes some comments about locking. Signed-off-by: Paul Mackerras <paulus@ozlabs.org> Reviewed-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com> Reviewed-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Paul Mackerras <paulus@ozlabs.org>
This commit is contained in:
@@ -197,7 +197,8 @@ extern int kvmppc_mmu_radix_translate_table(struct kvm_vcpu *vcpu, gva_t eaddr,
|
|||||||
extern int kvmppc_mmu_radix_xlate(struct kvm_vcpu *vcpu, gva_t eaddr,
|
extern int kvmppc_mmu_radix_xlate(struct kvm_vcpu *vcpu, gva_t eaddr,
|
||||||
struct kvmppc_pte *gpte, bool data, bool iswrite);
|
struct kvmppc_pte *gpte, bool data, bool iswrite);
|
||||||
extern void kvmppc_unmap_pte(struct kvm *kvm, pte_t *pte, unsigned long gpa,
|
extern void kvmppc_unmap_pte(struct kvm *kvm, pte_t *pte, unsigned long gpa,
|
||||||
unsigned int shift, struct kvm_memory_slot *memslot,
|
unsigned int shift,
|
||||||
|
const struct kvm_memory_slot *memslot,
|
||||||
unsigned int lpid);
|
unsigned int lpid);
|
||||||
extern bool kvmppc_hv_handle_set_rc(struct kvm *kvm, pgd_t *pgtable,
|
extern bool kvmppc_hv_handle_set_rc(struct kvm *kvm, pgd_t *pgtable,
|
||||||
bool writing, unsigned long gpa,
|
bool writing, unsigned long gpa,
|
||||||
@@ -215,10 +216,6 @@ extern int kvmppc_radix_init(void);
|
|||||||
extern void kvmppc_radix_exit(void);
|
extern void kvmppc_radix_exit(void);
|
||||||
extern int kvm_unmap_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
extern int kvm_unmap_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
||||||
unsigned long gfn);
|
unsigned long gfn);
|
||||||
extern void kvmppc_unmap_pte(struct kvm *kvm, pte_t *pte,
|
|
||||||
unsigned long gpa, unsigned int shift,
|
|
||||||
struct kvm_memory_slot *memslot,
|
|
||||||
unsigned int lpid);
|
|
||||||
extern int kvm_age_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
extern int kvm_age_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
||||||
unsigned long gfn);
|
unsigned long gfn);
|
||||||
extern int kvm_test_age_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
extern int kvm_test_age_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
||||||
@@ -242,7 +239,7 @@ extern kvm_pfn_t kvmppc_gpa_to_pfn(struct kvm_vcpu *vcpu, gpa_t gpa,
|
|||||||
bool writing, bool *writable);
|
bool writing, bool *writable);
|
||||||
extern void kvmppc_add_revmap_chain(struct kvm *kvm, struct revmap_entry *rev,
|
extern void kvmppc_add_revmap_chain(struct kvm *kvm, struct revmap_entry *rev,
|
||||||
unsigned long *rmap, long pte_index, int realmode);
|
unsigned long *rmap, long pte_index, int realmode);
|
||||||
extern void kvmppc_update_dirty_map(struct kvm_memory_slot *memslot,
|
extern void kvmppc_update_dirty_map(const struct kvm_memory_slot *memslot,
|
||||||
unsigned long gfn, unsigned long psize);
|
unsigned long gfn, unsigned long psize);
|
||||||
extern void kvmppc_invalidate_hpte(struct kvm *kvm, __be64 *hptep,
|
extern void kvmppc_invalidate_hpte(struct kvm *kvm, __be64 *hptep,
|
||||||
unsigned long pte_index);
|
unsigned long pte_index);
|
||||||
|
@@ -625,7 +625,7 @@ extern int kvmppc_create_pte(struct kvm *kvm, pgd_t *pgtable, pte_t pte,
|
|||||||
extern void kvmhv_insert_nest_rmap(struct kvm *kvm, unsigned long *rmapp,
|
extern void kvmhv_insert_nest_rmap(struct kvm *kvm, unsigned long *rmapp,
|
||||||
struct rmap_nested **n_rmap);
|
struct rmap_nested **n_rmap);
|
||||||
extern void kvmhv_remove_nest_rmap_range(struct kvm *kvm,
|
extern void kvmhv_remove_nest_rmap_range(struct kvm *kvm,
|
||||||
struct kvm_memory_slot *memslot,
|
const struct kvm_memory_slot *memslot,
|
||||||
unsigned long gpa, unsigned long hpa,
|
unsigned long gpa, unsigned long hpa,
|
||||||
unsigned long nbytes);
|
unsigned long nbytes);
|
||||||
|
|
||||||
|
@@ -284,7 +284,8 @@ static void kvmppc_pmd_free(pmd_t *pmdp)
|
|||||||
|
|
||||||
/* Called with kvm->mmu_lock held */
|
/* Called with kvm->mmu_lock held */
|
||||||
void kvmppc_unmap_pte(struct kvm *kvm, pte_t *pte, unsigned long gpa,
|
void kvmppc_unmap_pte(struct kvm *kvm, pte_t *pte, unsigned long gpa,
|
||||||
unsigned int shift, struct kvm_memory_slot *memslot,
|
unsigned int shift,
|
||||||
|
const struct kvm_memory_slot *memslot,
|
||||||
unsigned int lpid)
|
unsigned int lpid)
|
||||||
|
|
||||||
{
|
{
|
||||||
@@ -861,7 +862,7 @@ int kvmppc_book3s_radix_page_fault(struct kvm_run *run, struct kvm_vcpu *vcpu,
|
|||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Called with kvm->lock held */
|
/* Called with kvm->mmu_lock held */
|
||||||
int kvm_unmap_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
int kvm_unmap_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
||||||
unsigned long gfn)
|
unsigned long gfn)
|
||||||
{
|
{
|
||||||
@@ -876,7 +877,7 @@ int kvm_unmap_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Called with kvm->lock held */
|
/* Called with kvm->mmu_lock held */
|
||||||
int kvm_age_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
int kvm_age_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
||||||
unsigned long gfn)
|
unsigned long gfn)
|
||||||
{
|
{
|
||||||
@@ -895,7 +896,7 @@ int kvm_age_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
|||||||
return ref;
|
return ref;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Called with kvm->lock held */
|
/* Called with kvm->mmu_lock held */
|
||||||
int kvm_test_age_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
int kvm_test_age_radix(struct kvm *kvm, struct kvm_memory_slot *memslot,
|
||||||
unsigned long gfn)
|
unsigned long gfn)
|
||||||
{
|
{
|
||||||
|
@@ -723,7 +723,7 @@ static void kvmhv_remove_nest_rmap_list(struct kvm *kvm, unsigned long *rmapp,
|
|||||||
|
|
||||||
/* called with kvm->mmu_lock held */
|
/* called with kvm->mmu_lock held */
|
||||||
void kvmhv_remove_nest_rmap_range(struct kvm *kvm,
|
void kvmhv_remove_nest_rmap_range(struct kvm *kvm,
|
||||||
struct kvm_memory_slot *memslot,
|
const struct kvm_memory_slot *memslot,
|
||||||
unsigned long gpa, unsigned long hpa,
|
unsigned long gpa, unsigned long hpa,
|
||||||
unsigned long nbytes)
|
unsigned long nbytes)
|
||||||
{
|
{
|
||||||
|
@@ -107,7 +107,7 @@ void kvmppc_add_revmap_chain(struct kvm *kvm, struct revmap_entry *rev,
|
|||||||
EXPORT_SYMBOL_GPL(kvmppc_add_revmap_chain);
|
EXPORT_SYMBOL_GPL(kvmppc_add_revmap_chain);
|
||||||
|
|
||||||
/* Update the dirty bitmap of a memslot */
|
/* Update the dirty bitmap of a memslot */
|
||||||
void kvmppc_update_dirty_map(struct kvm_memory_slot *memslot,
|
void kvmppc_update_dirty_map(const struct kvm_memory_slot *memslot,
|
||||||
unsigned long gfn, unsigned long psize)
|
unsigned long gfn, unsigned long psize)
|
||||||
{
|
{
|
||||||
unsigned long npages;
|
unsigned long npages;
|
||||||
|
Reference in New Issue
Block a user