Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Apache: Dev

dtrace patches.

 

 

Apache dev RSS feed   Index | Next | Previous | View Threaded


jesus at omniti

Aug 25, 2008, 5:51 PM

Post #1 of 6 (351 views)
Permalink
dtrace patches.

This is just a followup to a previous mail:

http://mail-archives.apache.org/mod_mbox/httpd-dev/200805.mbox/%3C6AFBCFE8-4CCA-4A02-8A43-F9170689695D[at]omniti.com%3E

The apr-utils include mod got immediate uptake. Thanks Paul!

I didn't see much progress on the rest of it. How do I got about
guiding through the patch acceptance process? Or am I dense and it
was indeed committed and I'm just looking in the wrong place.

Thanks!

Keep me on the CC, I'm not on list.

--
Theo Schlossnagle
Esoteric Curio -- http://lethargy.org/
OmniTI Computer Consulting, Inc. -- http://omniti.com/


nick at webthing

Aug 26, 2008, 9:25 AM

Post #2 of 6 (327 views)
Permalink
Re: dtrace patches. [In reply to]

On Mon, 25 Aug 2008 20:51:22 -0400
Theo Schlossnagle <jesus[at]omniti.com> wrote:

> This is just a followup to a previous mail:
>
> http://mail-archives.apache.org/mod_mbox/httpd-dev/200805.mbox/%3C6AFBCFE8-4CCA-4A02-8A43-F9170689695D[at]omniti.com%3E
>
> The apr-utils include mod got immediate uptake. Thanks Paul!
>
> I didn't see much progress on the rest of it. How do I got about
> guiding through the patch acceptance process?

Sounds good to me.

The limiting factor is (as ever) round tuits amongst developers.
The more help you can provide in understanding and reviewing
the patches, the lower the barrier to those round tuits.


--
Nick Kew

Application Development with Apache - the Apache Modules Book
http://www.apachetutor.org/


rahul at Sun

Sep 8, 2008, 6:42 AM

Post #3 of 6 (276 views)
Permalink
Re: dtrace patches. [In reply to]

Hello,
- discussion on dtrace patches by Theo seems to have wound down
lately, with only the build steps being problematic.

Here is the least that we could do while the conf|build steps are
figured out. (so that more probes can be added in the meantime.)

1) we need apache_probes.h to be generated from apache_probes.d in all
platforms so that the macros AP*ENTRY|RETURN are defined. (For those
without dtrace, it needs to be generated by scripts.)
the changes in ap_config.h depends only on this.

2) ap.d, and apache_probes.d are independent, and could be added to
the trunk or left as patches to be integrated once the conf|build steps
are finalized..

3) The same with changes in each of *.c files, perhaps bugs(enh) with
a 'Dtrace' keyword? so that new entries can be added as patches by
others.

Once this is done, we can split the effort into well defined areas, and
continue in each independently.


| This is just a followup to a previous mail:
|
| http://mail-archives.apache.org/mod_mbox/httpd-dev/200805.mbox/%3C6AFBCFE8-4CCA-4A02-8A43-F9170689695D[at]omniti.com%3E
|
| The apr-utils include mod got immediate uptake. Thanks Paul!
|
| I didn't see much progress on the rest of it. How do I got about
| guiding through the patch acceptance process? Or am I dense and it
| was indeed committed and I'm just looking in the wrong place.


rahul
--
1. e4 _


rahul at Sun

Sep 19, 2008, 1:45 AM

Post #4 of 6 (223 views)
Permalink
Re: dtrace patches. [In reply to]

Pinging the list once again,

If atleast the below goes in, it would be possible to submit patches
to increase the coverage independently of the build patches. (I would
like to add more coverage in other interesting modules too.)

| - discussion on dtrace patches by Theo seems to have wound down
| lately, with only the build steps being problematic.
|
| Here is the least that we could do while the conf|build steps are
| figured out. (so that more probes can be added in the meantime.)
|
| 1) we need apache_probes.h to be generated from apache_probes.d in all
| platforms so that the macros AP*ENTRY|RETURN are defined. (For those
| without dtrace, it needs to be generated by scripts.)
| the changes in ap_config.h depends only on this.
|
| 2) ap.d, and apache_probes.d are independent, and could be added to
| the trunk or left as patches to be integrated once the conf|build steps
| are finalized..
|
| 3) The same with changes in each of *.c files, perhaps bugs(enh) with
| a 'Dtrace' keyword? so that new entries can be added as patches by
| others.
|
| Once this is done, we can split the effort into well defined areas, and
| continue in each independently.
|
|
| | This is just a followup to a previous mail:
| |
| | http://mail-archives.apache.org/mod_mbox/httpd-dev/200805.mbox/%3C6AFBCFE8-4CCA-4A02-8A43-F9170689695D[at]omniti.com%3E
| |
| | The apr-utils include mod got immediate uptake. Thanks Paul!
| |
| | I didn't see much progress on the rest of it. How do I got about
| | guiding through the patch acceptance process? Or am I dense and it
| | was indeed committed and I'm just looking in the wrong place.


rahul
--
1. e4 _


chip at force-elite

Dec 4, 2008, 7:52 PM

Post #5 of 6 (79 views)
Permalink
Re: dtrace patches. [In reply to]

Theo Schlossnagle wrote:
> This is just a followup to a previous mail:
>
> http://mail-archives.apache.org/mod_mbox/httpd-dev/200805.mbox/%3C6AFBCFE8-4CCA-4A02-8A43-F9170689695D[at]omniti.com%3E
>
>
> The apr-utils include mod got immediate uptake. Thanks Paul!
>
> I didn't see much progress on the rest of it. How do I got about
> guiding through the patch acceptance process? Or am I dense and it was
> indeed committed and I'm just looking in the wrong place.
>
> Thanks!
>

I've committed most of your patch to trunk now.

I did not however include the bits modifying the build, they seemed a
little too creative and didn't work right on my mac -- I'll try to get
something fixed up soon, but patches welcome.

(This means it doesn't build by default, and it fails to compile right
if you enable it, but I think I can get it fixed before cutting the
2.3.0-alpha this weekend).

Thanks,

Paul


jesus at omniti

Dec 4, 2008, 8:19 PM

Post #6 of 6 (78 views)
Permalink
Re: dtrace patches. [In reply to]

On Dec 4, 2008, at 10:52 PM, Paul Querna wrote:

> Theo Schlossnagle wrote:
>> This is just a followup to a previous mail:
>> http://mail-archives.apache.org/mod_mbox/httpd-dev/200805.mbox/%3C6AFBCFE8-4CCA-4A02-8A43-F9170689695D[at]omniti.com%3E
>> The apr-utils include mod got immediate uptake. Thanks Paul!
>> I didn't see much progress on the rest of it. How do I got about
>> guiding through the patch acceptance process? Or am I dense and it
>> was indeed committed and I'm just looking in the wrong place.
>> Thanks!
>
> I've committed most of your patch to trunk now.
>
> I did not however include the bits modifying the build, they seemed
> a little too creative and didn't work right on my mac -- I'll try to
> get something fixed up soon, but patches welcome.
>
> (This means it doesn't build by default, and it fails to compile
> right if you enable it, but I think I can get it fixed before
> cutting the 2.3.0-alpha this weekend).
>
> Thanks,
>
> Paul


Awesome news! Deep thanks for your time investment on this! Some
cool possibilities are coming with that changeset.

--
Theo Schlossnagle
Esoteric Curio -- http://lethargy.org/
OmniTI Computer Consulting, Inc. -- http://omniti.com/

Apache dev RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact lists@gossamer-threads.com
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.