perf enahancements for ppc64

Bug #1521678 reported by bugproxy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
linux (Ubuntu)
Fix Released
Undecided
Tim Gardner
Xenial
Fix Released
Undecided
Tim Gardner

Bug Description

perf kvm stat record/report is not supported on powerpc currently. With perf kvm stat record/report analysis of kvm exit events and hypervisor calls can be done. The statistics for these events can be shown individually for each running VM in the host and can be very useful in giving an idea of the performance of a VM under a certain workload.

This can show how many number of KVM exits have happened during a certain period of time. Along with the total number of exits, all the exits are grouped according to their specific exit reasons. Also, frequency for individual exit reasons is shown. Same can be done with hypervisor calls too.

Updating this with two more features :
- perf intr regs sampling support. Patches are currently in v3.
https://lkml.org/lkml/2015/11/3/25

- Guest symbol resolution:
To enable perf kvm record for powerpc, we need to sample the instruction pointer from kvm guest_exit tracepoint. The patches for this feature are currently being sent.

bugproxy (bugproxy)
tags: added: architecture-ppc64 bugnameltc-133242 severity-medium targetmilestone-inin1604
Changed in ubuntu:
assignee: nobody → Taco Screen team (taco-screen-team)
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better. It seems that your bug report is not filed about a specific source package though, rather it is just filed against Ubuntu in general. It is important that bug reports be filed about source packages so that people interested in the package can find the bugs about it. You can find some hints about determining what package your bug might be about at https://wiki.ubuntu.com/Bugs/FindRightPackage. You might also ask for help in the #ubuntu-bugs irc channel on Freenode.

To change the source package that this bug is filed about visit https://bugs.launchpad.net/ubuntu/+bug/1521678/+editstatus and add the package name in the text box next to the word Package.

[This is an automated message. I apologize if it reached you inappropriately; please just reply to this message indicating so.]

tags: added: bot-comment
Steve Langasek (vorlon)
affects: ubuntu → linux (Ubuntu)
Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2016-02-02 10:19 EDT-------
Patches are still under review.

Revision history for this message
bugproxy (bugproxy) wrote :

------- Comment From <email address hidden> 2016-02-08 03:37 EDT-------
Patches for guest exit statistics (perf kvm stat record/report) have made into upstream :
https://lists.ozlabs.org/pipermail/linuxppc-dev/2016-January/138563.html

Revision history for this message
Breno Leitão (breno-leitao) wrote :

I understand that this bug is ready for Canonical consumption.

Revision history for this message
Tim Gardner (timg-tpi) wrote :

Breno - it is not clear to me to which patches you refer. I found these patches in linux-next:

perf kvm/{x86,s390}: Remove dependency on uapi/kvm_perf.h
perf kvm/{x86,s390}: Remove const from kvm_events_tp
perf kvm/powerpc: Port perf kvm stat to powerpc
perf kvm/powerpc: Add support for HCALL reasons

Changed in linux (Ubuntu Xenial):
assignee: Taco Screen team (taco-screen-team) → Tim Gardner (timg-tpi)
status: New → In Progress
Revision history for this message
bugproxy (bugproxy) wrote :

------- Comment From <email address hidden> 2016-02-08 11:58 EDT-------
> Breno - it is not clear to me to which patches you refer. I found these
> patches in linux-next:
Hello Tim, Sorry, the patches required to be included is, in fact, confusing.

> perf kvm/{x86,s390}: Remove dependency on uapi/kvm_perf.h
> perf kvm/{x86,s390}: Remove const from kvm_events_tp
> perf kvm/powerpc: Port perf kvm stat to powerpc
> perf kvm/powerpc: Add support for HCALL reasons

I understand that all of them might be part of the same feature (fix perf kvm stat), but I would prefer Hemant to confirm.

Revision history for this message
bugproxy (bugproxy) wrote :

------- Comment From <email address hidden> 2016-02-09 00:21 EDT-------
(In reply to comment #14)
> > Breno - it is not clear to me to which patches you refer. I found these
> > patches in linux-next:
> Hello Tim, Sorry, the patches required to be included is, in fact, confusing.
>
> > perf kvm/{x86,s390}: Remove dependency on uapi/kvm_perf.h
> > perf kvm/{x86,s390}: Remove const from kvm_events_tp
> > perf kvm/powerpc: Port perf kvm stat to powerpc
> > perf kvm/powerpc: Add support for HCALL reasons
>
> I understand that all of them might be part of the same feature (fix perf
> kvm stat), but I would prefer Hemant to confirm.

The above patches are needed for this feauture (all of them). These commits are from linux-next :
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git

78e6c39b231a8e31e193534fdbe29291b7fd8f37 perf kvm/powerpc: Add support for HCALL reasons
066d3593e1b14690dc1131d50cacbb0b7eb3f160 perf kvm/powerpc: Port perf kvm stat to powerpc
48deaa74fcdad516a94fe38a4af706747d9e4745 perf kvm/{x86,s390}: Remove const from kvm_events_tp
162607ea20fafb4a76234ebe4314cd733345482e perf kvm/{x86,s390}: Remove dependency on uapi/kvm_perf.h

Revision history for this message
Tim Gardner (timg-tpi) wrote :

Applied

Changed in linux (Ubuntu Xenial):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.4 KiB)

This bug was fixed in the package linux - 4.4.0-6.21

---------------
linux (4.4.0-6.21) xenial; urgency=low

  [ Tim Gardner ]

  * Release Tracking Bug
    - LP: #1546283

  * Naples/Zen, NTB Driver (LP: #1542071)
    - [Config] CONFIG_NTB_AMD=m
    - NTB: Add support for AMD PCI-Express Non-Transparent Bridge

  * [Hyper-V] kernel panic occurs when installing Ubuntu Server x32 (LP: #1495983)
    - SAUCE: storvsc: use small sg_tablesize on x86

  * Enable arm64 emulation of removed ARMv7 instructions (LP: #1545542)
    - [Config] CONFIG_ARMV8_DEPRECATED=y

  * Surelock-GA2:kernel panic/ exception @ pcibios_set_pcie_reset_state+0x118/0x280 + cxl_reset+0x5c/0xc0 (LP: #1545037)
    - powerpc/eeh: Fix stale cached primary bus

  * Miscellaneous Ubuntu changes
    - SAUCE: fs: Add user namesapace member to struct super_block
    - SAUCE: fs: Limit file caps to the user namespace of the super block
    - SAUCE: Smack: Add support for unprivileged mounts from user namespaces
    - SAUCE: block_dev: Support checking inode permissions in lookup_bdev()
    - SAUCE: block_dev: Check permissions towards block device inode when mounting
    - SAUCE: fs: Treat foreign mounts as nosuid
    - SAUCE: selinux: Add support for unprivileged mounts from user namespaces
    - SAUCE: userns: Replace in_userns with current_in_userns
    - SAUCE: Smack: Handle labels consistently in untrusted mounts
    - SAUCE: fs: Check for invalid i_uid in may_follow_link()
    - SAUCE: cred: Reject inodes with invalid ids in set_create_file_as()
    - SAUCE: fs: Refuse uid/gid changes which don't map into s_user_ns
    - SAUCE: fs: Update posix_acl support to handle user namespace mounts
    - SAUCE: fs: Ensure the mounter of a filesystem is privileged towards its inodes
    - SAUCE: fs: Don't remove suid for CAP_FSETID in s_user_ns
    - SAUCE: fs: Allow superblock owner to access do_remount_sb()
    - SAUCE: capabilities: Allow privileged user in s_user_ns to set security.* xattrs
    - SAUCE: fuse: Add support for pid namespaces
    - SAUCE: fuse: Support fuse filesystems outside of init_user_ns
    - SAUCE: fuse: Restrict allow_other to the superblock's namespace or a descendant
    - SAUCE: fuse: Allow user namespace mounts
    - SAUCE: mtd: Check permissions towards mtd block device inode when mounting
    - SAUCE: fs: Update i_[ug]id_(read|write) to translate relative to s_user_ns
    - SAUCE: quota: Convert ids relative to s_user_ns
    - SAUCE: evm: Translate user/group ids relative to s_user_ns when computing HMAC
    - SAUCE: fs: Allow CAP_SYS_ADMIN in s_user_ns to freeze and thaw filesystems
    - SAUCE: quota: Treat superblock owner as privilged
    - SAUCE: ima/evm: Allow root in s_user_ns to set xattrs
    - SAUCE: block_dev: Forbid unprivileged mounting when device is opened for writing
    - SAUCE: ext4: Add support for unprivileged mounts from user namespaces
    - SAUCE: ext4: Add module parameter to enable user namespace mounts
    - SAUCE: fuse: Add module parameter to enable user namespace mounts

  * Miscellaneous upstream changes
    - megaraid: Fix possible NULL pointer deference in mraid_mm_ioctl
    - libahci: Implement the capability to override th...

Read more...

Changed in linux (Ubuntu Xenial):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.