From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Aneesh Kumar K. V" Subject: Re: [V9fs-developer] [PATCH 1/3] fs/9p: Update TLCREATE to allow create without open Date: Wed, 06 Oct 2010 11:48:15 +0530 Message-ID: References: <1286298090-2422-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain Cc: v9fs-developer@lists.sourceforge.net, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org To: Eric Van Hensbergen Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Tue, 5 Oct 2010 13:15:42 -0400, Eric Van Hensbergen wrote: > Ouch. Protocol change. > > Also - not sure I understand what's going on here. Why does mknod go > through TLCREATE, shouldn't we have our > own protocol messages for mknod?// This is needed for the below mknod usage mknod("k2", S_IFREG) ; > > Why add an additional flag field instead of just using the existing > flags field since we are only talking about a bit? > TLCREATE is size[4] Tlcreate tag[2] fid[4] name[s] flags[4] mode[4] gid[4] size[4] Rlcreate tag[2] qid[13] iounit[4] The flags argument represent Linux access mode flags with which the caller is requesting to open the file with. Protocol allows all the Linux access Having a -1 with the flag would indicate all the access mode flags are set. So i found that to be ugly. And value 0 indicate O_RDONLY. So was not sure whether -1 in flags or having a separate lookup flags is the right thing to do. -aneesh