Project

General

Profile

Actions

Feature #9068

closed

[PATCH (trivial)] thread.c: reduce rb_mutex_t size by 8 bytes on x86_64

Added by normalperson (Eric Wong) over 10 years ago. Updated over 10 years ago.

Status:
Closed
Assignee:
-
Target version:
[ruby-core:58114]

Description

gcc can pack the structure better when both 32-bit fields
are next to each other.

(I realized I could have 10K-100K Mutexes potentially in one of my projects,
but maybe I'll change the design :)


Files

Updated by kosaki (Motohiro KOSAKI) over 10 years ago

(10/31/13 3:42 PM), normalperson (Eric Wong) wrote:

Issue #9068 has been reported by normalperson (Eric Wong).


Feature #9068: [PATCH (trivial)] thread.c: reduce rb_mutex_t size by 8 bytes on x86_64
https://bugs.ruby-lang.org/issues/9068

Author: normalperson (Eric Wong)
Status: Open
Priority: Low
Assignee:
Category: core
Target version: current: 2.1.0

gcc can pack the structure better when both 32-bit fields
are next to each other.

(I realized I could have 10K-100K Mutexes potentially in one of my projects,
but maybe I'll change the design :)

ok, please commit.

Updated by normalperson (Eric Wong) over 10 years ago

KOSAKI Motohiro wrote:

ok, please commit.

I never accepted commit bit.

However I've been reconsidering that (because of dtas). But I can't
provide a useful GPG-signed pubkey, no web-of-trust connections at all.

So http://yhbt.net/id_rsa.pub has been online for many years, but I
cannot guarantee the corresponding private key is not compromised.
I screw up credential management very frequently :x

Actions #3

Updated by kosaki (Motohiro KOSAKI) over 10 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r43497.
Eric, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • thread.c (rb_mutex_struct): reduce rb_mutex_t size by 8 bytes
    on 64bit platform. Patch by Eric Wong. [Feature #9068][ruby-core:58114]

Updated by kosaki (Motohiro KOSAKI) over 10 years ago

(10/31/13 5:12 PM), Eric Wong wrote:

KOSAKI Motohiro wrote:

ok, please commit.

I never accepted commit bit.

Sorry, I frequently forget about this. You are a virtual committer.
I've committed this instead of you.

However I've been reconsidering that (because of dtas). But I can't
provide a useful GPG-signed pubkey, no web-of-trust connections at all.

So http://yhbt.net/id_rsa.pub has been online for many years, but I
cannot guarantee the corresponding private key is not compromised.
I screw up credential management very frequently :x

You seems considering about keys too serious. Even if you lost your key,
we can revoke old key and install new one.

Updated by normalperson (Eric Wong) over 10 years ago

KOSAKI Motohiro wrote:

(10/31/13 5:12 PM), Eric Wong wrote:

KOSAKI Motohiro wrote:

ok, please commit.

I never accepted commit bit.

Sorry, I frequently forget about this. You are a virtual committer.
I've committed this instead of you.

Thanks.

However I've been reconsidering that (because of dtas). But I can't
provide a useful GPG-signed pubkey, no web-of-trust connections at all.

So http://yhbt.net/id_rsa.pub has been online for many years, but I
cannot guarantee the corresponding private key is not compromised.
I screw up credential management very frequently :x

You seems considering about keys too serious. Even if you lost your key,
we can revoke old key and install new one.

But nobody may know if my key is compromised.
Basically, I'm uncomfortable if people try to trust me with anything.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0