From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755411Ab2D3Il0 (ORCPT ); Mon, 30 Apr 2012 04:41:26 -0400 Received: from www17.your-server.de ([213.133.104.17]:32910 "EHLO www17.your-server.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753828Ab2D3IlY (ORCPT ); Mon, 30 Apr 2012 04:41:24 -0400 Message-ID: <1335775271.1363.4.camel@localhost.localdomain> Subject: Re: [PATCH v2] Introduce a version6 of autofs interface, to fix design error. From: Thomas Meyer To: Michael Tokarev Cc: Linus Torvalds , Alan Cox , David Miller , linux-kernel@vger.kernel.org, autofs@vger.kernel.org, raven@themaw.net, stable@kernel.org Date: Mon, 30 Apr 2012 10:41:11 +0200 In-Reply-To: <4F9D9C4C.7000904@msgid.tls.msk.ru> References: <20120427.151433.1007849975903946491.davem@davemloft.net> <20120427.151651.2274764174217164454.davem@davemloft.net> <20120427.152404.2292425516870981391.davem@davemloft.net> <20120427234215.079a1565@pyramind.ukuu.org.uk> <4F9CE18C.2060806@msgid.tls.msk.ru> <4F9CF191.5020607@msgid.tls.msk.ru> <4F9D9C4C.7000904@msgid.tls.msk.ru> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3 (3.2.3-3.fc16) Content-Transfer-Encoding: 7bit Mime-Version: 1.0 X-Authenticated-Sender: thomas@m3y3r.de Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am Sonntag, den 29.04.2012, 23:53 +0400 schrieb Michael Tokarev: > On 29.04.2012 23:09, Linus Torvalds wrote: > > > > This keeps Alan's idea of using O_DIRECT, but only makes it matter for > > the writer - because now the "packet" thing is a per-pipe-buffer state > > (we already had per-pipe-buffer flags, so this is not anything new). > > So when you *write* using a O_DIRECT pipe, it will create packetized > > buffers, and the reader just sees that directly. > > > > So this should "just work", and doesn't need any extra flags in the > > pipe_inode_info. And the user space interface remains the same: you > > can create these packet pipes with "pipe2(fd, O_DIRECT)" if you want > > to, and my test-program gives exactly the same output. > > Ok. I verified this on 3.3 kernel (with the original fix reverted), > and now both autofs5 and systemd works. hi, I can also confirm that "autofs: make the autofsv5 packet file descriptor use a packetized pipe" and "pipes: add a "packetized pipe" mode for writing" on top of 3.3.4 works correctly at least with the autofs feature of systemd. And sorry for the delayed answer, but I've been away from the internet for a few days. and a big sorry for all the trouble... with kind regards thomas