Re: [linux-cifs-client] [PATCH] cifs: wrap cifs_dnotify_thread in CONFIG_BROKEN

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The thread may be renamed/rewritten etc. - but we badly need to finish
the dnotify / inotify code ... removing the start on this that someone
wrote in the past isn't getting us any closer to this.

dnotify/inotify support was added to Linux in the first place because
Samba used it to handle this common operation (it is commonly used by
OTHER cifs clients) ... but none of the Linux cluster/network file
systems have had time to finish the code ... we need to finish this -
the network protocol portion of this is now well documented (and we
presumably will need a thread to process the dnotify multishot
notification responses so we don't clog up the demultiplex thread
waiting on kde and gnoe)

On Thu, Jan 22, 2009 at 3:06 PM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
> On Thu, Jan 22, 2009 at 11:34:38AM -0500, Jeff Layton wrote:
>> This thread never actually did anything useful, so comment it out by
>> #ifdef'ing it in CONFIG_BROKEN. My preference would be to remove it
>> altogether, but it's not my call.
>
> Please just kill it.  It's not only useless but also ugly, so keeping it
> around it a double disservice to the mother nature.
>
>



-- 
Thanks,

Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux