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

2012-10-01 Thread Jonathan Nieder
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 b...@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-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20121001074243.GA7300@elie.Belkin



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

2012-10-01 Thread Aaron M. Ucko
Jonathan Nieder jrnie...@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/?a...@monk.mit.edu


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



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

2012-10-01 Thread Uwe Kleine-König
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-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20121001152640.gm16...@pengutronix.de



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

2012-10-01 Thread Jonathan Nieder
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 b...@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-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20121001190254.GE16391@elie.Belkin



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

2012-10-01 Thread Jonathan Nieder
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-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20121001194328.GA27024@elie.Belkin



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

2010-02-03 Thread Ingo Kasten
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-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



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

2010-02-02 Thread Aaron M. Ucko
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-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org