linux-ext4.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Nikolai Joukov <kolya@cs.sunysb.edu>
To: David Chinner <dgc@sgi.com>
Cc: linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org
Subject: Re: [RFC][PATCH] Secure Deletion and Trash-Bin Support for Ext4
Date: Tue, 5 Dec 2006 11:41:28 -0500 (EST)	[thread overview]
Message-ID: <Pine.GSO.4.53.0612051049200.27677@compserv1> (raw)
In-Reply-To: <20061204235042.GS33919298@melbourne.sgi.com>

> > As we promised on the linux-ext4 list on October 31, here is the patch
> > that adds secure deletion via a trash-bin functionality for ext4.  It is a
> > compromise solution that combines secure deletion with the trash-bin support
> > (the latter had been requested by even more people than the former :-).
>
> Given that almost all of the code for this uses vfs interfaces and
> only a couple of simple filesystem hooks, is there any reason for
> this being ext4 specific?  i.e. why aren't you hooking the vfs layer
> so we get a single undelete/secure delete implementation for all
> filesystems?

You are right.  Actually, we mentioned it as a benefit number 4 of this
approach in the original post.  Most of the code is not
file-system--specific and can be used by any other (all other?) file
system(s).  The only complication is that only ext2/3/4 and reiser file
systems already support the per-file secure deletion and undelete
attributes.

Since ext4 is in early development now, we believe it'd be easier to get
such code into ext4 than into the main-line VFS.  If there's enough
interested among the kernel maintainers, we'd be happy to move this
functionality to the VFS and provide f/s hooks for
secure-deletion/trash-bin.

I guess, the right thing to do would be to move the common trash-bin
(tb.c and tb.h) code into the /fs and /include/linux directories.
Also, VFS would require just a couple of trivial changes to support
something like '-o trashbin' mount-time option for all file systems.
In addition, individual file systems may support per-file attributes for
this (e.g., ext2/3/4).

Since I just postponed my moving house (I am joining IBM soon) I think we
may have time to try it out this week.

Nikolai.
---------------
Filesystems and Storage Laboratory,
Stony Brook University

  reply	other threads:[~2006-12-05 16:41 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-12-04 18:33 [RFC][PATCH] Secure Deletion and Trash-Bin Support for Ext4 Nikolai Joukov
2006-12-04 23:50 ` David Chinner
2006-12-05 16:41   ` Nikolai Joukov [this message]
2006-12-06  9:11     ` David Chinner
2006-12-06 22:16       ` Nathan Scott
2006-12-07  0:56       ` Josef Sipek
2006-12-07  1:44         ` David Chinner
2006-12-07  2:35           ` Josef Sipek
2006-12-07  2:49             ` David Chinner
2006-12-07  3:14               ` Nicholas Miell
2006-12-07  5:35                 ` Josef Sipek
2006-12-07  5:45           ` Nathan Scott
2006-12-09  1:44       ` Nikolai Joukov

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=Pine.GSO.4.53.0612051049200.27677@compserv1 \
    --to=kolya@cs.sunysb.edu \
    --cc=dgc@sgi.com \
    --cc=linux-ext4@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).