You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@openoffice.apache.org by Damjan Jovanovic <da...@apache.org> on 2018/04/14 10:42:17 UTC
Upstreaming FreeBSD ports patches before 4.2.0?
Hi
Can we please upstream the patches from FreeBSD ports, before the 4.2.0
release?
That idlc memory alignment SIGBUS crash from
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=216206 is Clang specific,
not FreeBSD specific, and could happen on other operating systems.
Thank you
Damjan
Re: Upstreaming FreeBSD ports patches before 4.2.0?
Posted by Damjan Jovanovic <da...@apache.org>.
Great, thank you.
No idea. Either Sun/Oracle never got around to implementing it, or they
planned on scrapping it.
On Sat, Apr 28, 2018 at 4:51 AM Don Lewis <tr...@apache.org> wrote:
> I just committed a fix for this in r1830406.
>
> Without this I get build failures due to crashes in the unit tests if
> --enable-debug is specified. With this change I am able to get a
> working build with both the system alloc and the internal alloc.
>
> I was unaware that this was not getting used on the gbuild side of
> things. Why not? It seems like a useful debug feature.
>
> On 15 Apr, Damjan Jovanovic wrote:
> > Alright thank you.
> >
> > Would it be better to just scrap cpprt, like gbuild modules already do?
> >
> > On Sat, Apr 14, 2018 at 4:59 PM, Don Lewis <tr...@apache.org> wrote:
> >
> >> On 14 Apr, Damjan Jovanovic wrote:
> >> > Hi
> >> >
> >> > Can we please upstream the patches from FreeBSD ports, before the
> 4.2.0
> >> > release?
> >> >
> >> > That idlc memory alignment SIGBUS crash from
> >> > https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=216206 is Clang
> >> specific,
> >> > not FreeBSD specific, and could happen on other operating systems.
> >>
> >> That one is a bit complicated to upstream. In the FreeBSD port I only
> >> apply the patch conditionally when building with recent clang on amd64.
> >> I could be harmful in terms of memory consumption on 32-bit machines.
> >> The changes in the patch would need to be #ifdefed in order to import
> >> it.
> >>
> >> Also the changes for the internal allocator are lightly tested at best.
> >>
> >> https://svnweb.freebsd.org/ports/head/editors/openoffice-
> >> devel/files/extra-patch-align16?revision=432895&view=markup
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
> >> For additional commands, e-mail: dev-help@openoffice.apache.org
> >>
> >>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
> For additional commands, e-mail: dev-help@openoffice.apache.org
>
>
Re: Upstreaming FreeBSD ports patches before 4.2.0?
Posted by Don Lewis <tr...@apache.org>.
I just committed a fix for this in r1830406.
Without this I get build failures due to crashes in the unit tests if
--enable-debug is specified. With this change I am able to get a
working build with both the system alloc and the internal alloc.
I was unaware that this was not getting used on the gbuild side of
things. Why not? It seems like a useful debug feature.
On 15 Apr, Damjan Jovanovic wrote:
> Alright thank you.
>
> Would it be better to just scrap cpprt, like gbuild modules already do?
>
> On Sat, Apr 14, 2018 at 4:59 PM, Don Lewis <tr...@apache.org> wrote:
>
>> On 14 Apr, Damjan Jovanovic wrote:
>> > Hi
>> >
>> > Can we please upstream the patches from FreeBSD ports, before the 4.2.0
>> > release?
>> >
>> > That idlc memory alignment SIGBUS crash from
>> > https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=216206 is Clang
>> specific,
>> > not FreeBSD specific, and could happen on other operating systems.
>>
>> That one is a bit complicated to upstream. In the FreeBSD port I only
>> apply the patch conditionally when building with recent clang on amd64.
>> I could be harmful in terms of memory consumption on 32-bit machines.
>> The changes in the patch would need to be #ifdefed in order to import
>> it.
>>
>> Also the changes for the internal allocator are lightly tested at best.
>>
>> https://svnweb.freebsd.org/ports/head/editors/openoffice-
>> devel/files/extra-patch-align16?revision=432895&view=markup
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
>> For additional commands, e-mail: dev-help@openoffice.apache.org
>>
>>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org
Re: Upstreaming FreeBSD ports patches before 4.2.0?
Posted by Damjan Jovanovic <da...@apache.org>.
Alright thank you.
Would it be better to just scrap cpprt, like gbuild modules already do?
On Sat, Apr 14, 2018 at 4:59 PM, Don Lewis <tr...@apache.org> wrote:
> On 14 Apr, Damjan Jovanovic wrote:
> > Hi
> >
> > Can we please upstream the patches from FreeBSD ports, before the 4.2.0
> > release?
> >
> > That idlc memory alignment SIGBUS crash from
> > https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=216206 is Clang
> specific,
> > not FreeBSD specific, and could happen on other operating systems.
>
> That one is a bit complicated to upstream. In the FreeBSD port I only
> apply the patch conditionally when building with recent clang on amd64.
> I could be harmful in terms of memory consumption on 32-bit machines.
> The changes in the patch would need to be #ifdefed in order to import
> it.
>
> Also the changes for the internal allocator are lightly tested at best.
>
> https://svnweb.freebsd.org/ports/head/editors/openoffice-
> devel/files/extra-patch-align16?revision=432895&view=markup
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
> For additional commands, e-mail: dev-help@openoffice.apache.org
>
>
Re: Upstreaming FreeBSD ports patches before 4.2.0?
Posted by Don Lewis <tr...@apache.org>.
On 14 Apr, Damjan Jovanovic wrote:
> Hi
>
> Can we please upstream the patches from FreeBSD ports, before the 4.2.0
> release?
>
> That idlc memory alignment SIGBUS crash from
> https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=216206 is Clang specific,
> not FreeBSD specific, and could happen on other operating systems.
That one is a bit complicated to upstream. In the FreeBSD port I only
apply the patch conditionally when building with recent clang on amd64.
I could be harmful in terms of memory consumption on 32-bit machines.
The changes in the patch would need to be #ifdefed in order to import
it.
Also the changes for the internal allocator are lightly tested at best.
https://svnweb.freebsd.org/ports/head/editors/openoffice-devel/files/extra-patch-align16?revision=432895&view=markup
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@openoffice.apache.org
For additional commands, e-mail: dev-help@openoffice.apache.org