inotify 0.15
Subsystems (4)
RFC PATCH 1/1 inotify 0.15
Hello,
In this release of inotify I have added a new ulimit for the number of watches a particular user can have. I am interested in peoples response to this addition and any alternative solutions.
Here is release 0.15.0 of inotify. Attached is a patch to 2.6.9
--New in this version--
-uses idr to manager watch descriptors (rml)
-drop limit on inotify instances (me)
-make watch limit a ulimit (me)
-dropped dnotify patch (me)
-misc cleanups (me,rml,Bryan Rittmeyer)
John McCutchan
Inotify documentation:
--HOWTO USE--
Inotify is a character device that when opened offers 2 IOCTL's.
INOTIFY_WATCH:
Which takes a path and event mask and returns a unique (to the instance of the driver) integer (wd [watch descriptor] from here on) that is a 1:1 mapping to the path passed. What happens is inotify refs the inode for the path and
adds a inotify_watch structure to the inode. Each inotify device instance can only watch an inode once.
INOTIFY_IGNORE:
Which takes an integer (that you got from INOTIFY_WATCH) representing a wd that you are not interested in watching anymore. This will:
send an IGNORE event to the device
remove the inotify_watch structure from the device and from the inode and unref the inode.
After you are watching 1 or more paths, you can read from the fd and get events. The events are struct inotify_event. If you are watching a directory and something happens to a file in the directory the event will contain the filename (just the filename not the full path).
-- EVENTS --IN_ACCESS - Sent when file is accessed.IN_MODIFY - Sent when file is modified.IN_ATTRIB - Sent when file is chmod'ed.IN_CLOSE_WRITE - Sent when file is closed and was opened for writing. IN_CLOSE_NOWRITE - Sent when file is closed but was not opened for writing.IN_CLOSE - either of the two above events.IN_OPEN - Sent when file is opened.IN_MOVED_FROM - Sent to the source folder of a move. IN_MOVED_TO - Sent to the destination folder of a move. IN_DELETE_SUBDIR - Sent when a sub directory is deleted.(When watching parent)IN_DELETE_FILE - Sent when a file is deleted. (When watching parent) IN_CREATE_SUBDIR - Sent when a sub directory is created.(When watching parent)IN_CREATE_FILE - Sent when a file is created. (When watching parent) IN_DELETE_SELF - Sent when file is deleted.IN_UNMOUNT - Sent when the filesystem is being unmounted. IN_Q_OVERFLOW - Sent when your event queue has over flowed.
The MOVED_FROM/MOVED_TO events are always sent in pairs. MOVED_FROM/MOVED_TO
is also sent when a file is renamed. The cookie field in the event structure
allows you to pair MOVED_FROM/MOVED_TO events. These two events are not guaranteed to be successive in the event stream. You must rely on the cookie to pair them up.
If you aren't watching the source and destination folders of a move/rename
You will only get MOVED_TO or MOVED_FROM. In this case, MOVED_TO is equivelent to a CREATE and MOVED_FROM is equivelent to a DELETE.
--Why Not dnotify and Why inotify (By Robert Love)--
Everyone seems quick to deride the blunder known as "dnotify" and applaud a
replacement, any replacement, man anything but that current mess, but in the
name of fairness I present my treatise on why dnotify is what one might call
not good:
* dnotify requires the opening of one fd per each directory that you intend to
watch.
o The file descriptor pins the directory, disallowing the backing device to be unmounted, which absolutely wrecks havoc with removable media.
o Watching many directories results in many open file descriptors, possibly hitting a per-process fd limit.
* dnotify is directory-based. You only learn about changes to directories. Sure, a change to a file in a directory affects the directory, but you are then forced to keep a cache of stat structures around to compare things in order to find out which file.
* dnotify's interface to user-space is awful.
o dnotify uses signals to communicate with user-space.
o Specifically, dnotify uses SIGIO.
o But then you can pick a different signal! So by "signals," I really meant you need to use real-time signals if you want to queue the events.
* dnotify basically ignores any problems that would arise in the VFS from hard
links.
* Rumor is that the "d" in "dnotify" does not stand for "directory" but for "suck."
A suitable replacement is "inotify." And now, my tract on what inotify brings
to the table:
* inotify's interface is a device node, not SIGIO.
o You open only a single fd, to the device node. No more pinning directories or opening a million file descriptors.
o Usage is nice: open the device, issue simple commands via ioctl(), and then block on the device. It returns events when, well, there are events to be returned.
o You can select() on the device node and so it integrates with main loops like coffee mixed with vanilla milkshake.
* inotify has an event that says "the filesystem that the item you were watching is on was unmounted" (this is particularly cool).
* inotify can watch directories or files.
* The "i" in inotify does not stand for "suck" but for "inode" -- the logical
choice since inotify is inode-based.
--COMPLEXITY--
I have been asked what the complexity of inotify is. Inotify has 2 path codes where complexity could be an issue:
Adding a watch to a device
This code has to check if the inode is already being watched by the device, this is O(N) where N is the number of devices watching the inode
Removing a watch from a device
This code
This code has to do a search of all watches on the device to find the watch descriptor that is being asked to remove. This involves a linear search, but should not really be an issue because it is limited to 8192 entries. If this does turn in to a concern, I would replace the list of watches on the device with a sorted binary tree, so that the search could be done very quickly.
In the near future this will be based on idr, and the complexity will be O(logn)
The calls to inotify from the VFS code has a complexity of O(1).
--MEMORY USAGE--
The inotify data structures are light weight:
inotify watch is 40 bytes
inotify device is 68 bytes
inotify event is 272 bytes
So assuming a device has 8192 watches, the structures are only going to consume 320KB of memory. With a maximum number of 8 devices allowed to exist at a time, this is still only 2.5 MB
Each device can also have 256 events queued at a time, which sums to 68KB per device. And only .5 MB if all devices are opened and have a full event queue.
So approximately 3 MB of memory are used in the rare case of everything open and full.
Each inotify watch pins the inode of a directory/file in memory, the size of an inode is different per file system but lets assume that it is 512 byes.
So assuming the maximum number of global watches are active, this would pin down 32 MB of inodes in the inode cache. Again not a problem on a modern system.
On smaller systems, the maximum watches / events could be lowered to provide a smaller foot print.
Keep in mind that this is an absolute worst case memory analysis. In reality it will most likely cost approximately 5MB.
--KERNEL CHANGES--
inotify char device driver.
Adding calls to inotify_inode_queue_event and inotify_dentry_parent_queue_event from VFS operations. Dnotify has the same function calls.
Adding a call to inotify_super_block_umount fromgeneric_shutdown_superblock
inotify_super_block_umount:
find all of the inodes that are on the super block being shut down foreach of these inodes:
remove the watchers on them
send UNMOUNT and IGNORED events
unref the inode
Signature · 1 line
John McCutchan <[email protected]>
drivers/char/inotify.c renamed from linux/drivers/char/inotify.c +963 -0
@@ -0,0 +1,963 @@ +/* +* Inode based directory notifications for Linux. +* +* Copyright (C) 2004 John McCutchan +* +* This program is free software; you can redistribute it and/or modify it +* under the terms of the GNU General Public License as published by the +* Free Software Foundation; either version 2, or (at your option) any +* later version. +* +* This program is distributed in the hope that it will be useful, but +* WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU +* General Public License for more details. +*/ + +/* TODO: +* unmount events don't get sent if filesystem is mounted in two places +* dynamically allocate event filename +*/ + +#include <linux/module.h> +#include <linux/kernel.h> +#include <linux/sched.h> +#include <linux/spinlock.h> +#include <linux/idr.h> +#include <linux/slab.h> +#include <linux/fs.h> +#include <linux/namei.h> +#include <linux/poll.h> +#include <linux/device.h> +#include <linux/miscdevice.h> +#include <linux/init.h> +#include <linux/list.h> +#include <linux/writeback.h> +#include <linux/inotify.h> + +static atomic_t watch_count; +static atomic_t inotify_cookie; +static kmem_cache_t *watch_cachep; +static kmem_cache_t *event_cachep; +static kmem_cache_t *inode_data_cachep; +static int sysfs_attrib_max_queued_events; + +/* +* struct inotify_device - represents an open instance of an inotify device +* +* For each inotify device, we need to keep track of the events queued on it, +* a list of the inodes that we are watching, and so on. +* +* This structure is protected by 'lock'. Lock ordering: +* +* inode->i_lock +* dev->lock +* dev->wait->lock +* +* FIXME: Look at replacing i_lock with i_sem. +*/ +struct inotify_device { +wait_queue_head_t wait; +struct idr idr; +struct list_head events; +struct list_head watches; +spinlock_t lock; +unsigned int event_count; +unsigned int nr_watches; +unsigned int max_events; +}; + +struct inotify_watch { +s32 wd;/* watch descriptor */ +u32 mask; +struct inode * inode; +struct inotify_device * dev; +struct list_head d_list;/* device list */ +struct list_head i_list;/* inode list */ +struct list_head u_list;/* unmount list */ +}; +#define inotify_watch_d_list (pos) list_entry((pos), struct inotify_watch, d_list) +#define inotify_watch_i_list (pos) list_entry((pos), struct inotify_watch, i_list) +#define inotify_watch_u_list (pos) list_entry((pos), struct inotify_watch, u_list) + +static ssize_t show_max_queued_events (struct device *dev,char *buf) +{ +sprintf (buf,"%d" , sysfs_attrib_max_queued_events); + +return strlen (buf)+1 ; +} + +static ssize_t store_max_queued_events (struct device *dev, +const char *buf, +size_t count) +{ +return 0 ; +} + +static DEVICE_ATTR(max_queued_events, S_IRUSR|S_IWUSR|S_IRGRP|S_IROTH, show_max_queued_events, store_max_queued_events); + +static struct device_attribute *inotify_device_attrs[] = { + &dev_attr_max_queued_events, +NULL +}; + + +/* +* A list of these is attached to each instance of the driver +* when the drivers read() gets called, this list is walked and +* all events that can fit in the buffer get delivered +*/ +struct inotify_kernel_event { +struct list_head list; +struct inotify_event event; +}; + +/* +* find_inode - resolve a user-given path to a specific inode and iget() it +*/ +static struct inode *find_inode (const char __user *dirname) +{ +struct inode *inode; +struct nameidata nd; +int error; + + error =__user_walk (dirname,LOOKUP_FOLLOW , &nd); +if (error) { + inode =ERR_PTR (error); + goto out; + } + + inode = nd.dentry->d_inode; + +/* you can only watch an inode if you have read permissions on it */ + error =vfs_permission (inode,MAY_READ ); +if (error) { + inode =ERR_PTR (error); + goto release_and_out; + } + +__iget (inode); +release_and_out: +path_release (&nd); +out: +return inode; +} + +static inline void unref_inode (struct inode *inode) +{ +iput (inode); +} + +struct inotify_kernel_event *kernel_event (s32 wd,u32 mask,u32 cookie, +const char *filename) +{ +struct inotify_kernel_event *kevent; + + kevent =kmem_cache_alloc (event_cachep,GFP_ATOMIC ); +if (!kevent) + goto out; + +/* we hand this out to user-space, so zero it out just in case */ +memset (kevent,0 ,sizeof (struct inotify_kernel_event )); + + kevent->event.wd = wd; + kevent->event.mask = mask; + kevent->event.cookie = cookie; +INIT_LIST_HEAD (&kevent->list); + +if (filename) { +strncpy (kevent->event.filename, filename, +INOTIFY_FILENAME_MAX ); + kevent->event.filename[INOTIFY_FILENAME_MAX -1 ] ='\0' ; + }else + kevent->event.filename[0 ] ='\0' ; + +out: +return kevent; +} + +void delete_kernel_event (struct inotify_kernel_event *kevent) +{ +if (!kevent) +return ; +kmem_cache_free (event_cachep, kevent); +} + +#define list_to_inotify_kernel_event (pos) list_entry((pos), struct inotify_kernel_event, list) +#define inotify_dev_get_event (dev) (list_to_inotify_kernel_event(dev->events.next)) +#define inotify_dev_has_events (dev) (!list_empty(&dev->events)) + +/* Does this events mask get sent to the watch ? */ +#define event_and (event_mask,watches_mask) ((event_mask == IN_UNMOUNT) || \ + (event_mask == IN_IGNORED) || \ + (event_mask & watches_mask)) + +/* +* inotify_dev_queue_event - add a new event to the given device +* +* Caller must hold dev->lock. +*/ +static void inotify_dev_queue_event (struct inotify_device *dev, +struct inotify_watch *watch,u32 mask, +u32 cookie,const char *filename) +{ +struct inotify_kernel_event *kevent, *last; + +/* +* Check if the new event is a duplicate of the last event queued. +*/ + last =inotify_dev_get_event (dev); +if (dev->event_count && last->event.mask == mask && + last->event.wd == watch->wd) { +/* Check if the filenames match */ +if (!filename && last->event.filename[0 ] =='\0' ) +return ; +if (filename && !strcmp (last->event.filename, filename)) +return ; + } + +/* +* the queue has already overflowed and we have already sent the +* Q_OVERFLOW event +*/ +if (dev->event_count > dev->max_events) +return ; + +/* the queue has just overflowed and we need to notify user space */ +if (dev->event_count == dev->max_events) { + kevent =kernel_event (-1 ,IN_Q_OVERFLOW , cookie,NULL ); + goto add_event_to_queue; + } + +if (!event_and (mask, watch->inode->inotify_data->watch_mask) || + !event_and (mask, watch->mask)) +return ; + + dev->event_count++; + kevent =kernel_event (watch->wd, mask, cookie, filename); + +add_event_to_queue: +if (!kevent) { + dev->event_count--; +return ; + } + +/* queue the event and wake up anyone waiting */ +list_add_tail (&kevent->list, &dev->events); +wake_up_interruptible (&dev->wait); +} + +/* +* inotify_dev_event_dequeue - destroy an event on the given device +* +* Caller must hold dev->lock. +*/ +static void inotify_dev_event_dequeue (struct inotify_device *dev) +{ +struct inotify_kernel_event *kevent; + +if (!inotify_dev_has_events (dev)) +return ; + + kevent =inotify_dev_get_event (dev); +list_del_init (&kevent->list); + dev->event_count--; +delete_kernel_event (kevent); + +} + +/* +* inotify_dev_get_wd - returns the next WD for use by the given dev +* +* This function can sleep. +*/ +static int inotify_dev_get_wd (struct inotify_device *dev, +struct inotify_watch *watch) +{ +int ret; + +if (!dev || !watch || dev->nr_watches >= current->rlim[RLIMIT_IWATCHES ].rlim_cur) +return -1 ; + +repeat: +if (!idr_pre_get (&dev->idr,GFP_KERNEL )) { +return -ENOSPC ; + } +spin_lock (&dev->lock); + ret =idr_get_new (&dev->idr, watch, &watch->wd); +spin_unlock (&dev->lock); +if (ret == -EAGAIN )/* more memory is required, try again */ + goto repeat; +else if (ret)/* the idr is full! */ +return -ENOSPC ; + + dev->nr_watches++; + +return 0 ; +} + +/* +* inotify_dev_put_wd - release the given WD on the given device +* +* Caller must hold dev->lock. +*/ +static int inotify_dev_put_wd (struct inotify_device *dev,s32 wd) +{ +if (!dev || wd <0 ) +return -1 ; + + dev->nr_watches--; +idr_remove (&dev->idr, wd); + +return 0 ; +} + +/* +* create_watch - creates a watch on the given device. +* +* Grabs dev->lock, so the caller must not hold it. +*/ +static struct inotify_watch *create_watch (struct inotify_device *dev, +u32 mask,struct inode *inode) +{ +struct inotify_watch *watch; + + watch =kmem_cache_alloc (watch_cachep,GFP_KERNEL ); +if (!watch) +return NULL ; + + watch->mask = mask; + watch->inode = inode; + watch->dev = dev; +INIT_LIST_HEAD (&watch->d_list); +INIT_LIST_HEAD (&watch->i_list); +INIT_LIST_HEAD (&watch->u_list); + +if (inotify_dev_get_wd (dev, watch)) { +kmem_cache_free (watch_cachep, watch); +return NULL ; + } + +return watch; +} + +/* +* delete_watch - removes the given 'watch' from the given 'dev' +* +* Caller must hold dev->lock. +*/ +static void delete_watch (struct inotify_device *dev, +struct inotify_watch *watch) +{ +inotify_dev_put_wd (dev, watch->wd); +kmem_cache_free (watch_cachep, watch); +} + +/* +* inotify_find_dev - find the watch associated with the given inode and dev +* +* Caller must hold dev->lock. +*/ +static struct inotify_watch *inode_find_dev (struct inode *inode, +struct inotify_device *dev) +{ +struct inotify_watch *watch; + +if (!inode->inotify_data) +return NULL ; + +list_for_each_entry (watch, &inode->inotify_data->watches, i_list) { +if (watch->dev == dev) +return watch; + } + +return NULL ; +} + +/* +* dev_find_wd - given a (dev,wd) pair, returns the matching inotify_watcher +* +* Returns the results of looking up (dev,wd) in the idr layer. NULL is +* returned on error. +* +* The caller must hold dev->lock. +*/ +static inline struct inotify_watch *dev_find_wd (struct inotify_device *dev, +u32 wd) +{ +return idr_find (&dev->idr, wd); +} + +static int inotify_dev_is_watching_inode (struct inotify_device *dev, +struct inode *inode) +{ +struct inotify_watch *watch; + +list_for_each_entry (watch, &dev->watches, d_list) { +if (watch->inode == inode) { +return 1 ; + } + } + +return 0 ; +} + +/* +* inotify_dev_add_watcher - add the given watcher to the given device instance +* +* Caller must hold dev->lock. +*/ +static int inotify_dev_add_watch (struct inotify_device *dev, +struct inotify_watch *watch) +{ +if (!dev || !watch) { +return -EINVAL ; + } + +list_add (&watch->d_list, &dev->watches); +return 0 ; +} + +/* +* inotify_dev_rm_watch - remove the given watch from the given device +* +* Caller must hold dev->lock because we call inotify_dev_queue_event(). +*/ +static int inotify_dev_rm_watch (struct inotify_device *dev, +struct inotify_watch *watch) +{ +if (!watch) +return -EINVAL ; + +inotify_dev_queue_event (dev, watch,IN_IGNORED ,0 ,NULL ); +list_del_init (&watch->d_list); + +return 0 ; +} + +void inode_update_watch_mask (struct inode *inode) +{ +struct inotify_watch *watch; +u32 new_mask; + +if (!inode->inotify_data) +return ; + + new_mask =0 ; +list_for_each_entry (watch, &inode->inotify_data->watches, i_list) + new_mask |= watch->mask; + + inode->inotify_data->watch_mask = new_mask; +} + +/* +* inode_add_watch - add a watch to the given inode +* +* Callers must hold dev->lock, because we call inode_find_dev(). +*/ +static int inode_add_watch (struct inode *inode, +struct inotify_watch *watch) +{ +if (!inode || !watch) +return -EINVAL ; + +/* +* This inode doesn't have an inotify_data structure attached to it +*/ +if (!inode->inotify_data) { + inode->inotify_data =kmem_cache_alloc (inode_data_cachep, +GFP_ATOMIC ); + +if (!inode->inotify_data) +return -ENOMEM ; + +INIT_LIST_HEAD (&inode->inotify_data->watches); + inode->inotify_data->watch_mask =0 ; + inode->inotify_data->watch_count =0 ; + } + +if (inode_find_dev (inode, watch->dev)) + { +return -EINVAL ; + } + +list_add (&watch->i_list, &inode->inotify_data->watches); + inode->inotify_data->watch_count++; +inode_update_watch_mask (inode); + +return 0 ; +} + +static int inode_rm_watch (struct inode *inode, +struct inotify_watch *watch) +{ +if (!inode || !watch || !inode->inotify_data) +return -EINVAL ; + +list_del_init (&watch->i_list); + inode->inotify_data->watch_count--; + +if (!inode->inotify_data->watch_count) { +kmem_cache_free (inode_data_cachep, inode->inotify_data); + inode->inotify_data =NULL ; +return 0 ; + } + +inode_update_watch_mask (inode); + +return 0 ; +} + +/* Kernel API */ + +void inotify_inode_queue_event (struct inode *inode,u32 mask,u32 cookie, +const char *filename) +{ +struct inotify_watch *watch; + +if (!inode->inotify_data) +return ; + +spin_lock (&inode->i_lock); + +list_for_each_entry (watch, &inode->inotify_data->watches, i_list) { +spin_lock (&watch->dev->lock); +inotify_dev_queue_event (watch->dev, watch, mask, cookie, + filename); +spin_unlock (&watch->dev->lock); + } + +spin_unlock (&inode->i_lock); +} +EXPORT_SYMBOL_GPL (inotify_inode_queue_event); + +void inotify_dentry_parent_queue_event (struct dentry *dentry,u32 mask, +u32 cookie,const char *filename) +{ +struct dentry *parent; + + parent =dget_parent (dentry); +inotify_inode_queue_event (parent->d_inode, mask, cookie, filename); +dput (parent); +} +EXPORT_SYMBOL_GPL (inotify_dentry_parent_queue_event); + +u32 inotify_get_cookie () +{ +atomic_inc (&inotify_cookie); + +return atomic_read (&inotify_cookie); +} +EXPORT_SYMBOL_GPL (inotify_get_cookie); + +static void ignore_helper (struct inotify_watch *watch,int event) +{ +struct inotify_device *dev; +struct inode *inode; + + inode = watch->inode; + dev = watch->dev; + +spin_lock (&inode->i_lock); +spin_lock (&dev->lock); + +if (event) +inotify_dev_queue_event (dev, watch, event,0 ,NULL ); + +inode_rm_watch (inode, watch); +inotify_dev_rm_watch (watch->dev, watch); +list_del (&watch->u_list); + +delete_watch (dev, watch); +spin_unlock (&dev->lock); +spin_unlock (&inode->i_lock); + +unref_inode (inode); +} + +static void process_umount_list (struct list_head *umount) +{ +struct inotify_watch *watch, *next; + +list_for_each_entry_safe (watch, next, umount, u_list) +ignore_helper (watch,IN_UNMOUNT ); +} + +/* +* build_umount_list - build a list of watches affected by an unmount. +* +* Caller must hold inode_lock. +*/ +static void build_umount_list (struct list_head *head,struct super_block *sb, +struct list_head *umount) +{ +struct inode *inode; + +list_for_each_entry (inode, head, i_list) { +struct inotify_watch *watch; + +if (inode->i_sb != sb) +continue ; + +if (!inode->inotify_data) +continue ; + +spin_lock (&inode->i_lock); + +list_for_each_entry (watch, &inode->inotify_data->watches, + i_list) { +list_add (&watch->u_list, umount); + } + +spin_unlock (&inode->i_lock); + } +} + +void inotify_super_block_umount (struct super_block *sb) +{ +struct list_head umount; + +INIT_LIST_HEAD (&umount); + +spin_lock (&inode_lock); +build_umount_list (&inode_in_use, sb, &umount); +spin_unlock (&inode_lock); + +process_umount_list (&umount); +} +EXPORT_SYMBOL_GPL (inotify_super_block_umount); + +/* +* inotify_inode_is_dead - an inode has been deleted, cleanup any watches +* +* FIXME: Callers need to always hold inode->i_lock. +*/ +void inotify_inode_is_dead (struct inode *inode) +{ +struct inotify_watch *watch, *next; +struct inotify_inode_data *data; + + data = inode->inotify_data; +if (!data) +return ; + +list_for_each_entry_safe (watch, next, &data->watches, i_list) +ignore_helper (watch,0 ); +} +EXPORT_SYMBOL_GPL (inotify_inode_is_dead); + +/* The driver interface is implemented below */ + +static unsigned int inotify_poll (struct file *file,poll_table *wait) +{ +struct inotify_device *dev; + + dev = file->private_data; + +poll_wait (file, &dev->wait, wait); + +if (inotify_dev_has_events (dev)) +return POLLIN |POLLRDNORM ; + +return 0 ; +} + +static ssize_t inotify_read (struct file *file,char __user *buf, +size_t count,loff_t *pos) +{ +size_t event_size; +struct inotify_device *dev; +char *start; +DECLARE_WAITQUEUE (wait, current); + + start = buf; + dev = file->private_data; + +/* We only hand out full inotify events */ + event_size =sizeof (struct inotify_event ); +if (count < event_size) +return -EINVAL ; + +while (1 ) { +int has_events; + +spin_lock (&dev->lock); + has_events =inotify_dev_has_events (dev); +spin_unlock (&dev->lock); +if (has_events) +break ; + +if (file->f_flags &O_NONBLOCK ) +return -EAGAIN ; + +if (signal_pending (current)) +return -ERESTARTSYS ; + +add_wait_queue (&dev->wait, &wait); +set_current_state (TASK_INTERRUPTIBLE ); + +schedule (); + +set_current_state (TASK_RUNNING ); +remove_wait_queue (&dev->wait, &wait); + } + +while (count >= event_size) { +struct inotify_kernel_event *kevent; + +spin_lock (&dev->lock); +if (!inotify_dev_has_events (dev)) { +spin_unlock (&dev->lock); +break ; + } + kevent =inotify_dev_get_event (dev); +spin_unlock (&dev->lock); +if (copy_to_user (buf, &kevent->event, event_size)) +return -EFAULT ; + +spin_lock (&dev->lock); +inotify_dev_event_dequeue (dev); +spin_unlock (&dev->lock); + count -= event_size; + buf += event_size; + } + +return buf - start; +} + +static int inotify_open (struct inode *inode,struct file *file) +{ +struct inotify_device *dev; + +atomic_inc (&watch_count); + + dev =kmalloc (sizeof (struct inotify_device ),GFP_KERNEL ); +if (!dev) +return -ENOMEM ; + +idr_init (&dev->idr); + +INIT_LIST_HEAD (&dev->events); +INIT_LIST_HEAD (&dev->watches); +init_waitqueue_head (&dev->wait); + + dev->event_count =0 ; + dev->nr_watches =0 ; + dev->lock =SPIN_LOCK_UNLOCKED ; + dev->max_events = sysfs_attrib_max_queued_events; + + file->private_data = dev; + +return 0 ; +} + +/* +* inotify_release_all_watches - destroy all watches on a given device +* +* FIXME: Do we want a lock here? +*/ +static void inotify_release_all_watches (struct inotify_device *dev) +{ +struct inotify_watch *watch,*next; + +list_for_each_entry_safe (watch, next, &dev->watches, d_list) { +ignore_helper (watch,0 ); + } +} + +/* +* inotify_release_all_events - destroy all of the events on a given device +*/ +static void inotify_release_all_events (struct inotify_device *dev) +{ +spin_lock (&dev->lock); +while (inotify_dev_has_events (dev)) +inotify_dev_event_dequeue (dev); +spin_unlock (&dev->lock); +} + +static int inotify_release (struct inode *inode,struct file *file) +{ +struct inotify_device *dev; + + dev = file->private_data; +inotify_release_all_watches (dev); +inotify_release_all_events (dev); +kfree (dev); + +atomic_dec (&watch_count); +return 0 ; +} + +static int inotify_watch (struct inotify_device *dev, +struct inotify_watch_request *request) +{ +struct inode *inode; +struct inotify_watch *watch; +int ret; + + inode =find_inode (request->dirname); +if (IS_ERR (inode)) { +return PTR_ERR (inode); + } + +spin_lock (&inode->i_lock); +spin_lock (&dev->lock); + +/* +* This handles the case of re-adding a directory we are already +* watching, we just update the mask and return 0 +*/ +if (inotify_dev_is_watching_inode (dev, inode)) { +struct inotify_watch *owatch;/* the old watch */ + + owatch =inode_find_dev (inode, dev); + owatch->mask = request->mask; +inode_update_watch_mask (inode); +spin_unlock (&dev->lock); +spin_unlock (&inode->i_lock); +unref_inode (inode); + +return owatch->wd; + } + +spin_unlock (&dev->lock); +spin_unlock (&inode->i_lock); + + watch =create_watch (dev, request->mask, inode); +if (!watch) { +unref_inode (inode); +return -ENOSPC ; + } + +spin_lock (&inode->i_lock); +spin_lock (&dev->lock); + +/* We can't add anymore watches to this device */ +if (inotify_dev_add_watch (dev, watch)) { +delete_watch (dev, watch); +spin_unlock (&dev->lock); +spin_unlock (&inode->i_lock); +unref_inode (inode); +return -EINVAL ; + } + + ret =inode_add_watch (inode, watch); +if (ret <0 ) { +list_del_init (&watch->d_list);/* inotify_dev_rm_watch w/o event */ +delete_watch (dev, watch); +spin_unlock (&dev->lock); +spin_unlock (&inode->i_lock); +unref_inode (inode); +return ret; + } + +spin_unlock (&dev->lock); +spin_unlock (&inode->i_lock); + +return watch->wd; +} + +static int inotify_ignore (struct inotify_device *dev,s32 wd) +{ +struct inotify_watch *watch; + +/* +* FIXME: Silly to grab dev->lock here and then drop it, when +* ignore_helper() grabs it anyway a few lines down. +*/ +spin_lock (&dev->lock); + watch =dev_find_wd (dev, wd); +spin_unlock (&dev->lock); +if (!watch) +return -EINVAL ; +ignore_helper (watch,0 ); + +return 0 ; +} + +/* +* inotify_ioctl() - our device file's ioctl method +* +* The VFS serializes all of our calls via the BKL and we rely on that. We +* could, alternatively, grab dev->lock. Right now lower levels grab that +* where needed. +*/ +static int inotify_ioctl (struct inode *ip,struct file *fp, +unsigned int cmd,unsigned long arg) +{ +struct inotify_device *dev; +struct inotify_watch_request request; +void __user *p; +s32 wd; + + dev = fp->private_data; + p = (void __user *) arg; + +switch (cmd) { +case INOTIFY_WATCH : +if (copy_from_user (&request, p,sizeof (request))) +return -EFAULT ; +return inotify_watch (dev, &request); +case INOTIFY_IGNORE : +if (copy_from_user (&wd, p,sizeof (wd))) +return -EFAULT ; +return inotify_ignore (dev, wd); +default : +return -ENOTTY ; + } +} + +static struct file_operations inotify_fops = { + .owner =THIS_MODULE , + .poll = inotify_poll, + .read = inotify_read, + .open = inotify_open, + .release = inotify_release, + .ioctl = inotify_ioctl, +}; + +struct miscdevice inotify_device = { + .minor =MISC_DYNAMIC_MINOR , + .name ="inotify" , + .fops = &inotify_fops, +}; + +static int __init inotify_init (void ) +{ +int ret,i; + + ret =misc_register (&inotify_device); +if (ret) +return ret; + + sysfs_attrib_max_queued_events =16384 ; + +for (i =0 ;inotify_device_attrs[i];i++) + { +device_create_file (inotify_device.dev, inotify_device_attrs[i]); + } + +atomic_set (&watch_count,0 ); +atomic_set (&inotify_cookie,0 ); + + watch_cachep =kmem_cache_create ("inotify_watch_cache" , +sizeof (struct inotify_watch ),0 ,SLAB_PANIC , +NULL ,NULL ); + + event_cachep =kmem_cache_create ("inotify_event_cache" , +sizeof (struct inotify_kernel_event ),0 , +SLAB_PANIC ,NULL ,NULL ); + + inode_data_cachep =kmem_cache_create ("inotify_inode_data_cache" , +sizeof (struct inotify_inode_data ),0 ,SLAB_PANIC , +NULL ,NULL ); + +printk (KERN_INFO "inotify device minor=%d\n" , inotify_device.minor); + +return 0 ; +} + + + +module_init (inotify_init);
include/linux/inotify.h renamed from linux/include/linux/inotify.h +156 -0
156 lines not on this page · raw message
include/linux/fs.h renamed from linux/include/linux/fs.h +7 -2
@@ -27,6 +27,7 @@struct kstatfs ;struct vm_area_struct ;struct vfsmount ; +struct inotify_inode_data ;/* * It's silly to have NR_OPEN bigger than NR_FILE, but you can change
@@ -61,7 +62,7 @@ };extern struct inodes_stat_t inodes_stat; -extern int leases_enable, dir_notify_enable, lease_break_time; +extern int leases_enable, lease_break_time;#define NR_FILE 8192/* this can well be larger on a larger system */ #define NR_RESERVED_FILES 10/* reserved for root */
@@ -462,6 +463,10 @@unsigned long i_dnotify_mask;/* Directory notify events */ struct dnotify_struct *i_dnotify;/* for directory notifications */ +#ifdef CONFIG_INOTIFY +struct inotify_inode_data *inotify_data; +#endif +unsigned long i_state;unsigned long dirtied_when;/* jiffies of first dirtying */
@@ -1336,7 +1341,7 @@extern int do_remount_sb (struct super_block *sb,int flags,void *data,int force);extern sector_t bmap (struct inode *,sector_t ); -extern int setattr_mask (unsigned int ); +extern void setattr_mask (unsigned int ,int *,u32 *);extern int notify_change (struct dentry *,struct iattr *);extern int permission (struct inode *,int ,struct nameidata *);extern int vfs_permission (struct inode *,int );
fs/super.c renamed from linux/fs/super.c +2 -0
@@ -36,6 +36,7 @@#include <linux/writeback.h> /* for the emergency remount stuff */ #include <linux/idr.h> #include <asm/uaccess.h> +#include <linux/inotify.h> void get_filesystem (struct file_system_type *fs);
@@ -225,6 +226,7 @@if (root) { sb->s_root =NULL ; +inotify_super_block_umount (sb);shrink_dcache_parent (root);shrink_dcache_anon (&sb->s_anon);dput (root);
fs/read_write.c renamed from linux/fs/read_write.c +26 -7
@@ -11,6 +11,7 @@#include <linux/uio.h> #include <linux/smp_lock.h> #include <linux/dnotify.h> +#include <linux/inotify.h> #include <linux/security.h> #include <linux/module.h>
@@ -216,8 +217,14 @@ ret = file->f_op->read (file, buf, count, pos); else ret =do_sync_read (file, buf, count, pos); -if (ret >0 ) -dnotify_parent (file->f_dentry,DN_ACCESS ); +if (ret >0 ) { +struct dentry *dentry = file->f_dentry; +dnotify_parent (dentry,DN_ACCESS ); +inotify_dentry_parent_queue_event (dentry, +IN_ACCESS ,0 , dentry->d_name.name); +inotify_inode_queue_event (inode,IN_ACCESS ,0 , +NULL ); + } } }
@@ -260,8 +267,14 @@ ret = file->f_op->write (file, buf, count, pos); else ret =do_sync_write (file, buf, count, pos); -if (ret >0 ) -dnotify_parent (file->f_dentry,DN_MODIFY ); +if (ret >0 ) { +struct dentry *dentry = file->f_dentry; +dnotify_parent (dentry,DN_MODIFY ); +inotify_dentry_parent_queue_event (dentry, +IN_MODIFY ,0 , dentry->d_name.name); +inotify_inode_queue_event (inode,IN_MODIFY ,0 , +NULL ); + } } }
@@ -493,9 +506,15 @@ out:if (iov != iovstack)kfree (iov); -if ((ret + (type ==READ )) >0 ) -dnotify_parent (file->f_dentry, - (type ==READ ) ?DN_ACCESS :DN_MODIFY ); +if ((ret + (type ==READ )) >0 ) { +struct dentry *dentry = file->f_dentry; +dnotify_parent (dentry, (type ==READ ) ?DN_ACCESS :DN_MODIFY ); +inotify_dentry_parent_queue_event (dentry, + (type ==READ ) ?IN_ACCESS :IN_MODIFY ,0 , + dentry->d_name.name); +inotify_inode_queue_event (dentry->d_inode, + (type ==READ ) ?IN_ACCESS :IN_MODIFY ,0 ,NULL ); + }return ret; }
fs/open.c renamed from linux/fs/open.c +5 -0
@@ -11,6 +11,7 @@#include <linux/smp_lock.h> #include <linux/quotaops.h> #include <linux/dnotify.h> +#include <linux/inotify.h> #include <linux/module.h> #include <linux/slab.h> #include <linux/tty.h>
@@ -955,6 +956,10 @@ error =PTR_ERR (f);if (IS_ERR (f)) goto out_error; +inotify_inode_queue_event (f->f_dentry->d_inode, +IN_OPEN ,0 ,NULL ); +inotify_dentry_parent_queue_event (f->f_dentry,IN_OPEN , +0 , f->f_dentry->d_name.name);fd_install (fd, f); } out:
fs/namei.c renamed from linux/fs/namei.c +34 -1
@@ -22,6 +22,7 @@#include <linux/quotaops.h> #include <linux/pagemap.h> #include <linux/dnotify.h> +#include <linux/inotify.h> #include <linux/smp_lock.h> #include <linux/personality.h> #include <linux/security.h>
@@ -1226,6 +1227,8 @@ error = dir->i_op->create (dir, dentry, mode, nd);if (!error) {inode_dir_notify (dir,DN_CREATE ); +inotify_inode_queue_event (dir,IN_CREATE_FILE , +0 , dentry->d_name.name);security_inode_post_create (dir, dentry, mode); }return error;
@@ -1540,6 +1543,8 @@ error = dir->i_op->mknod (dir, dentry, mode, dev);if (!error) {inode_dir_notify (dir,DN_CREATE ); +inotify_inode_queue_event (dir,IN_CREATE_FILE ,0 , + dentry->d_name.name);security_inode_post_mknod (dir, dentry, mode, dev); }return error;
@@ -1613,6 +1618,8 @@ error = dir->i_op->mkdir (dir, dentry, mode);if (!error) {inode_dir_notify (dir,DN_CREATE ); +inotify_inode_queue_event (dir,IN_CREATE_SUBDIR ,0 , + dentry->d_name.name);security_inode_post_mkdir (dir,dentry, mode); }return error;
@@ -1708,6 +1715,11 @@up (&dentry->d_inode->i_sem);if (!error) {inode_dir_notify (dir,DN_DELETE ); +inotify_inode_queue_event (dir,IN_DELETE_SUBDIR ,0 , + dentry->d_name.name); +inotify_inode_queue_event (dentry->d_inode,IN_DELETE_SELF ,0 , +NULL ); +inotify_inode_is_dead (dentry->d_inode);d_delete (dentry); }dput (dentry);
@@ -1780,8 +1792,13 @@/* We don't d_delete() NFS sillyrenamed files--they still exist. */ if (!error && !(dentry->d_flags &DCACHE_NFSFS_RENAMED )) { -d_delete (dentry);inode_dir_notify (dir,DN_DELETE ); +inotify_inode_queue_event (dir,IN_DELETE_FILE ,0 , + dentry->d_name.name); +inotify_inode_queue_event (dentry->d_inode,IN_DELETE_SELF ,0 , +NULL ); +inotify_inode_is_dead (dentry->d_inode); +d_delete (dentry); }return error; }
@@ -1857,6 +1874,8 @@ error = dir->i_op->symlink (dir, dentry, oldname);if (!error) {inode_dir_notify (dir,DN_CREATE ); +inotify_inode_queue_event (dir,IN_CREATE_FILE ,0 , + dentry->d_name.name);security_inode_post_symlink (dir, dentry, oldname); }return error;
@@ -1930,6 +1949,8 @@up (&old_dentry->d_inode->i_sem);if (!error) {inode_dir_notify (dir,DN_CREATE ); +inotify_inode_queue_event (dir,IN_CREATE_FILE ,0 , + new_dentry->d_name.name);security_inode_post_link (old_dentry, dir, new_dentry); }return error;
@@ -2093,6 +2114,8 @@ {int error;int is_dir =S_ISDIR (old_dentry->d_inode->i_mode); +char *old_name; +u32 cookie;if (old_dentry->d_inode == new_dentry->d_inode)return 0 ;
@@ -2114,6 +2137,8 @@DQUOT_INIT (old_dir);DQUOT_INIT (new_dir); + old_name =inotify_oldname_init (old_dentry); +if (is_dir) error =vfs_rename_dir (old_dir,old_dentry,new_dir,new_dentry);else
@@ -2125,7 +2150,15 @@inode_dir_notify (old_dir,DN_DELETE );inode_dir_notify (new_dir,DN_CREATE ); } + + cookie =inotify_get_cookie (); + +inotify_inode_queue_event (old_dir,IN_MOVED_FROM , cookie, old_name); +inotify_inode_queue_event (new_dir,IN_MOVED_TO , cookie, + new_dentry->d_name.name); } +inotify_oldname_free (old_name); +return error; }
fs/inode.c renamed from linux/fs/inode.c +3 -0
@@ -114,6 +114,9 @@if (inode) {struct address_space *const mapping = &inode->i_data; +#ifdef CONFIG_INOTIFY + inode->inotify_data =NULL ; +#endif inode->i_sb = sb; inode->i_blkbits = sb->s_blocksize_bits; inode->i_flags =0 ;
fs/attr.c renamed from linux/fs/attr.c +53 -20
97 lines not on this page · raw message
fs/file_table.c renamed from linux/fs/file_table.c +7 -0
@@ -16,6 +16,7 @@#include <linux/eventpoll.h> #include <linux/mount.h> #include <linux/cdev.h> +#include <linux/inotify.h> /* sysctl tunables... */ struct files_stat_struct files_stat = {
@@ -122,6 +123,12 @@struct dentry *dentry = file->f_dentry;struct vfsmount *mnt = file->f_vfsmnt;struct inode *inode = dentry->d_inode; +u32 mask; + + + mask = (file->f_mode &FMODE_WRITE ) ?IN_CLOSE_WRITE :IN_CLOSE_NOWRITE ; +inotify_dentry_parent_queue_event (dentry, mask,0 , dentry->d_name.name); +inotify_inode_queue_event (inode, mask,0 ,NULL );might_sleep ();/*
drivers/char/Makefile renamed from linux/drivers/char/Makefile +2 -0
@@ -9,6 +9,8 @@ obj-y += mem.o random.o tty_io.o n_tty.o tty_ioctl.o + +obj-$(CONFIG_INOTIFY) += inotify.o obj-$(CONFIG_LEGACY_PTYS) += pty.o obj-$(CONFIG_UNIX98_PTYS) += pty.o obj-y += misc.o
drivers/char/Kconfig renamed from linux/drivers/char/Kconfig +13 -0
@@ -62,6 +62,19 @@ depends on VT && !S390 && !USERMODE default y +config INOTIFY + bool "Inotify file change notification support" + default y + ---help--- + Say Y here to enable inotify support and the /dev/inotify character + device. Inotify is a file change notification system and a + replacement for dnotify. Inotify fixes numerous shortcomings in + dnotify and introduces several new features. It allows monitoring + of both files and directories via a single open fd. Multiple file + events are supported. + + If unsure, say Y. + config SERIAL_NONSTANDARD bool "Non-standard serial port support" ---help---
include/linux/idr.h renamed from linux/include/linux/idr.h +1 -0
@@ -29,6 +29,7 @@# error "BITS_PER_LONG is not 32 or 64"#endif +#define IDR_MAX_ID 0x7fffffff#define IDR_SIZE (1 << IDR_BITS)#define IDR_MASK ((1 << IDR_BITS)-1)
include/asm-i386/resource.h renamed from linux/include/asm-i386/resource.h +3 -1
@@ -18,8 +18,9 @@#define RLIMIT_LOCKS 10/* maximum file locks held */ #define RLIMIT_SIGPENDING 11/* max number of pending signals */ #define RLIMIT_MSGQUEUE 12/* maximum bytes in POSIX mqueues */ +#define RLIMIT_IWATCHES 13/* maximum number of inotify watches */ -#define RLIM_NLIMITS 13 +#define RLIM_NLIMITS 14/*
@@ -45,6 +46,7 @@ { RLIM_INFINITY, RLIM_INFINITY }, \ { MAX_SIGPENDING, MAX_SIGPENDING }, \ { MQ_BYTES_MAX, MQ_BYTES_MAX }, \ + { IWATCHES_SOFT_LIMIT, IDR_MAX_ID }, \ } #endif /* __KERNEL__ */
arch/i386/kernel/init_task.c renamed from linux/arch/i386/kernel/init_task.c +2 -0
@@ -5,6 +5,8 @@#include <linux/init_task.h> #include <linux/fs.h> #include <linux/mqueue.h> +#include <linux/idr.h> +#include <linux/inotify.h> #include <asm/uaccess.h> #include <asm/pgtable.h>
Quoted 5 lines · John McCutchan
In this release of inotify I have added a new ulimit for the number of watches a particular user can have. I am interested in peoples response to this addition and any alternative solutions.
Here is release 0.15.0 of inotify. Attached is a patch to 2.6.9
By and by, people using 2.6.10-rc1 or -mm or whatever will find that several interfaces used by this patch coincidentally changed.
So I thought I'd point you all at an updated 2.6.10-rc1 patch:
http://www.kernel.org/pub/linux/kernel/people/rml/inotify/v2.6/0.15/
Which might be easier.
Robert Love
PATCH 1/1 inotify: change default number of queued events
In reply to John McCutchanHola, John.
The default of 16384 maximum queued events is way too high. That is a lot of physical memory that the user can pin.
Attached patch takes it to 512.
Robert Love
change the default number of queued events down from 16384 to 512
drivers/char/inotify.c | 2 +- 1 files changed, 1 insertion(+), 1 deletion(-)
drivers/char/inotify.c +1 -1
@@ -921,7 +921,7 @@if (ret)return ret; - sysfs_attrib_max_queued_events =16384 ; + sysfs_attrib_max_queued_events =512 ;for (i =0 ;inotify_device_attrs[i];i++)device_create_file (inotify_device.dev, inotify_device_attrs[i]);
PATCH 1/1 inotify: misc. style cleanup
In reply to John McCutchanJohn!
Misc. style clean up.
Robert Love
misc. style cleanup
drivers/char/inotify.c | 46 ++++++++++++++++------------------------------ 1 files changed, 16 insertions(+), 30 deletions(-)
drivers/char/inotify.c +16 -30
@@ -80,21 +80,20 @@#define inotify_watch_i_list (pos) list_entry((pos), struct inotify_watch, i_list)#define inotify_watch_u_list (pos) list_entry((pos), struct inotify_watch, u_list) -static ssize_t show_max_queued_events (struct device *dev,char *buf) +static ssize_t show_max_queued_events (struct device *dev,char *buf) {sprintf (buf,"%d" , sysfs_attrib_max_queued_events); - -return strlen (buf)+1 ; +return strlen (buf) +1 ; } -static ssize_t store_max_queued_events (struct device *dev, -const char *buf, -size_t count) +static ssize_t store_max_queued_events (struct device *dev,const char *buf, +size_t count) {return 0 ; } -static DEVICE_ATTR(max_queued_events, S_IRUSR|S_IWUSR|S_IRGRP|S_IROTH, show_max_queued_events, store_max_queued_events); +static DEVICE_ATTR(max_queued_events, S_IRUGO | S_IWUSR, show_max_queued_events, + store_max_queued_events);static struct device_attribute *inotify_device_attrs[] = { &dev_attr_max_queued_events,
@@ -203,9 +202,7 @@ {struct inotify_kernel_event *kevent, *last; -/* -* Check if the new event is a duplicate of the last event queued. -*/ +/* check if the new event is a duplicate of the last event queued. */ last =inotify_dev_get_event (dev);if (dev->event_count && last->event.mask == mask && last->event.wd == watch->wd) {
@@ -272,17 +269,16 @@* This function can sleep. */ static int inotify_dev_get_wd (struct inotify_device *dev, -struct inotify_watch *watch) +struct inotify_watch *watch) {int ret; -if (!dev || !watch || dev->nr_watches >= current->signal->rlim[RLIMIT_IWATCHES ].rlim_cur) +if (dev->nr_watches >= current->rlim[RLIMIT_IWATCHES ].rlim_cur)return -1 ; repeat: -if (!idr_pre_get (&dev->idr,GFP_KERNEL )) { +if (!idr_pre_get (&dev->idr,GFP_KERNEL ))return -ENOSPC ; - }spin_lock (&dev->lock); ret =idr_get_new (&dev->idr, watch, &watch->wd);spin_unlock (&dev->lock);
@@ -394,9 +390,8 @@struct inotify_watch *watch;list_for_each_entry (watch, &dev->watches, d_list) { -if (watch->inode == inode) { +if (watch->inode == inode)return 1 ; - } }return 0 ;
@@ -410,9 +405,8 @@static int inotify_dev_add_watch (struct inotify_device *dev,struct inotify_watch *watch) { -if (!dev || !watch) { +if (!dev || !watch)return -EINVAL ; - }list_add (&watch->d_list, &dev->watches);return 0 ;
@@ -477,9 +471,7 @@ }if (inode_find_dev (inode, watch->dev)) - {return -EINVAL ; - }list_add (&watch->i_list, &inode->inotify_data->watches); inode->inotify_data->watch_count++;
@@ -678,7 +670,7 @@if (count < event_size)return -EINVAL ; -while (1 ) { +while (1 ) {int has_events;spin_lock (&dev->lock);
@@ -760,9 +752,8 @@ {struct inotify_watch *watch,*next; -list_for_each_entry_safe (watch, next, &dev->watches, d_list) { +list_for_each_entry_safe (watch, next, &dev->watches, d_list)ignore_helper (watch,0 ); - } }/*
@@ -797,9 +788,8 @@int ret; inode =find_inode (request->dirname); -if (IS_ERR (inode)) { +if (IS_ERR (inode))return PTR_ERR (inode); - }spin_lock (&inode->i_lock);spin_lock (&dev->lock);
@@ -934,9 +924,7 @@ sysfs_attrib_max_queued_events =16384 ;for (i =0 ;inotify_device_attrs[i];i++) - { -device_create_file (inotify_device.dev, inotify_device_attrs[i]); - } +device_create_file (inotify_device.dev, inotify_device_attrs[i]);atomic_set (&watch_count,0 );atomic_set (&inotify_cookie,0 );
@@ -958,6 +946,4 @@return 0 ; } - -module_init (inotify_init);
PATCH 1/1 inotify: idr stuff
In reply to John McCutchanJohn.
MAX_ID_MASK is already set to the maximum IDR value, so we don't need to add IDR_MAX_ID. I know this was my idea--I was, uh, being metaphoric. :)
Plus, MAX_ID_MASK is computed based on IDR_BITS, so it is "always right."
Also add an IWATCHES_HARD_LIMIT, set to MAX_ID_MASK, instead of using MAX_ID_MASK directly.
Finally, this uncovers that idr.h is not protected against double inclusion, so wrap it in ifndef's.
Best,
Robert Love
idr stuff
include/asm-i386/resource.h | 4 ++-- include/linux/idr.h | 5 ++++- include/linux/inotify.h | 4 +++- 3 files changed, 9 insertions(+), 4 deletions(-)
include/asm-i386/resource.h +2 -2
@@ -18,7 +18,7 @@#define RLIMIT_LOCKS 10/* maximum file locks held */ #define RLIMIT_SIGPENDING 11/* max number of pending signals */ #define RLIMIT_MSGQUEUE 12/* maximum bytes in POSIX mqueues */ -#define RLIMIT_IWATCHES 13/* maximum number of inotify watches */ +#define RLIMIT_IWATCHES 13/* maximum number of inotify watches */ #define RLIM_NLIMITS 14
@@ -46,7 +46,7 @@ { RLIM_INFINITY, RLIM_INFINITY }, \ { MAX_SIGPENDING, MAX_SIGPENDING }, \ { MQ_BYTES_MAX, MQ_BYTES_MAX }, \ - { IWATCHES_SOFT_LIMIT, IDR_MAX_ID }, \ + { IWATCHES_SOFT_LIMIT, IWATCHES_HARD_LIMIT }, \ } #endif /* __KERNEL__ */
include/linux/idr.h +4 -1
@@ -1,3 +1,5 @@ +#ifndef _LINUX_IDR_H +#define _LINUX_IDR_H/* * include/linux/idr.h *
@@ -29,7 +31,6 @@# error "BITS_PER_LONG is not 32 or 64"#endif -#define IDR_MAX_ID 0x7fffffff#define IDR_SIZE (1 << IDR_BITS)#define IDR_MASK ((1 << IDR_BITS)-1)
@@ -77,3 +78,5 @@int idr_get_new_above (struct idr *idp,void *ptr,int starting_id,int *id);void idr_remove (struct idr *idp,int id);void idr_init (struct idr *idp); + +#endif /* _LINUX_IDR_H */
include/linux/inotify.h +3 -1
@@ -72,8 +72,10 @@#include <linux/dcache.h> #include <linux/fs.h> #include <linux/config.h> +#include <linux/idr.h> -#define IWATCHES_SOFT_LIMIT 16384 +#define IWATCHES_SOFT_LIMIT 16384 +#define IWATCHES_HARD_LIMIT MAX_ID_MASKstruct inotify_inode_data {struct list_head watches;
PATCH 1/1 inotify: fix dnotify compile
In reply to John McCutchanHi, John.
Looks like the dnotify patch was only half removed. The declaration of dir_notify_enable in include/linux/fs.h is still removed by the patch. So e.g. sysctl.c does not compile.
You can remove that hunk from the patch or commit the following.
Thanks,
Robert Love
fix CONFIG_DNOTIFY compile
include/linux/fs.h | 2 +- 1 files changed, 1 insertion(+), 1 deletion(-)
include/linux/fs.h +1 -1
@@ -62,7 +62,7 @@ };extern struct inodes_stat_t inodes_stat; -extern int leases_enable, lease_break_time; +extern int leases_enable, dir_notify_enable, lease_break_time;#define NR_FILE 8192/* this can well be larger on a larger system */ #define NR_RESERVED_FILES 10/* reserved for root */
Re: [patch] inotify: idr stuff
In reply to Robert LoveQuoted 12 lines · Robert Love
John.
MAX_ID_MASKis already set to the maximum IDR value, so we don't need to addIDR_MAX_ID. I know this was my idea--I was, uh, being metaphoric. :)Plus,
MAX_ID_MASKis computed based onIDR_BITS, so it is "always right."Also add an
IWATCHES_HARD_LIMIT, set toMAX_ID_MASK, instead of usingMAX_ID_MASKdirectly.
Quoted 6 lines
Finally, this uncovers that
idr.his not protected against double inclusion, so wrap it in ifndef's.Best,
Robert Love
Anybody know whats going on, I just got 8 copies of this [...]
Signature · 8 lines
Cheers, Gene "There are four boxes to be used in defense of liberty: soap, ballot, jury, and ammo. Please use in that order." -Ed Howdershelt (Author) 99.28% setiathome rank, not too shabby for a WV hillbilly Yahoo.com attorneys please note, additions to this message by Gene Heskett are: Copyright 2004 by Maurice Eugene Heskett, all rights reserved.