Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752578AbbHONye (ORCPT ); Sat, 15 Aug 2015 09:54:34 -0400 Received: from imap.thunk.org ([74.207.234.97]:60293 "EHLO imap.thunk.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751975AbbHONyc (ORCPT ); Sat, 15 Aug 2015 09:54:32 -0400 Date: Sat, 15 Aug 2015 09:54:22 -0400 From: "Theodore Ts'o" To: Michal Hocko Cc: Greg Thelen , LKML , linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, Andrew Morton , Johannes Weiner , Tetsuo Handa , Dave Chinner , linux-btrfs@vger.kernel.org, linux-ext4@vger.kernel.org, Jan Kara Subject: Re: [RFC 4/8] jbd, jbd2: Do not fail journal because of frozen_buffer allocation failure Message-ID: <20150815135422.GA2976@thunk.org> Mail-Followup-To: Theodore Ts'o , Michal Hocko , Greg Thelen , LKML , linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, Andrew Morton , Johannes Weiner , Tetsuo Handa , Dave Chinner , linux-btrfs@vger.kernel.org, linux-ext4@vger.kernel.org, Jan Kara References: <1438768284-30927-1-git-send-email-mhocko@kernel.org> <1438768284-30927-5-git-send-email-mhocko@kernel.org> <20150812091411.GB14940@dhcp22.suse.cz> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150812091411.GB14940@dhcp22.suse.cz> User-Agent: Mutt/1.5.23 (2014-03-12) X-SA-Exim-Connect-IP: X-SA-Exim-Mail-From: tytso@thunk.org X-SA-Exim-Scanned: No (on imap.thunk.org); SAEximRunCond expanded to false Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 634 Lines: 18 On Wed, Aug 12, 2015 at 11:14:11AM +0200, Michal Hocko wrote: > > Is this "if (!committed_data) {" check now dead code? > > > > I also see other similar suspected dead sites in the rest of the series. > > You are absolutely right. I have updated the patches. Have you sent out an updated version of these patches? Maybe I missed it, but I don't think I saw them. Thanks, - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/