Bug 267938 - UI blocking when old UM started
Summary: UI blocking when old UM started
Status: RESOLVED WONTFIX
Alias: None
Product: Equinox
Classification: Eclipse Project
Component: p2 (show other bugs)
Version: 3.4.2   Edit
Hardware: PC Linux
: P3 normal (vote)
Target Milestone: 3.5   Edit
Assignee: P2 Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: polish
Depends on:
Blocks:
 
Reported: 2009-03-10 14:21 EDT by Andrew Overholt CLA
Modified: 2009-04-13 16:44 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Overholt CLA 2009-03-10 14:21:36 EDT
I tried to install a feature with install handlers and in the process of launching the old Update Manager the entire UI thread was blocked.
Comment 1 John Arthorne CLA 2009-03-10 14:32:05 EDT
Blocked as in never comes back, or just blocked for some non-trivial time? What build?
Comment 2 Andrew Overholt CLA 2009-03-10 14:34:05 EDT
(In reply to comment #1)
> Blocked as in never comes back, or just blocked for some non-trivial time? What
> build?

Blocked as in the UI thread is busy.  Sorry, should have been more clear.  I20090309-0800 upgraded to I20090310-0100.
Comment 3 Susan McCourt CLA 2009-03-10 15:11:08 EDT
I tried using the site in the test plan
http://www.developer.com/img/2007/03/upd-install-site.xml
and was able to successfully install using UM without a hang.

I was working on WinXP with a I20090309-1300 that had been upgraded to I20090310-0100.

Which site were you using?
If it was the site above, did you see the "Splash!" bitmap come up during the install?

I'm wondering if the handler itself hung...
Comment 4 Andrew Overholt CLA 2009-03-10 15:14:33 EDT
(In reply to comment #3)
> I tried using the site in the test plan
> http://www.developer.com/img/2007/03/upd-install-site.xml
> and was able to successfully install using UM without a hang.
> 
> I was working on WinXP with a I20090309-1300 that had been upgraded to
> I20090310-0100.
> 
> Which site were you using?
> If it was the site above, did you see the "Splash!" bitmap come up during the
> install?
> 
> I'm wondering if the handler itself hung...

I didn't actually go through with the install.  I was just trying to point out that the launching of the old UM seemed to be done in the UI thread.
Comment 5 Susan McCourt CLA 2009-03-10 19:30:26 EDT
Well, yes, the UM wizard is launched from the p2 wizard, so it's in the UI thread.  So you get a modal UM wizard which is parented by a modal p2 wizard.  I don't see that this is a huge problem, or are you saying the UI thread seemed blocked (ie, no paints, no updating, something more serious than a modal dialog?)

DJ, did you observe anything funky here on Linux?
Comment 6 Andrew Overholt CLA 2009-03-11 09:02:41 EDT
(In reply to comment #5)
> I don't see that this is a huge problem, or are you saying the UI thread seemed
> blocked (ie, no paints, no updating, something more serious than a modal
> dialog?)

Yes, no paints, no updating, etc.
Comment 7 John Arthorne CLA 2009-03-11 09:37:27 EDT
I think the problem is that the very first time update is touched, update.configurator does a bunch of work (see also bug 266236). ui.ide used to start update during its startup, but they have recently been cleaning up references to update so it may no longer be happening. So, it may just be that this was the first contact with update and it is initializing.
Comment 8 DJ Houghton CLA 2009-03-11 10:37:22 EDT
(In reply to comment #5)
> Well, yes, the UM wizard is launched from the p2 wizard, so it's in the UI
> thread.  So you get a modal UM wizard which is parented by a modal p2 wizard. 
> I don't see that this is a huge problem, or are you saying the UI thread seemed
> blocked (ie, no paints, no updating, something more serious than a modal
> dialog?)
> 
> DJ, did you observe anything funky here on Linux?
> 

I see the same behaviour as Andrew... no paints until the UM dialog comes up.
Comment 9 Susan McCourt CLA 2009-03-11 11:06:24 EDT
I'll mark this for the 3.5 polish list, but I'm not sure if I can fix this outside of UM.  I'll examine starting UM up in a job and then invoking the UI (just don't know if there's API for that)...
Comment 10 John Arthorne CLA 2009-04-13 16:44:13 EDT
I don't think there is much we can do about this. There is a non-trival hit when the update bundles are first started. This might happen here, or it might happen elsewhere. We could play tricks with manually starting the update bundles in the background prior to invoking the update UI, but this is getting quite fancy for a backward compatibility layer. People will be steadily migrating away from UM install handlers towards p2 touchpoints so this issue will become even less interesting in the future. Consider it just one more bit of incentive for people to migrate their sites/features away from UM install handlers.