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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <524f69650901221513m7857488by6713eca64ee31681@mail.gmail.com>
Date:	Thu, 22 Jan 2009 17:13:37 -0600
From:	Steve French <smfrench@...il.com>
To:	Christoph Hellwig <hch@...radead.org>
Cc:	Jeff Layton <jlayton@...hat.com>, linux-fsdevel@...r.kernel.org,
	linux-cifs-client@...ts.samba.org, linux-kernel@...r.kernel.org
Subject: Re: [linux-cifs-client] [PATCH] cifs: wrap cifs_dnotify_thread in 
	CONFIG_BROKEN

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@...radead.org> 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-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/