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


Groups > linux.kernel > #1334889

Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count

From Theodore Ts'o <tytso@mit.edu>
Newsgroups linux.kernel
Subject Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count
Date 2016-02-16 01:50 +0100
Message-ID <r2C01-aC-5@gated-at.bofh.it> (permalink)
References <r1sm5-3kI-1@gated-at.bofh.it> <r2cff-8he-7@gated-at.bofh.it> <r2w4j-4N8-35@gated-at.bofh.it> <r2Arg-7MI-13@gated-at.bofh.it> <r2BdD-83f-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Mon, Feb 15, 2016 at 03:52:31PM -0800, Daniel Walker wrote:
> >>We need it to determine accurately what the free memory in the
> >>system is. If you know where we can get this information already
> >>please tell, we aren't aware of it. For instance /proc/meminfo isn't
> >>accurate enough.
> 
> Approximate point-in-time indication is an accurate characterization
> of what we are doing. This is good enough for us. NO matter what we
> do, we are never going to be able to address the "time of check to
> time of use” window.  But, this approximation works reasonably well
> for our use case.

Why do you need such accuracy, and what do you consider "good enough".
Having something which iterates over all of the inodes in the system
is something that really shouldn't be in a general production kernel
At the very least it should only be accessible by root (so now only a
careless system administrator can DOS attack the system) but the
Dave's original question still stands.  Why do you need a certain
level of accuracy regarding how much memory is available after
dropping all of the caches?  What problem are you trying to
solve/avoid?

It may be that you are going about things completely the wrong way,
which is why understanding the higher order problem you are trying to
solve might be helpful in finding something which is safer,
architecturally cleaner, and something that could go into the upstream
kernel.

Cheers,

						- Ted

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


Thread

[PATCH] kernel: fs: drop_caches: add dds drop_caches_count Daniel Walker <danielwa@cisco.com> - 2016-02-12 21:20 +0100
  Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Dave Chinner <david@fromorbit.com> - 2016-02-14 22:20 +0100
    Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Daniel Walker <danielwa@cisco.com> - 2016-02-15 19:30 +0100
      Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Dave Chinner <david@fromorbit.com> - 2016-02-16 00:10 +0100
        Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Daniel Walker <danielwa@cisco.com> - 2016-02-16 01:00 +0100
          Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Theodore Ts'o <tytso@mit.edu> - 2016-02-16 01:50 +0100
            Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count "Nag Avadhanam (nag)" <nag@cisco.com> - 2016-02-16 04:10 +0100
              Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Dave Chinner <david@fromorbit.com> - 2016-02-16 06:40 +0100
                Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Nag Avadhanam <nag@cisco.com> - 2016-02-16 08:20 +0100
                Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Dave Chinner <david@fromorbit.com> - 2016-02-16 09:40 +0100
              Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Vladimir Davydov <vdavydov@virtuozzo.com> - 2016-02-16 09:50 +0100
                Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Nag Avadhanam <nag@cisco.com> - 2016-02-16 19:40 +0100
          Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Dave Chinner <david@fromorbit.com> - 2016-02-16 06:30 +0100
            Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Nag Avadhanam <nag@cisco.com> - 2016-02-16 07:10 +0100
              Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Dave Chinner <david@fromorbit.com> - 2016-02-16 09:30 +0100
            Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Rik van Riel <riel@redhat.com> - 2016-02-16 17:20 +0100

csiph-web