From mboxrd@z Thu Jan 1 00:00:00 1970 From: Al Viro Subject: Re: [GIT PULL] f2fs updates for v4.2 Date: Thu, 25 Jun 2015 07:32:36 +0100 Message-ID: <20150625063235.GV17109@ZenIV.linux.org.uk> References: <20150624202401.GA40031@jaegeuk-mac02.mot.com> <20150625043334.GT17109@ZenIV.linux.org.uk> <20150625054917.GA43469@jaegeuk-mac02.hsd1.ca.comcast.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: Linux FS Dev Mailing List , Stephen Rothwell , Linus Torvalds , Linux Kernel Mailing List , Linux F2FS Dev Mailing List To: Jaegeuk Kim Return-path: Content-Disposition: inline In-Reply-To: <20150625054917.GA43469@jaegeuk-mac02.hsd1.ca.comcast.net> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linux-f2fs-devel-bounces@lists.sourceforge.net List-Id: linux-fsdevel.vger.kernel.org On Wed, Jun 24, 2015 at 10:50:10PM -0700, Jaegeuk Kim wrote: > Right, I missed merging the fix-up patch in linux-next into my pull-request. > At a glance, I think there is no problem; except 80 column width, though. > > Also, agreed that I need to take a look at deleting the dentry to deal with that > failure case. Why not set the symlink up first and create a directory entry only if that succeeds? Looks like it's less work that way... ------------------------------------------------------------------------------ Monitor 25 network devices or servers for free with OpManager! OpManager is web-based network management software that monitors network devices and physical & virtual servers, alerts via email & sms for fault. Monitor 25 devices for free with no restriction. Download now http://ad.doubleclick.net/ddm/clk/292181274;119417398;o