All of lore.kernel.org
 help / color / mirror / Atom feed
From: Edward Shishkin <edward@namesys.com>
To: Peter van Hardenberg <pvh@uvic.ca>
Cc: reiserfs-list@namesys.com
Subject: Re: Regular Plugin
Date: Thu, 17 Nov 2005 22:56:17 +0300	[thread overview]
Message-ID: <437CE061.10203@namesys.com> (raw)
In-Reply-To: <200511161004.51277.pvh@uvic.ca>

Peter van Hardenberg wrote:

>The notion of the regular plugin seems odd to me. It is a seperate plugin type 
>with distinct plugins that only serve to refer to "file" plugins (which are 
>found in plugin/object.c, not in plugin/file/*.c).
>  
>

In accordance with the last policy file plugins, that implement objects 
which are known
(for vfs) as regular files, should be stored in plugin/file directory, 
so in the latest versions
of reiser4 the file regular.c has been moved to the upper common 
directory "plugin".

>So when I create a new "file" plugin, I cannot use it unless I also create a 
>"regular" plugin which serves no purpose except to allow me to create those 
>"file"s by storing their ID. 
>  
>

You are right.

>A few words explaining why the "regular" plugin field couldn't have somehow 
>storeda "file" plugin would be educational. 
>

Because changing a file plugin id is unsafe operation (at least in the 
case when there is
more then one field of file plugin type in struct plugin_set): suppose 
you allow a some
file plugin to be changed, it means that users also will be able to 
change foo/..../plugin/file,
the main plugin which manages the file "foo". This is why we added a 
special plugin type:
feel free to change these plugins.

>Also, regular is a rather unclear 
>name. Something like "defaultfileplugin" might be easier to deduce, or even 
>"defaultchild".
>
>  
>

Perhaps you are right about clearness, but the reason for the name 
"regular" was the need for
nominated children to be "regular files for vfs". This is why 
"defaultfileplugin" sounds not
so good (file plugin can be a directory file plugin, whereas mkdir 
doesnt look at this field
at all, and "default" sounds like something that is not to be modified). 
Maybe "regularchild"
will improve the situation?

Thanks,
Edward.

>-pvh
>
>  
>


      reply	other threads:[~2005-11-17 19:56 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-16 18:04 Regular Plugin Peter van Hardenberg
2005-11-17 19:56 ` Edward Shishkin [this message]

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=437CE061.10203@namesys.com \
    --to=edward@namesys.com \
    --cc=pvh@uvic.ca \
    --cc=reiserfs-list@namesys.com \
    /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.