Re: [PATCH] systemtap: Patches for version 2.1

[Saul Wold <sgw@linux.intel.com>]

On 08/28/2014 03:23 AM, Muzaffar Mahmood wrote:
> From: Mikhail Durnev <Mikhail_Durnev@mentor.com>
>
Please tell us why?  If you up-revving, does it make more sense to 
change where we are pointing the git hash?

> Signed-off-by: Mikhail Durnev <mikhail_durnev@mentor.com>
> Signed-off-by: Muzaffar Mahmood <muzaffar_mahmood@mentor.com>
> ---
>   .../systemtap/systemtap/system_map_location.patch  |   13 +++++++++++++
>   meta/recipes-kernel/systemtap/systemtap_git.inc    |    1 +
>   2 files changed, 14 insertions(+)
>   create mode 100644 meta/recipes-kernel/systemtap/systemtap/system_map_location.patch
>
> diff --git a/meta/recipes-kernel/systemtap/systemtap/system_map_location.patch b/meta/recipes-kernel/systemtap/systemtap/system_map_location.patch
> new file mode 100644
> index 0000000..cd4f19e
> --- /dev/null
> +++ b/meta/recipes-kernel/systemtap/systemtap/system_map_location.patch

As with the other patch, this patch file is missing Upstream-Status and 
Signed-off-by.

See the http://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines

Thanks
	Sau!

> @@ -0,0 +1,13 @@
> +Index: git/session.cxx
> +===================================================================
> +--- git.orig/session.cxx
> ++++ git/session.cxx
> +@@ -1634,7 +1634,7 @@ systemtap_session::parse_kernel_function
> + 	clog << _F("Kernel symbol table %s unavailable, (%s)",
> + 		   system_map_path.c_str(), strerror(errno)) << endl;
> +
> +-      system_map_path = "/boot/System.map-" + kernel_release;
> ++      system_map_path = kernel_build_tree + "/System.map-" + kernel_release;
> +       system_map.clear();
> +       system_map.open(system_map_path.c_str(), ifstream::in);
> +       if (! system_map.is_open())
> diff --git a/meta/recipes-kernel/systemtap/systemtap_git.inc b/meta/recipes-kernel/systemtap/systemtap_git.inc
> index 3e1e80a..33aa136 100644
> --- a/meta/recipes-kernel/systemtap/systemtap_git.inc
> +++ b/meta/recipes-kernel/systemtap/systemtap_git.inc
> @@ -6,6 +6,7 @@ PV = "2.5+git${SRCPV}"
>   SRC_URI = "git://sourceware.org/git/systemtap.git \
>              file://docproc-build-fix.patch \
>              file://obsolete_automake_macros.patch \
> +           file://system_map_location.patch \
>              file://tapset-linux-sendfile-syscall.patch \
>             "
>
>