All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dilger, Andreas <andreas.dilger@intel.com>
To: lustre-devel@lists.lustre.org
Subject: [Lustre-devel] Removal of SOM code
Date: Fri, 6 Feb 2015 23:33:29 +0000	[thread overview]
Message-ID: <D0FA9D65.DA3FB%andreas.dilger@intel.com> (raw)

The Size on MDT (SOM) feature has been in a prototype state for several
years, with no signs of moving beyond this prototype stage.

Several problems exist in the code today, primarily that recovery is not
really implemented, yet the existing code adds complexity on the clients
and servers. Without proper recovery, the current code risks file data
loss if the SOM data isn't updated on the MDS consistently with data
writes to the OST.


We're planning to remove the SOM code from the master branch as a result,
tracked under https://jira.hpdd.intel.com/browse/LU-6047:
- http://review.whamcloud.com/13126
- http://review.whamcloud.com/13169
- http://review.whamcloud.com/13442
- http://review.whamcloud.com/13443

Some of the performance improvements of SOM have been implemented by
statahead.

I think a case could be made for a very stripped down SOM to be
implemented in the future, that only deals with single-client writers and
synchronously invalidates the file size on open-for-write, which isn't so
bad with flash storage for the MDT as is typical today.  The size of files
that do not get set at initial write or are invalidated by an open can be
updated asynchronously by LFSCK doing a periodic scan in the background.
Since this stripped-down implementation would have very little to do with
the current implementation, there isn't much benefit to even trying to fix
the current code in place.

I definitely prefer presenting about new features going into Lustre, but I
also think it is important that people are aware when a semi-feature like
this is being removed.  I don't believe that anyone is actually using this
feature today, and the reduction in code maintenance and complexity will
help both ongoing maintenance and bug fixing, as well as make it a that
much easier for new developers to understand the code.

Cheers, Andreas
-- 
Andreas Dilger

Lustre Software Architect
Intel High Performance Data Division

             reply	other threads:[~2015-02-06 23:33 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-06 23:33 Dilger, Andreas [this message]
     [not found] ` <CAC_cOcKVa=5XGcA81yimZHJ81H+5YQ24UXtQrdYBC9bsN1ihRw@mail.gmail.com>
2015-02-08  1:56   ` [Lustre-devel] [HPDD-discuss] Removal of SOM code Dilger, Andreas
     [not found]     ` <CAB_j=MdDBW8o67ysrrjFL8xTtp4MEQ0gdcQnTwXoe3xgkbJ96Q@mail.gmail.com>
2015-02-10 21:47       ` Dilger, Andreas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=D0FA9D65.DA3FB%andreas.dilger@intel.com \
    --to=andreas.dilger@intel.com \
    --cc=lustre-devel@lists.lustre.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.