=== JanC_ is now known as JanC [06:38] hi, I have a question regarding the state of perf in the xenial ubuntu kernel source [06:39] While looking for the append option of the `perf record` command I noticed that this is missing in the current ubuntu kernel source of perf (I'm running 4.4.0-78-generic #99~14.04.2-Ubuntu SMP Thu Apr 27 18:49:46 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux) [06:41] This is surprising to me since the official sources seam to have this since version 2.6.31 https://github.com/torvalds/linux/commit/abaff32a03e26e5d6674cb2a26ad882efe7493a3 [06:45] Is there anyone at the ubuntu kernel team especially responsible/taking care of maintaining `perf` to ask about why this is and whats the general state of perf in ubuntu? [06:56] doob, hmmm, well noone speific looks after it. if it has an issue that needs investigating do file a bug against the kernel [06:56] doob, as that is produced as part of the kernel build [06:57] I was just wondering if it has just been forgotten somehow or if there is reason to patch away or not include this compared to the main kernel tree [06:58] ok [06:58] doob, we do not carry any patches for perf [06:59] but do you compare/sync with the upstream kernel tree at some point regulary or does it mean you never ever get upstream code for perf as long as nobody files a bug report? [07:00] I'm not sure if I got you right with "we do not carry any patches for perf" [07:02] doob, perf is kernel version specific and carried without modification as far as i recall [07:03] Ok. This is what I was always thinking how it is. This is why I'm supprised to not find the code in the ubuntu tree that is in the upstream tree since 2.6.31. [07:05] doob, the source for 4.4 has the append option [07:08] Am I looking at the wrong place? http://kernel.ubuntu.com/git/ubuntu/ubuntu-xenial.git/tree/tools/perf/builtin-record.c [07:14] doob, oh no i am wrong, that is an append option in another command [07:14] doob, the -A was removed a long time ago [07:14] commit 563aecb2e63a16f86c7daabfdcfee23f3e7c5955 [07:14] Author: Jiri Olsa [07:14] Date: Wed Jun 5 13:35:06 2013 +0200 [07:14] perf record: Remove -A/--append option [07:14] It's no longer working and needed. [07:14] Quite straightforward discussion/vote was in here: [07:14] http://marc.info/?t=137028288300004&r=1&w=2 [07:14] (the commit's words) [07:16] Ok. Thx. In the meantime I also noted I did a mistake. I think I grabbed some very old commit message from GH and was misleaded by the branch information saying that it is in all the current branches. But of course the commit will stay there for ever even if the code has been removed. Sorry for the confusion. [07:18] np === klebers_ is now known as klebers === klebers_ is now known as klebers === dannf` is now known as dannf === jdstrand_ is now known as jdstrand === jdstrand is now known as jdstrand_ === jdstrand_ is now known as jdstrand === JanC_ is now known as JanC [22:05] sforshee: I was able to test the package for bug1686815 for the bluetooth issue. It fixes the bug. [23:23] sforshee: sorry for the dely on this, I finally tracked down the memory corruption, this passes the regression tests, except for long_path which is deliberately broken atm, and will have to be fixed in some follow on patches [23:23] http://paste.ubuntu.com/24583768/ [23:24] sforshee: I have included the fixes that are on 4.12, so it should be a clean rebase to the 4.12 kernel