From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Wilcox Subject: Re: [Linux-nvdimm] [PATCH 4/6] SQUSHME: pmem: Micro cleaning Date: Tue, 31 Mar 2015 15:40:05 -0400 Message-ID: <20150331194005.GT4003@linux.intel.com> References: <1427358764-6126-1-git-send-email-hch@lst.de> <55143A8B.2060304@plexistor.com> <20150331092526.GA25958@lst.de> <551A9EB3.8000605@plexistor.com> <551AA0AD.4000604@plexistor.com> <551ABC26.3000008@plexistor.com> <551AC0AC.6080205@plexistor.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Dan Williams , Jens Axboe , linux-nvdimm , X86 ML , "linux-kernel@vger.kernel.org" , linux-fsdevel , Christoph Hellwig To: Boaz Harrosh Return-path: Received: from mga02.intel.com ([134.134.136.20]:13923 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751595AbbCaTkL (ORCPT ); Tue, 31 Mar 2015 15:40:11 -0400 Content-Disposition: inline In-Reply-To: <551AC0AC.6080205@plexistor.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Tue, Mar 31, 2015 at 06:43:40PM +0300, Boaz Harrosh wrote: > But not here. This is not an optimization this is the *error path*. > What I'm saying is: > "No compiler nor CPU, even if 99% of the time this branch is taken > I still consider it cold. Because it is the error case and > I do not care for it" GCC already understands that "if (foo) goto FORWARD_LABEL" should be predicted unlikely by default. All you're doing is cluttering the source code.