harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Elena Semukhina" <elena.semukh...@gmail.com>
Subject Re: [classlib][net] java.net.URLClassLoader implementation is not thread-safe
Date Tue, 30 Jan 2007 13:35:54 GMT
On 1/30/07, Geir Magnusson Jr. <geir@pobox.com> wrote:
>
> I have a comment, that I'll bring out to the dev list -
>
> Is there any problem with some other thread adding the entry for
> currentURL while the  if (jf==null) block is doing it's work? That's
> one difference I can see between sync-ing the block, vs just syncing
> the map.
>
> I don't know if it matters, though.


Mikhail's approach leads to a deadlock in the test. I don't know so far what
is the exact reason.

Elena



>
> geir
>
> On Jan 30, 2007, at 7:48 AM, Mikhail Markov wrote:
>
> > Elena, thanks for the patch!
> > I've reviewed it and posted the comment to this JIRA.
> >
> > Regards,
> > Mikhail
> >
> >
> > On 1/30/07, Elena Semukhina <elena.semukhina@gmail.com> wrote:
> >>
> >> The DRLVM smoke test classloader.StressLoader fails intermittently.
> >> Investigation revealed that java.net.URLClassLoader implementation
> >> does
> >> not
> >> take care of synchronizing on access to its IdentityHashMap field. I
> >> suggested a patch to fix the issue at
> >> https://issues.apache.org/jira/browse/HARMONY-2982.
> >>
> >> Could anyone please take a look and review the patch?
> >>
> >> --
> >> Thanks,
> >> Elena
> >>
> >>
>
>


-- 
Thanks,
Elena

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message