2006-09-14 03:40:34

by Mathieu Desnoyers

[permalink] [raw]
Subject: [PATCH 1/11] LTTng-core 0.5.108 : build

1- LTTng menu options and Makefiles
(do not enable blktrace for now : kernel/relay.o is disabled)
patch-2.6.17-lttng-core-0.5.108-build.diff


OpenPGP public key: http://krystal.dyndns.org:8080/key/compudj.gpg
Key fingerprint: 8CD5 52C3 8E3C 4140 715F BA06 3F25 A8FE 3BAE 9A68


Attachments:
(No filename) (296.00 B)
patch-2.6.17-lttng-core-0.5.108-build.diff (7.98 kB)
Download all attachments

2006-09-14 07:49:34

by Martin Waitz

[permalink] [raw]
Subject: Re: [PATCH 1/11] LTTng-core 0.5.108 : build

hoi :)

On Wed, Sep 13, 2006 at 11:40:30PM -0400, Mathieu Desnoyers wrote:
> 1- LTTng menu options and Makefiles

adding Makefiles before the needed .c files breaks bisecting.

> (do not enable blktrace for now : kernel/relay.o is disabled)

If kernel/relay.c becomes obsolete, then perhaps this should be a separate
patch, removing it entirely.

--
Martin Waitz


Attachments:
(No filename) (364.00 B)
signature.asc (189.00 B)
Digital signature
Download all attachments

2006-09-14 09:54:47

by Paul Mundt

[permalink] [raw]
Subject: Re: [PATCH 1/11] LTTng-core 0.5.108 : build

On Wed, Sep 13, 2006 at 11:40:30PM -0400, Mathieu Desnoyers wrote:
> 1- LTTng menu options and Makefiles
> (do not enable blktrace for now : kernel/relay.o is disabled)

And the rationale behind this would be..?

Apparently you've also decided to try and add the file system back in,
what exactly is so "special" about LTTng that it can't use CONFIG_RELAY
in conjunction with debugfs, in the same way that blktrace does now?

2006-09-14 15:52:05

by Mathieu Desnoyers

[permalink] [raw]
Subject: Re: [PATCH 1/11] LTTng-core 0.5.108 : build

* Martin Waitz ([email protected]) wrote:
> hoi :)
>
> On Wed, Sep 13, 2006 at 11:40:30PM -0400, Mathieu Desnoyers wrote:
> > 1- LTTng menu options and Makefiles
>
> adding Makefiles before the needed .c files breaks bisecting.
>
Ok, next time I will send the patches in a different order.
(meanwhile, you can apply this patch at the end)

> > (do not enable blktrace for now : kernel/relay.o is disabled)
>
> If kernel/relay.c becomes obsolete, then perhaps this should be a separate
> patch, removing it entirely.
>

No, it is not. It's a temporary solution only used because relayfs has been such
a moving target lately that I decided to wait for things to settle down before
switching. I plan to switch to the new relay.o and use DebugFS for filesystem. I
just haven't done the change yet.

Mathieu


> --
> Martin Waitz


OpenPGP public key: http://krystal.dyndns.org:8080/key/compudj.gpg
Key fingerprint: 8CD5 52C3 8E3C 4140 715F BA06 3F25 A8FE 3BAE 9A68


Attachments:
(No filename) (988.00 B)
signature.asc (189.00 B)
Digital signature
Download all attachments