From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8499FC433DF for ; Wed, 5 Aug 2020 20:11:30 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 511CD22B42 for ; Wed, 5 Aug 2020 20:11:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1596658290; bh=VH9ZXPRys3w5CJ+DJFHkL4Z9y1wA1TDZNwVzjEJQMoY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=NuKTNYBL5Q0eNI/l7P7neWGZBghTZWmnkC4noIX2erpgKh4r9KGcbJJUOgupE/ch5 /ePvKs5o4Qz8vQeQwZ1TOHzIyV0zOiqUqM+gK4wMi5ZLW08y1FxDx3D2CZq83/CH8w nhGwhgQrZiN6oB4Q3s8uNiHDoWEPIhhEoZGZdyq8= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729624AbgHEUL0 (ORCPT ); Wed, 5 Aug 2020 16:11:26 -0400 Received: from mail.kernel.org ([198.145.29.99]:49480 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727017AbgHEQ1n (ORCPT ); Wed, 5 Aug 2020 12:27:43 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 5C8732313B; Wed, 5 Aug 2020 14:32:25 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1596637945; bh=VH9ZXPRys3w5CJ+DJFHkL4Z9y1wA1TDZNwVzjEJQMoY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=mXDTUV14QDTkt/yjDUfiRuddYcnnNkrpHoQJN0sdCRrSbC6kvZNObNiIrf7fgPEYJ AKNuvxZZTkswKUcrd3u+YaLwgMm+maQ/RVsB/uzRcUMEZwwF3E+1s//nylQur456h1 JHkXhjqzCYS12U7tyxbtETyk/kuHKqQA/92F52ms= Date: Wed, 5 Aug 2020 16:32:42 +0200 From: Greg KH To: Mike Snitzer Cc: John Donnelly , Linux Kernel Mailing List , stable@vger.kernel.org Subject: Re: [(resend) PATCH v3: {linux-4.14.y} ] dm cache: submit writethrough writes in parallel to origin and cache Message-ID: <20200805143242.GC2154236@kroah.com> References: <8CFF8DA9-C105-461C-8F5A-DA2BF448A135@oracle.com> <20200804124735.GA219143@kroah.com> <20200804182037.GA15453@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200804182037.GA15453@redhat.com> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Aug 04, 2020 at 02:20:38PM -0400, Mike Snitzer wrote: > On Tue, Aug 04 2020 at 8:47am -0400, > Greg KH wrote: > > > On Tue, Aug 04, 2020 at 07:33:05AM -0500, John Donnelly wrote: > > > From: Mike Snitzer > > > > > > Discontinue issuing writethrough write IO in series to the origin and > > > then cache. > > > > > > Use bio_clone_fast() to create a new origin clone bio that will be > > > mapped to the origin device and then bio_chain() it to the bio that gets > > > remapped to the cache device. The origin clone bio does _not_ have a > > > copy of the per_bio_data -- as such check_if_tick_bio_needed() will not > > > be called. > > > > > > The cache bio (parent bio) will not complete until the origin bio has > > > completed -- this fulfills bio_clone_fast()'s requirements as well as > > > the requirement to not complete the original IO until the write IO has > > > completed to both the origin and cache device. > > > > > > Signed-off-by: Mike Snitzer > > > > > > (cherry picked from commit 2df3bae9a6543e90042291707b8db0cbfbae9ee9) > > > > > > Fixes: 4ec34f2196d125ff781170ddc6c3058c08ec5e73 (dm bio record: > > > save/restore bi_end_io and bi_integrity ) > > > > > > 4ec34f21 introduced a mkfs.ext4 hang on a LVM device that has been > > > modified with lvconvert --cachemode=writethrough. > > > > > > CC:stable@vger.kernel.org for 4.14.y > > > > > > Signed-off-by: John Donnelly > > > Reviewed-by: Somasundaram Krishnasamy > > > > > > conflicts: > > > drivers/md/dm-cache-target.c. - Corrected usage of > > > writethrough_mode(&cache->feature) that was caught by > > > compiler, and removed unused static functions : writethrough_endio(), > > > defer_writethrough_bio(), wake_deferred_writethrough_worker() > > > that generated warnings. > > > > What is this "conflicts nonsense"? You don't see that in any other > > kernel patch changelog, do you? > > > > > --- > > > drivers/md/dm-cache-target.c | 92 ++++++++++++++++++-------------------------- > > > 1 file changed, 37 insertions(+), 55 deletions(-) > > > > Please fix your email client up, it's totally broken and this does not > > work at all and is getting frustrating from my side here. > > > > Try sending emails to yourself and see if you can apply the patches, as > > the one you sent here does not work, again: > > John's inability to submit a patch that can apply aside: I do not like > how this patch header is constructed (yet attributed "From" me). It is > devoid of detail as it relates to stable@. > > Greg, please don't apply the v4 of this patch either. I'll craft a > proper stable@ patch that explains the reason for change and why we're > left having to resolve conflicts in stable@. > > But first I need to focus on sending DM changes to Linus for v5.9 merge. Ok, no worries, I'll drop all of these from my review queue and wait for something from you sometime in the future. thanks, greg k-h