Bug#568165: linux-kbuild-2.6.32: please ship recordmcount.pl
 

Package: linux-kbuild-2.6.32
Version: 2.6.32-1
Severity: important

Enabling CONFIG_FTRACE_MCOUNT_RECORD in the kernel's configuration calls
for the addition of recordmcount.pl to the set of scripts to install (as
listed in scripts/Makefile, AFAICT); could you please do so?

Thanks!

-- System Information:
Debian Release: squeeze/sid
APT prefers unstable
APT policy: (500, 'unstable'), (500, 'testing'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-trunk-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages linux-kbuild-2.6.32 depends on:
ii libc6 2.10.2-5 Embedded GNU C Library: Shared lib

linux-kbuild-2.6.32 recommends no packages.

linux-kbuild-2.6.32 suggests no packages.

-- no debconf information



--
To UNSUBSCRIBE, email to debian-kernel-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org

Bug#568165: linux-kbuild-2.6.32: please ship recordmcount.pl
 

As a workaround in the meantime you could extract recordmcount.pl from
the /script directory of /usr/src/linux-source-2.6.32.tar.bz2 (if you
installed the source package) and copy it to
/usr/src/linux-kbuild-2.6.32/scripts. Afterwards you can build your
kernel packages, at least it worked for me for the nvidia-kernel-source
package.
Don't know if the added file has to be removed by hand before upgrading
the linux-kbuild package in order not to entangle your package manager.


Regards
Ingo



--
To UNSUBSCRIBE, email to debian-kernel-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org

Bug#568165: linux-kbuild-2.6.32: please ship recordmcount.pl
 

reassign 568165 src:linux-tools linux-kbuild-2.6/2.6.32-1
found 568165 linux-tools/3.5-1~experimental.1 , linux-tools/3.2.17-1
tags 568165 + patch
quit

Hi,

In 2010, Aaron M. Ucko wrote:

> Enabling CONFIG_FTRACE_MCOUNT_RECORD in the kernel's configuration calls
> for the addition of recordmcount.pl to the set of scripts to install (as
> listed in scripts/Makefile, AFAICT); could you please do so?

Thanks, and sorry for the long silence. In
/usr/lib/linux-kbuild-3.2/scripts/Makefile.build I see:

define rule_cc_o_c
$(call echo-cmd,checksrc) $(cmd_checksrc)
$(call echo-cmd,cc_o_c) $(cmd_cc_o_c);
$(cmd_modversions)
$(call echo-cmd,record_mcount)
[...]

Does the following work?

Index: debian/changelog
================================================== =================
--- debian/changelog (révision 19403)
+++ debian/changelog (copie de travail)
@@ -1,9 +1,13 @@
linux-tools (3.5-1~experimental.2) UNRELEASED; urgency=low

+ [ Ben Hutchings ]
* genorig: Accept xz-compressed upstream tarballs and patches, and
generate an xz-compressed orig tarball, thanks to Sedat Dilek
* source: Enable xz-compression for debian directory tarball

+ [ Jonathan Nieder ]
+ * Include recordmcount.pl (Closes: #568165)
+
-- Ben Hutchings <ben@decadent.org.uk> Tue, 21 Aug 2012 05:05:42 +0100

linux-tools (3.5-1~experimental.1) experimental; urgency=low
Index: debian/build/scripts/Makefile
================================================== =================
--- debian/build/scripts/Makefile (révision 19403)
+++ debian/build/scripts/Makefile (copie de travail)
@@ -32,6 +32,7 @@
mkuboot.sh
namespace.pl
patch-kernel
+ recordmcount.pl
setlocalversion
ver_linux



--
To UNSUBSCRIBE, email to debian-kernel-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
Archive: http://lists.debian.org/20121001074243.GA7300@elie.Belkin

Bug#568165: linux-kbuild-2.6.32: please ship recordmcount.pl
 

Jonathan Nieder <jrnieder@gmail.com> writes:

> Thanks, and sorry for the long silence.

No problem; truth be told, the bug hasn't affected me in a while (due to
changes in either the kernel's configuration or OpenAFS's build system,
I don't remember which). That said, I can confirm that your patch does
indeed add recordmcount.pl to linux-kbuild-3.5, thanks.

--
Aaron M. Ucko, KB1CJC (amu at alum.mit.edu, ucko at debian.org)
http://www.mit.edu/~amu/ | http://stuff.mit.edu/cgi/finger/?amu@monk.mit.edu


--
To UNSUBSCRIBE, email to debian-kernel-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
Archive: udlvceunshe.fsf@dr-wily.mit.edu">http://lists.debian.org/udlvceunshe.fsf@dr-wily.mit.edu

Bug#568165: linux-kbuild-2.6.32: please ship recordmcount.pl
 

Hello,

On Mon, Oct 01, 2012 at 12:42:43AM -0700, Jonathan Nieder wrote:
> reassign 568165 src:linux-tools linux-kbuild-2.6/2.6.32-1
> found 568165 linux-tools/3.5-1~experimental.1 , linux-tools/3.2.17-1
> tags 568165 + patch
> quit
>
> Hi,
>
> In 2010, Aaron M. Ucko wrote:
>
> > Enabling CONFIG_FTRACE_MCOUNT_RECORD in the kernel's configuration calls
> > for the addition of recordmcount.pl to the set of scripts to install (as
> > listed in scripts/Makefile, AFAICT); could you please do so?
>
> Thanks, and sorry for the long silence. In
> /usr/lib/linux-kbuild-3.2/scripts/Makefile.build I see:
>
> define rule_cc_o_c
> $(call echo-cmd,checksrc) $(cmd_checksrc)
> $(call echo-cmd,cc_o_c) $(cmd_cc_o_c);
> $(cmd_modversions)
> $(call echo-cmd,record_mcount)
> [...]
>
> Does the following work?
>
> Index: debian/changelog
> ================================================== =================
> --- debian/changelog (révision 19403)
> +++ debian/changelog (copie de travail)
> @@ -1,9 +1,13 @@
> linux-tools (3.5-1~experimental.2) UNRELEASED; urgency=low
>
> + [ Ben Hutchings ]
> * genorig: Accept xz-compressed upstream tarballs and patches, and
> generate an xz-compressed orig tarball, thanks to Sedat Dilek
> * source: Enable xz-compression for debian directory tarball
>
> + [ Jonathan Nieder ]
> + * Include recordmcount.pl (Closes: #568165)
> +
Most archs use the C-version of recordmcount nowadays. I don't know
about 3.2, but for new versions we'd need a binary and rewrite
Makefile.build IIRC.

Uwe

--
Pengutronix e.K. | Uwe Kleine-König |
Industrial Linux Solutions | http://www.pengutronix.de/ |


--
To UNSUBSCRIBE, email to debian-kernel-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
Archive: 20121001152640.GM16606@pengutronix.de">http://lists.debian.org/20121001152640.GM16606@pengutronix.de

Bug#568165: linux-kbuild-2.6.32: please ship recordmcount.pl
 

Uwe Kleine-König wrote:

> Most archs use the C-version of recordmcount nowadays. I don't know
> about 3.2, but for new versions we'd need a binary and rewrite
> Makefile.build IIRC.

The magic happens in the toplevel Makefile, which is not installed in
kbuild:

| ifdef CONFIG_FUNCTION_TRACER
| KBUILD_CFLAGS += -pg
| ifdef CONFIG_DYNAMIC_FTRACE
| ifdef CONFIG_HAVE_C_RECORDMCOUNT
| BUILD_C_RECORDMCOUNT := y
| export BUILD_C_RECORDMCOUNT
| endif
| endif
| endif

In scripts/Makefile.build the behavior depends on the
BUILD_C_RECORDMOUNT setting.

| ifdef CONFIG_FTRACE_MCOUNT_RECORD
[...]
| cmd_record_mcount =
| if [ "$(findstring -pg,$(_c_flags))" = "-pg" ]; then
| $(sub_cmd_record_mcount)
| fi;
| endif

So the patch I sent might happen to work. Here's a presumably better
patch (still untested).

Index: debian/changelog
================================================== =================
--- debian/changelog (révision 19403)
+++ debian/changelog (copie de travail)
@@ -1,9 +1,13 @@
linux-tools (3.5-1~experimental.2) UNRELEASED; urgency=low

+ [ Ben Hutchings ]
* genorig: Accept xz-compressed upstream tarballs and patches, and
generate an xz-compressed orig tarball, thanks to Sedat Dilek
* source: Enable xz-compression for debian directory tarball

+ [ Jonathan Nieder ]
+ * Include recordmcount and recordmcount.pl (Closes: #568165)
+
-- Ben Hutchings <ben@decadent.org.uk> Tue, 21 Aug 2012 05:05:42 +0100

linux-tools (3.5-1~experimental.1) experimental; urgency=low
Index: debian/build/scripts/Makefile
================================================== =================
--- debian/build/scripts/Makefile (révision 19403)
+++ debian/build/scripts/Makefile (copie de travail)
@@ -2,7 +2,8 @@
bin2c
conmakehash
kallsyms
- pnmtologo
+ pnmtologo
+ recordmcount

DATA =
Kbuild.include
@@ -32,6 +33,7 @@
mkuboot.sh
namespace.pl
patch-kernel
+ recordmcount.pl
setlocalversion
ver_linux



--
To UNSUBSCRIBE, email to debian-kernel-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
Archive: http://lists.debian.org/20121001190254.GE16391@elie.Belkin

Bug#568165: linux-kbuild-2.6.32: please ship recordmcount.pl
 

clone 568165 -1
reassign -1 src:linux-kbuild-2.6 2.6.32-1
tags -1 + upstream
severity 568165 normal
quit

Aaron M. Ucko wrote:

> No problem; truth be told, the bug hasn't affected me in a while (due to
> changes in either the kernel's configuration or OpenAFS's build system,
> I don't remember which).

Probably due to v2.6.38-rc1~2^2~57^2~2 (ftrace: Speed up recordmcount,
2010-10-28).


--
To UNSUBSCRIBE, email to debian-kernel-REQUEST@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
Archive: http://lists.debian.org/20121001194328.GA27024@elie.Belkin