2006-08-23 08:15:55

by Stephane Eranian

[permalink] [raw]
Subject: [PATCH 1/18] 2.6.17.9 perfmon2 patch for review: introduction

Hello,

The following series of patches includes the generic perfmon2
subsystem and the support for i386, x86_64, and powerpc. The perfmon2
subsystem also works on MIPS and all Itanium processors. The Itanium support
is not posted because it does not easily accomodate the 100k message
limit of lkml. The powerpc support is still very preliminary.

The patches are relative to 2.6.17.9

I have already posted on the list about this subsystem. I am submitting
today to get reviews and make progress towards getting the subsystem
merged into the mainline kernel.

The patches are split up between common and arch-specific. Each part
is further decomposed into new files and modified files. The generic
code is now split up by functionality to make reading easier.

For each new or modified files, I provide a detailed description of
the changes in each E-mail.

Thanks.

--
-Stephane


2006-08-23 15:27:18

by Christoph Hellwig

[permalink] [raw]
Subject: Re: [PATCH 1/18] 2.6.17.9 perfmon2 patch for review: introduction

On Wed, Aug 23, 2006 at 01:05:52AM -0700, Stephane Eranian wrote:
> Hello,
>
> The following series of patches includes the generic perfmon2
> subsystem and the support for i386, x86_64, and powerpc. The perfmon2
> subsystem also works on MIPS and all Itanium processors. The Itanium support
> is not posted because it does not easily accomodate the 100k message
> limit of lkml. The powerpc support is still very preliminary.
>
> The patches are relative to 2.6.17.9

pleas submit patches always against latest Linus' tree or -mm. 2.6.17.9
is already megabytes of diffs away from mainline.

2006-08-23 15:28:53

by Christoph Hellwig

[permalink] [raw]
Subject: Re: [PATCH 1/18] 2.6.17.9 perfmon2 patch for review: introduction

oh, and please give the patches useful subjects that descript the
patch, e.g. this one should be just:


[PATCH 0/17] perfmon2: introduction

(yes, it's convention to number the introduction 0 and the actual patches
1 to n)

2006-08-23 15:57:25

by Alexey Dobriyan

[permalink] [raw]
Subject: Re: [PATCH 1/18] 2.6.17.9 perfmon2 patch for review: introduction

On Wed, Aug 23, 2006 at 04:28:31PM +0100, Christoph Hellwig wrote:
> oh, and please give the patches useful subjects that descript the
> patch, e.g. this one should be just:
>
>
> [PATCH 0/17] perfmon2: introduction
>
> (yes, it's convention to number the introduction 0 and the actual patches
> 1 to n)

Padding with zeros makes it even more useful:

[PATCH 00/17]
[PATCH 01/17]
...
[PATCH 17/17]

2006-08-23 16:05:21

by Christoph Hellwig

[permalink] [raw]
Subject: Re: [PATCH 1/18] 2.6.17.9 perfmon2 patch for review: introduction

On Wed, Aug 23, 2006 at 07:57:16PM +0400, Alexey Dobriyan wrote:
> On Wed, Aug 23, 2006 at 04:28:31PM +0100, Christoph Hellwig wrote:
> > oh, and please give the patches useful subjects that descript the
> > patch, e.g. this one should be just:
> >
> >
> > [PATCH 0/17] perfmon2: introduction
> >
> > (yes, it's convention to number the introduction 0 and the actual patches
> > 1 to n)
>
> Padding with zeros makes it even more useful:
>
> [PATCH 00/17]
> [PATCH 01/17]
> ...
> [PATCH 17/17]

To be honest I utterly hate that convention, and the double-padded
version [PATCH 001/17] some people use is even worse.

2006-08-23 19:08:49

by Andrew Morton

[permalink] [raw]
Subject: Re: [PATCH 1/18] 2.6.17.9 perfmon2 patch for review: introduction

On Wed, 23 Aug 2006 17:04:58 +0100
Christoph Hellwig <[email protected]> wrote:

> > Padding with zeros makes it even more useful:
> >
> > [PATCH 00/17]
> > [PATCH 01/17]
> > ...
> > [PATCH 17/17]
>
> To be honest I utterly hate that convention

It's so they'll correctly alphasort at the recipient's end.

I doubt if many MUAs do numeric sorting..

2006-08-23 19:11:34

by Christoph Hellwig

[permalink] [raw]
Subject: Re: [PATCH 1/18] 2.6.17.9 perfmon2 patch for review: introduction

On Wed, Aug 23, 2006 at 11:58:57AM -0700, Andrew Morton wrote:
> > > Padding with zeros makes it even more useful:
> > >
> > > [PATCH 00/17]
> > > [PATCH 01/17]
> > > ...
> > > [PATCH 17/17]
> >
> > To be honest I utterly hate that convention
>
> It's so they'll correctly alphasort at the recipient's end.

I suspect most mailers sort by date and not by subject anyway.

At least mine does :)

2006-08-23 20:20:58

by Stephane Eranian

[permalink] [raw]
Subject: Re: [PATCH 1/18] 2.6.17.9 perfmon2 patch for review: introduction

Andrew,
On Wed, Aug 23, 2006 at 11:58:57AM -0700, Andrew Morton wrote:
> On Wed, 23 Aug 2006 17:04:58 +0100
> Christoph Hellwig <[email protected]> wrote:
>
> > > Padding with zeros makes it even more useful:
> > >
> > > [PATCH 00/17]
> > > [PATCH 01/17]
> > > ...
> > > [PATCH 17/17]
> >
> > To be honest I utterly hate that convention
>
> It's so they'll correctly alphasort at the recipient's end.

That makes sense, I'll fix that in my next patch.

--

-Stephane

2006-08-23 20:56:57

by Andrew Morton

[permalink] [raw]
Subject: Re: [PATCH 1/18] 2.6.17.9 perfmon2 patch for review: introduction

On Wed, 23 Aug 2006 20:11:32 +0100
Christoph Hellwig <[email protected]> wrote:

> On Wed, Aug 23, 2006 at 11:58:57AM -0700, Andrew Morton wrote:
> > > > Padding with zeros makes it even more useful:
> > > >
> > > > [PATCH 00/17]
> > > > [PATCH 01/17]
> > > > ...
> > > > [PATCH 17/17]
> > >
> > > To be honest I utterly hate that convention
> >
> > It's so they'll correctly alphasort at the recipient's end.
>
> I suspect most mailers sort by date and not by subject anyway.
>

Emails arrive out-of-order.

I used to have a `sleep 10' in my patch-bomb script to fix that, but half
an hour is too long to be sitting there hoping the DSL stays up, the kernel
stays up and that the familial hair-drier+iron+air-conditioner conspiracy
doesn't strike.

2006-08-24 17:38:23

by Junio C Hamano

[permalink] [raw]
Subject: Re: [PATCH 1/18] 2.6.17.9 perfmon2 patch for review: introduction

Andrew Morton <[email protected]> writes:

> On Wed, 23 Aug 2006 17:04:58 +0100
> Christoph Hellwig <[email protected]> wrote:
>
>> > Padding with zeros makes it even more useful:
>> >
>> > [PATCH 00/17]
>> > [PATCH 01/17]
>> > ...
>> > [PATCH 17/17]
>>
>> To be honest I utterly hate that convention
>
> It's so they'll correctly alphasort at the recipient's end.
>
> I doubt if many MUAs do numeric sorting..

I wonder if 'git-format-patch --numbered' should be updated to
do the zero padding. Right now we don't.

It should be a trivial patch to do if somebody is so inclined
(it is around ll.133 in log-tree.c).