All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bernd Petrovitsch <bernd-JY8rjfDgnXcb9sJ47jD7nuTv7YV0F9Eg@public.gmane.org>
To: Stef Bon <stefbon-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Steve French <smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	"linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	linux-cifs <linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: Possible to make nfs aware of a inotify watch has been set.
Date: Fri, 30 Nov 2012 14:37:20 +0100	[thread overview]
Message-ID: <1354282642.22426.7.camel@thorin> (raw)
In-Reply-To: <CANXojcwRy5cBz=E1S567J-1BFC4AiamLr=35bKrX0Zq-Me2b3w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

Hi!

Since you introduced the "user experience" ignoring the fact that each
user is quite different - especially if it comes to the "Average Joes
using Ubuntu vs techies" department -, you should be prepared to eat
your own dog food:

On Fre, 2012-11-30 at 08:10 +0100, Stef Bon wrote:
[...]
> At this moment the different events look like:

Perhaps you should have used hexadecimal notation so that it is easier
for the usual/average users of source code to see the bits more easily.
In short: Please improve the user experience. Thank you.

> #define NOTIFYFS_FSEVENT_META_ATTRIB_NOTSET                     2
> #define NOTIFYFS_FSEVENT_META_ATTRIB_MODE                       4
> #define NOTIFYFS_FSEVENT_META_ATTRIB_OWNER                      8
> #define NOTIFYFS_FSEVENT_META_ATTRIB_GROUP                      16
> #define NOTIFYFS_FSEVENT_META_ATTRIB                            28
And that number seems pretty random - not a power of 2 or a bit mask or
the sum of the above.

> #define NOTIFYFS_FSEVENT_META_XATTR_NOTSET                      32
> #define NOTIFYFS_FSEVENT_META_XATTR_CREATE                      64
> #define NOTIFYFS_FSEVENT_META_XATTR_MODIFY                      128
> #define NOTIFYFS_FSEVENT_META_XATTR_DELETE                      256
> #define NOTIFYFS_FSEVENT_META_XATTR                             448
And that number seems pretty random too - not a power of 2 or a bit mask
or the sum of the above.

> #define NOTIFYFS_FSEVENT_FILE_NOTSET                            512
> #define NOTIFYFS_FSEVENT_FILE_MODIFIED                          1024
> #define NOTIFYFS_FSEVENT_FILE_SIZE                              2048
> #define NOTIFYFS_FSEVENT_FILE_OPEN                              4096
> #define NOTIFYFS_FSEVENT_FILE_READ                              8192
> #define NOTIFYFS_FSEVENT_FILE_CLOSE_WRITE                       16384
> #define NOTIFYFS_FSEVENT_FILE_CLOSE_NOWRITE                     32768

> #define NOTIFYFS_FSEVENT_MOVE_NOTSET                            65536

And from here own I'm too lazy to calculate (or even guess) if these are
actually powers of 2 or not.
> #define NOTIFYFS_FSEVENT_MOVE_CREATED                           131072
> #define NOTIFYFS_FSEVENT_MOVE_MOVED                             262144
> #define NOTIFYFS_FSEVENT_MOVE_MOVED_FROM                        524288
> #define NOTIFYFS_FSEVENT_MOVE_MOVED_TO                          1048576
> #define NOTIFYFS_FSEVENT_MOVE_DELETED                           2097152
> 
> #define NOTIFYFS_FSEVENT_FS_NOTSET                              4194304
> #define NOTIFYFS_FSEVENT_FS_MOUNT                               8388608
> #define NOTIFYFS_FSEVENT_FS_UNMOUNT                             16777216
> #define NOTIFYFS_FSEVENT_FS_NLINKS                              33554432

MfG,
	Bernd
-- 
Bernd Petrovitsch                  Email : bernd-JY8rjfDgnXcb9sJ47jD7nuTv7YV0F9Eg@public.gmane.org
                     LUGA : http://www.luga.at

WARNING: multiple messages have this Message-ID (diff)
From: Bernd Petrovitsch <bernd@petrovitsch.priv.at>
To: Stef Bon <stefbon@gmail.com>
Cc: Steve French <smfrench@gmail.com>,
	"linux-nfs@vger.kernel.org" <linux-nfs@vger.kernel.org>,
	linux-cifs <linux-cifs@vger.kernel.org>
Subject: Re: Possible to make nfs aware of a inotify watch has been set.
Date: Fri, 30 Nov 2012 14:37:20 +0100	[thread overview]
Message-ID: <1354282642.22426.7.camel@thorin> (raw)
In-Reply-To: <CANXojcwRy5cBz=E1S567J-1BFC4AiamLr=35bKrX0Zq-Me2b3w@mail.gmail.com>

Hi!

Since you introduced the "user experience" ignoring the fact that each
user is quite different - especially if it comes to the "Average Joes
using Ubuntu vs techies" department -, you should be prepared to eat
your own dog food:

On Fre, 2012-11-30 at 08:10 +0100, Stef Bon wrote:
[...]
> At this moment the different events look like:

Perhaps you should have used hexadecimal notation so that it is easier
for the usual/average users of source code to see the bits more easily.
In short: Please improve the user experience. Thank you.

> #define NOTIFYFS_FSEVENT_META_ATTRIB_NOTSET                     2
> #define NOTIFYFS_FSEVENT_META_ATTRIB_MODE                       4
> #define NOTIFYFS_FSEVENT_META_ATTRIB_OWNER                      8
> #define NOTIFYFS_FSEVENT_META_ATTRIB_GROUP                      16
> #define NOTIFYFS_FSEVENT_META_ATTRIB                            28
And that number seems pretty random - not a power of 2 or a bit mask or
the sum of the above.

> #define NOTIFYFS_FSEVENT_META_XATTR_NOTSET                      32
> #define NOTIFYFS_FSEVENT_META_XATTR_CREATE                      64
> #define NOTIFYFS_FSEVENT_META_XATTR_MODIFY                      128
> #define NOTIFYFS_FSEVENT_META_XATTR_DELETE                      256
> #define NOTIFYFS_FSEVENT_META_XATTR                             448
And that number seems pretty random too - not a power of 2 or a bit mask
or the sum of the above.

> #define NOTIFYFS_FSEVENT_FILE_NOTSET                            512
> #define NOTIFYFS_FSEVENT_FILE_MODIFIED                          1024
> #define NOTIFYFS_FSEVENT_FILE_SIZE                              2048
> #define NOTIFYFS_FSEVENT_FILE_OPEN                              4096
> #define NOTIFYFS_FSEVENT_FILE_READ                              8192
> #define NOTIFYFS_FSEVENT_FILE_CLOSE_WRITE                       16384
> #define NOTIFYFS_FSEVENT_FILE_CLOSE_NOWRITE                     32768

> #define NOTIFYFS_FSEVENT_MOVE_NOTSET                            65536

And from here own I'm too lazy to calculate (or even guess) if these are
actually powers of 2 or not.
> #define NOTIFYFS_FSEVENT_MOVE_CREATED                           131072
> #define NOTIFYFS_FSEVENT_MOVE_MOVED                             262144
> #define NOTIFYFS_FSEVENT_MOVE_MOVED_FROM                        524288
> #define NOTIFYFS_FSEVENT_MOVE_MOVED_TO                          1048576
> #define NOTIFYFS_FSEVENT_MOVE_DELETED                           2097152
> 
> #define NOTIFYFS_FSEVENT_FS_NOTSET                              4194304
> #define NOTIFYFS_FSEVENT_FS_MOUNT                               8388608
> #define NOTIFYFS_FSEVENT_FS_UNMOUNT                             16777216
> #define NOTIFYFS_FSEVENT_FS_NLINKS                              33554432

MfG,
	Bernd
-- 
Bernd Petrovitsch                  Email : bernd@petrovitsch.priv.at
                     LUGA : http://www.luga.at


  parent reply	other threads:[~2012-11-30 13:37 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-29  9:32 Possible to make nfs aware of a inotify watch has been set Stef Bon
2012-11-29  9:32 ` Stef Bon
     [not found] ` <CANXojcy9thLBwrENTkOTSSE17L3N17A8XsTmyNqq3oNdhW_Q_w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-11-29 14:03   ` Myklebust, Trond
2012-11-29 14:03     ` Myklebust, Trond
     [not found]     ` <4FA345DA4F4AE44899BD2B03EEEC2FA90B329CFD-VGJ86jkg8PeBTCzCEi1xmXOR4cbS7gtM96Bgd4bDwmQ@public.gmane.org>
2012-11-29 14:49       ` Stef Bon
2012-11-29 14:49         ` Stef Bon
     [not found]         ` <CANXojcxR9673fRObMLPSScogAN05RSVhPFoni=ENc6YDWq9T7w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-11-29 14:54           ` Myklebust, Trond
2012-11-29 14:54             ` Myklebust, Trond
2012-11-29 15:22           ` simo
2012-11-29 15:22             ` simo
     [not found]             ` <1354202525.4064.1.camel-fj0lwfvWodpMy5p6ylGyhR2eb7JE58TQ@public.gmane.org>
2012-11-29 15:33               ` Myklebust, Trond
2012-11-29 15:33                 ` Myklebust, Trond
     [not found]                 ` <4FA345DA4F4AE44899BD2B03EEEC2FA90B32AEB7-VGJ86jkg8PeBTCzCEi1xmXOR4cbS7gtM96Bgd4bDwmQ@public.gmane.org>
2012-11-29 16:28                   ` Steve French
2012-11-29 16:28                     ` Steve French
     [not found]                     ` <CAH2r5muVNPGx74ppc6MZPZRcsCRdP=xM4gvmrzEnw5bAb3jW8w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-11-29 17:05                       ` Myklebust, Trond
2012-11-29 17:05                         ` Myklebust, Trond
2012-11-29 16:39                   ` Stef Bon
2012-11-29 16:39                     ` Stef Bon
     [not found]                     ` <CANXojcys4wR8N-R3oSjWRWJLsObn0Qf+i41vBUaNnJctdcrKVw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-11-29 17:11                       ` Myklebust, Trond
2012-11-29 17:11                         ` Myklebust, Trond
     [not found]                         ` <4FA345DA4F4AE44899BD2B03EEEC2FA90B32B2A0-VGJ86jkg8PeBTCzCEi1xmXOR4cbS7gtM96Bgd4bDwmQ@public.gmane.org>
2012-11-29 20:05                           ` simo
2012-11-29 20:05                             ` simo
     [not found]                             ` <1354219557.4064.7.camel-fj0lwfvWodpMy5p6ylGyhR2eb7JE58TQ@public.gmane.org>
2012-11-29 21:09                               ` Myklebust, Trond
2012-11-29 21:09                                 ` Myklebust, Trond
     [not found]                                 ` <4FA345DA4F4AE44899BD2B03EEEC2FA90B32D9D4-VGJ86jkg8PeBTCzCEi1xmXOR4cbS7gtM96Bgd4bDwmQ@public.gmane.org>
2012-11-29 22:26                                   ` simo
2012-11-29 22:26                                     ` simo
2012-11-29 20:10                           ` Stef Bon
2012-11-29 20:10                             ` Stef Bon
2012-11-29 20:20                       ` Al Viro
2012-11-29 20:20                         ` Al Viro
     [not found]                         ` <20121129202037.GQ4939-3bDd1+5oDREiFSDQTTA3OLVCufUGDwFn@public.gmane.org>
2012-11-29 20:31                           ` Stef Bon
2012-11-29 20:31                             ` Stef Bon
     [not found]                             ` <CANXojcw15RdJdRoZxB3Z59YbOda7vG7RDRegfpoCjG4Bv4AvsQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-11-29 21:16                               ` Al Viro
2012-11-29 21:16                                 ` Al Viro
     [not found]                                 ` <20121129211624.GS4939-3bDd1+5oDREiFSDQTTA3OLVCufUGDwFn@public.gmane.org>
2012-11-29 23:14                                   ` Stef Bon
2012-11-29 23:14                                     ` Stef Bon
     [not found]                                     ` <CANXojcxfrO+Ar4JSd0NiDJh7+_gg6s6cq5=UA7Pu1h4DRZ8Vkg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-11-29 23:44                                       ` Steve French
2012-11-29 23:44                                         ` Steve French
     [not found]                                         ` <CAH2r5muw1U2XNMZsCq0gApuTMyj+6SN6S9KsUq52YM8CyZEG4A-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-11-30  7:10                                           ` Stef Bon
2012-11-30  7:10                                             ` Stef Bon
     [not found]                                             ` <CANXojcwRy5cBz=E1S567J-1BFC4AiamLr=35bKrX0Zq-Me2b3w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2012-11-30  7:19                                               ` Stef Bon
2012-11-30  7:19                                                 ` Stef Bon
2012-11-30 13:37                                               ` Bernd Petrovitsch [this message]
2012-11-30 13:37                                                 ` Bernd Petrovitsch

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=1354282642.22426.7.camel@thorin \
    --to=bernd-jy8rjfdgnxcb9sj47jd7nutv7yv0f9eg@public.gmane.org \
    --cc=linux-cifs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-nfs-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=smfrench-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=stefbon-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.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.