Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1259555

Re: 答复: [PATCHv2 4.3-rc6] proc: fix convert from oom_score_adj to oom_adj

From Michal Hocko <mhocko@kernel.org>
Newsgroups linux.kernel
Subject Re: 答复: [PATCHv2 4.3-rc6] proc: fix convert from oom_score_adj to oom_adj
Date 2015-10-30 15:50 +0100
Message-ID <qpja9-70R-3@gated-at.bofh.it> (permalink)
References (1 earlier) <qnXOr-526-23@gated-at.bofh.it> <qoINj-Vk-9@gated-at.bofh.it> <qoINj-Vk-5@gated-at.bofh.it> <qoYS5-31A-13@gated-at.bofh.it> <qphrJ-5Uv-29@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Fri 30-10-15 13:59:03, Michal Hocko wrote:
> On Thu 29-10-15 18:04:22, Michal Hocko wrote:
> > On Wed 28-10-15 16:54:04, David Rientjes wrote:
> > [...]
> > > It's a bad situation, I agree, and we anticipated the complete removal of 
> > > /proc/pid/oom_adj years ago since it has been deprecated for years.  Maybe 
> > > one day we can convince Linus that is possible, but until then we're stuck 
> > > with it.
> > 
> > Let's do it then.
> 
> I've just checked debian code search and it seems that procps still
> relies on oom_adj. I have sent a patch but that sounds like we are not
> there yet. I will hunt for other projects still using the deprecated
> file exclusively. Hopefully there won't be too many of them.

It doesn't look that bad afterall:
$ curl -s http://codesearch.debian.net/results/7223e657af3f2ad0/packages.json
{"Packages":["tgt","ggobi","hurd","linux","condor","wine-gecko-2.21","android-platform-frameworks-native","nautilus","procps","wireshark","intel-gpu-tools","iceweasel","icedove","ardour","linux-tools","kde4libs","nss-pam-ldapd","chromium-browser","passenger","archipel-agent-virtualmachine-oomkiller","bleachbit","tilestache","slurm-llnl","ns3","nbd","open-iscsi","mhwaveedit","nilfs-tools","stress-ng","lvm2","gradm2","audit","postgresql-common","zfs-fuse","ocfs2-tools","gimp","advene","lldpad","reniced","pitivi","trinity","petri-foo","rtai","postgresql-9.4","procenv","multipath-tools","percona-toolkit","apparmor","upstart","watchdog","boinc","fusil","util-vserver","booth","geeqie","openssh","oar","android-platform-system-core","kinit","xournal","player","gimp-gap","android-tools"]}

Of those
* android-tools need a trivial patch - not sure who is upstream here
  so pushed through Debian bugzilla - https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=803485
* boinc-client.init need a trivial patch - Debian specific it seesm
  https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=803484
* ocfs2_controld - posted to debian as I wasn't sure about the upstream
  status - https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=803486
* procps needs a trivial patch - sent upstream already

This two are nasty because they consume only oom_adj scale so we have
to rescale explicitly :/
* archipel-agent-virtualmachine-oomkiller oom_adj is stored in the DB
* reniced - this is one is nasty as well because it consumes oom_adj
  from user

I will have a look at them early next week.
-- 
Michal Hocko
SUSE Labs
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: 答复: [PATCHv2 4.3-rc6] proc: fix convert from oom_score_adj to oom_adj David Rientjes <rientjes@google.com> - 2015-10-29 01:00 +0100
  Re: 答复: [PATCHv2 4.3-rc6] proc: fix convert  from oom_score_adj to oom_adj ebiederm@xmission.com (Eric W. Biederman) - 2015-10-29 05:00 +0100
    Re: 答复: [PATCHv2 4.3-rc6] proc: fix convert from oom_score_adj to oom_adj David Rientjes <rientjes@google.com> - 2015-11-01 05:40 +0100
  Re: 答复: [PATCHv2 4.3-rc6] proc:  fix convert from oom_score_adj to oom_adj Michal Hocko <mhocko@kernel.org> - 2015-10-29 18:10 +0100
    Re: 答复: [PATCHv2 4.3-rc6] proc:  fix convert from oom_score_adj to oom_adj Michal Hocko <mhocko@kernel.org> - 2015-10-30 14:00 +0100
      Re: 答复: [PATCHv2 4.3-rc6] proc:  fix convert from oom_score_adj to oom_adj Michal Hocko <mhocko@kernel.org> - 2015-10-30 15:50 +0100
        Re: 答复: [PATCHv2 4.3-rc6] proc: fix convert from oom_score_adj to oom_adj David Rientjes <rientjes@google.com> - 2015-11-01 05:40 +0100
          Re: 答复: [PATCHv2 4.3-rc6] proc:  fix convert from oom_score_adj to oom_adj Michal Hocko <mhocko@kernel.org> - 2015-11-02 14:10 +0100
          Re: 答复: [PATCHv2 4.3-rc6] proc:  fix convert from oom_score_adj to oom_adj Dave Jones <davej@codemonkey.org.uk> - 2015-11-02 18:20 +0100
            Re: 答复: [PATCHv2 4.3-rc6] proc:  fix convert from oom_score_adj to oom_adj Michal Hocko <mhocko@kernel.org> - 2015-11-02 21:30 +0100

csiph-web