Erlang Mailing Lists

Author Message

<  Erlang patches mailing list  ~  a patch for "OldHeap unsed"

Guest
Posted: Thu Apr 08, 2010 3:48 am Reply with quote
Guest
Hi,

system_info(procs) is currently reporting a same value for both
"OldHeap" and "OldHeap unsed".
This is a patch for erts/emulator/beam/break.c to report
"OLD_HEND(p) - OLD_HTOP(p)"
instead of
"OLD_HEND(p) - OLD_HEAP(p)"
for "OldHeap unsed".

git fetch git://github.com/kinogmt/otp.git old_heap_unused

kino

________________________________________________________________
erlang-patches (at) erlang.org mailing list.
See http://www.erlang.org/faq.html
To unsubscribe; mailto:erlang-patches-unsubscribe@erlang.org

Post received from mailinglist
Guest
Posted: Thu Apr 08, 2010 3:48 am Reply with quote
Guest
Hi,

system_info(procs) is currently reporting a same value for both
"OldHeap" and "OldHeap unsed".
This is a patch for erts/emulator/beam/break.c to report
"OLD_HEND(p) - OLD_HTOP(p)"
instead of
"OLD_HEND(p) - OLD_HEAP(p)"
for "OldHeap unsed".

git fetch git://github.com/kinogmt/otp.git old_heap_unused

kino

________________________________________________________________
erlang-patches (at) erlang.org mailing list.
See http://www.erlang.org/faq.html
To unsubscribe; mailto:erlang-patches-unsubscribe@erlang.org

Post received from mailinglist
Guest
Posted: Thu Apr 08, 2010 7:54 am Reply with quote
Guest
On Thu, Apr 8, 2010 at 5:48 AM, satoshi kinoshita <kino@geminimobile.com> wrote:

> system_info(procs) is currently reporting a same value for both
> "OldHeap" and "OldHeap unsed".
> This is a patch for erts/emulator/beam/break.c to report
> "OLD_HEND(p) - OLD_HTOP(p)"
> instead of
> "OLD_HEND(p) - OLD_HEAP(p)"
> for "OldHeap unsed".
>
> git fetch git://github.com/kinogmt/otp.git old_heap_unused
>

Thanks! The patch itself looks good, but we would prefer
to have a complete author name. Is OK to change the
author identification to "Satoshi Kinoshito <kino@geminimobile.com>"?

Also, I would like more information in the commit message,
for instance like this:

system_info(procs): Fix the value for "OldHeap unused"

The value for "OldHeap unused" in the output of
erlang:system_info(procs) and in crash dumps, was incorrectly
calculated as the size of the entire old heap.

If you approve the changes above, I can do the changes for you.
You don't have to re-submit the patch.

--
Björn Gustavsson, Erlang/OTP, Ericsson AB

________________________________________________________________
erlang-patches (at) erlang.org mailing list.
See http://www.erlang.org/faq.html
To unsubscribe; mailto:erlang-patches-unsubscribe@erlang.org

Post received from mailinglist
Guest
Posted: Thu Apr 08, 2010 7:55 am Reply with quote
Guest
On Thu, Apr 8, 2010 at 5:48 AM, satoshi kinoshita <kino@geminimobile.com> wrote:

> system_info(procs) is currently reporting a same value for both
> "OldHeap" and "OldHeap unsed".
> This is a patch for erts/emulator/beam/break.c to report
> "OLD_HEND(p) - OLD_HTOP(p)"
> instead of
> "OLD_HEND(p) - OLD_HEAP(p)"
> for "OldHeap unsed".
>
> git fetch git://github.com/kinogmt/otp.git old_heap_unused
>

Thanks! The patch itself looks good, but we would prefer
to have a complete author name. Is OK to change the
author identification to "Satoshi Kinoshito <kino@geminimobile.com>"?

Also, I would like more information in the commit message,
for instance like this:

system_info(procs): Fix the value for "OldHeap unused"

The value for "OldHeap unused" in the output of
erlang:system_info(procs) and in crash dumps, was incorrectly
calculated as the size of the entire old heap.

If you approve the changes above, I can do the changes for you.
You don't have to re-submit the patch.

--
Björn Gustavsson, Erlang/OTP, Ericsson AB

________________________________________________________________
erlang-patches (at) erlang.org mailing list.
See http://www.erlang.org/faq.html
To unsubscribe; mailto:erlang-patches-unsubscribe@erlang.org

Post received from mailinglist
Guest
Posted: Thu Apr 08, 2010 8:02 am Reply with quote
Guest
At Thu, 8 Apr 2010 09:54:17 +0200,
Björn Gustavsson wrote:
>
> On Thu, Apr 8, 2010 at 5:48 AM, satoshi kinoshita <kino@geminimobile.com> wrote:
>
> > system_info(procs) is currently reporting a same value for both
> > "OldHeap" and "OldHeap unsed".
> > This is a patch for erts/emulator/beam/break.c to report
> > "OLD_HEND(p) - OLD_HTOP(p)"
> > instead of
> > "OLD_HEND(p) - OLD_HEAP(p)"
> > for "OldHeap unsed".
> >
> > git fetch git://github.com/kinogmt/otp.git old_heap_unused
> >
>
> Thanks! The patch itself looks good, but we would prefer
> to have a complete author name. Is OK to change the
> author identification to "Satoshi Kinoshito <kino@geminimobile.com>"?



Sure.
Please go ahead fix it.
(uh, my famylyname is "Kinoshita" though:)

thanks,
kino

________________________________________________________________
erlang-patches (at) erlang.org mailing list.
See http://www.erlang.org/faq.html
To unsubscribe; mailto:erlang-patches-unsubscribe@erlang.org

Post received from mailinglist
Guest
Posted: Thu Apr 08, 2010 8:02 am Reply with quote
Guest
At Thu, 8 Apr 2010 09:54:17 +0200,
Björn Gustavsson wrote:
>
> On Thu, Apr 8, 2010 at 5:48 AM, satoshi kinoshita <kino@geminimobile.com> wrote:
>
> > system_info(procs) is currently reporting a same value for both
> > "OldHeap" and "OldHeap unsed".
> > This is a patch for erts/emulator/beam/break.c to report
> > "OLD_HEND(p) - OLD_HTOP(p)"
> > instead of
> > "OLD_HEND(p) - OLD_HEAP(p)"
> > for "OldHeap unsed".
> >
> > git fetch git://github.com/kinogmt/otp.git old_heap_unused
> >
>
> Thanks! The patch itself looks good, but we would prefer
> to have a complete author name. Is OK to change the
> author identification to "Satoshi Kinoshito <kino@geminimobile.com>"?



Sure.
Please go ahead fix it.
(uh, my famylyname is "Kinoshita" though:)

thanks,
kino

________________________________________________________________
erlang-patches (at) erlang.org mailing list.
See http://www.erlang.org/faq.html
To unsubscribe; mailto:erlang-patches-unsubscribe@erlang.org

Post received from mailinglist
Guest
Posted: Thu Apr 08, 2010 8:13 am Reply with quote
Guest
2010/4/8 satoshi kinoshita <kino@geminimobile.com>:

> Please go ahead fix it.
> (uh, my famylyname is "Kinoshita" though:)

Sorry for getting it wrong. I'll fix that and include your branch in 'pu'.

--
Björn Gustavsson, Erlang/OTP, Ericsson AB

________________________________________________________________
erlang-patches (at) erlang.org mailing list.
See http://www.erlang.org/faq.html
To unsubscribe; mailto:erlang-patches-unsubscribe@erlang.org

Post received from mailinglist
Guest
Posted: Thu Apr 08, 2010 8:13 am Reply with quote
Guest
2010/4/8 satoshi kinoshita <kino@geminimobile.com>:

> Please go ahead fix it.
> (uh, my famylyname is "Kinoshita" though:)

Sorry for getting it wrong. I'll fix that and include your branch in 'pu'.

--
Björn Gustavsson, Erlang/OTP, Ericsson AB

________________________________________________________________
erlang-patches (at) erlang.org mailing list.
See http://www.erlang.org/faq.html
To unsubscribe; mailto:erlang-patches-unsubscribe@erlang.org

Post received from mailinglist

Display posts from previous:  

All times are GMT
Page 1 of 1
This forum is locked: you cannot post, reply to, or edit topics.

Jump to:  

You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum
You cannot attach files in this forum
You cannot download files in this forum