diff options
author | Jan Beulich <jbeulich@suse.com> | 2012-03-08 17:02:57 +0100 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2012-03-08 17:02:57 +0100 |
commit | cae10a689a5ee104a24dc01131e1d71d4060cc70 (patch) | |
tree | 74c22cad4cd0d7263a4b99cb57e258491c5d6521 /xen/include/xen/xenoprof.h | |
parent | 48844dc21098fa59f1cac06eba31b548f576a446 (diff) | |
download | xen-cae10a689a5ee104a24dc01131e1d71d4060cc70.tar.gz xen-cae10a689a5ee104a24dc01131e1d71d4060cc70.tar.bz2 xen-cae10a689a5ee104a24dc01131e1d71d4060cc70.zip |
oprofile: don't pass around redundant, easily derived arguments
Passing both a struct vcpu pointer and the corresponding struct domain
one is simply pointless, especially when intermediate functions just
forward it without themselves making use of the already obtained value.
Also constify a few function parameters.
Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Keir Fraser <keir@xen.org>
Diffstat (limited to 'xen/include/xen/xenoprof.h')
-rw-r--r-- | xen/include/xen/xenoprof.h | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/xen/include/xen/xenoprof.h b/xen/include/xen/xenoprof.h index e34ad20ab3..7804e69329 100644 --- a/xen/include/xen/xenoprof.h +++ b/xen/include/xen/xenoprof.h @@ -68,8 +68,7 @@ int is_active(struct domain *d); int is_passive(struct domain *d); void free_xenoprof_pages(struct domain *d); -int xenoprof_add_trace(struct domain *d, struct vcpu *v, - uint64_t eip, int mode); +int xenoprof_add_trace(struct vcpu *, uint64_t pc, int mode); #define PMU_OWNER_NONE 0 #define PMU_OWNER_XENOPROF 1 @@ -77,8 +76,7 @@ int xenoprof_add_trace(struct domain *d, struct vcpu *v, int acquire_pmu_ownship(int pmu_ownership); void release_pmu_ownship(int pmu_ownership); -void xenoprof_log_event(struct vcpu *vcpu, - struct cpu_user_regs * regs, uint64_t eip, - int mode, int event); +void xenoprof_log_event(struct vcpu *, const struct cpu_user_regs *, + uint64_t pc, int mode, int event); #endif /* __XEN__XENOPROF_H__ */ |