Public bug reported:
qtmir currently has some cgmanager code.
However, since then ual has evolved and switched to systemd, and provides
better apis then quering cgmanager.
Plus quering cgmanager is pointless, since systemd doesn't use the same cgroups
paths.
<xnox> alan_g, Saviq - so ubuntu-app-launch has moved onto systemd, yet qtmir
is using cgmanager/upstart cgroup paths to determine related app pids
<xnox> so my understanding is that fetchAssociatedPids is now broken.
<xnox> and effectively always returns "which is not app-specific"
<xnox> (e.g. does not find any related pids)
<Saviq> that might very well be, greyback ↑ ?
<alan_g> greyback: ^^
<xnox> QSet<pid_t> DBusFocusInfo::fetchAssociatedPids(pid_t pid) always goes
down the else path, as no things in zesty+ have 'upstart' in their cgroup
<greyback> xnox: I think you're right yes. However that code is only used to
expose a pid list via dbus of focused apps, which is used only by autopilot I
think.
* mhall119 (~mhall@ubuntu/member/mhall119) has joined
<xnox> aha
<greyback> it is not important code, and we didn't notice this bug in the
systemd transition
<greyback> because almost nobody uses that dbus api
<xnox> greyback, i think i will make the code to go down the else path, without
changing any api/abi
<xnox> and drop build-dep and runtime dep on the cgmanager.
<xnox> meaning i'm not making things any worse.
<xnox> it is possible that autopilot uses ubuntu-app-launch apis directly for
the systemd code path =/
<greyback> xnox: I think that change won't hurt anyone that we care about
* AlbertA (~alber...@cpe-72-190-126-231.tx.res.rr.com) has joined
* AlbertA_ (~albe...@cpe-72-190-126-231.tx.res.rr.com) has joined
* AlbertA_ has quit (Remote host closed the connection)
* shiznix has quit (Ping timeout: 240 seconds)
* AlbertA has quit (Client Quit)
* AlbertA (~alber...@cpe-72-190-126-231.tx.res.rr.com) has joined
* ChrisTownsend has quit (Ping timeout: 260 seconds)
* mhall119 has quit (Remote host closed the connection)
<xnox> running tests now, fingers crossed
<xnox> horay!
<xnox> onto gles variant
* ChrisTownsend (~towns...@96-67-93-93-static.hfc.comcastbusiness.net) has
joined
* JanC_ (~janc@lugwv/member/JanC) has joined
* JanC has quit (Killed (barjavel.freenode.net (Nickname regained by services)))
* JanC_ is now known as JanC
* Son_Goku has quit (Quit: My MacBook has gone to sleep. ZZZzzz…)
<xnox> greyback, i've asked tedg if there is anyway to get all the pids from
ubuntu-app-launch and here is his response
<xnox> <tedg> xnox: Yup, you can grab it from the instance object, pids()
returns a vector.
<xnox> <tedg> xnox: You can also call ubuntu-app-launch-pids
<xnox> <xnox> nice.
<xnox> <tedg> xnox: But those will only really work on U8...
<xnox> so either autopilot should use that, or qtmir should re-expose that over
db
<greyback> xnox: right. UAL does make that possible, we just kept neglecting
that particular bit of code
<xnox> ah, ok.
<xnox> i shall open a bug report.
<xnox> at least
<tedg> xnox: Yup, you can grab it from the instance object, pids() returns a
vector.
<tedg> xnox: You can also call ubuntu-app-launch-pids
<xnox> nice.
<tedg> xnox: But those will only really work on U8...
<xnox> tedg, qtmir exposes something similar via dbus, by iterating things with
cgmanager DBusFocusInfo::fetchAssociatedPids(pid_t pid)
<xnox> tedg, do you happen to know if autopilot switched from qtmir dbus api,
to the ubuntu-app-launch pids() api?
<tedg> xnox: I'm not sure, but I think qtmir uses ual as its backend there.
<tedg> xnox: So it should be *much* faster under systemd than Upstart.
<xnox> tedg, nope, it links cgmanager, gets cgmanager dbus proxy object, looks
up cgroup path to contain /upstart/ and checks the freezer group for tasks.
<xnox> at least for that call, which is exposed via dbus, which supposedly only
autopilot uses
<xnox> maybe it uses ual api somewhere else, but i didn't look.
<xnox> tedg, thank you very much for your responses!
<xnox> tedg, from #ubuntu-mir channel <greyback> xnox: right. UAL does make
that possible, we just kept neglecting that particular bit of cod
<xnox> i shall open a bug report.
<greyback> yep, it is an old bit of code, new UAL can probably help it out
** Affects: qtmir (Ubuntu)
Importance: Wishlist
Status: New
** Changed in: qtmir (Ubuntu)
Importance: Undecided => Wishlist
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1685815
Title:
qtmir should probably use more of ual apis
To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/qtmir/+bug/1685815/+subscriptions
--
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
