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.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,MAILING_LIST_MULTI, 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 59DFEC4338F for ; Wed, 11 Aug 2021 05:28:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 2EEBA60F55 for ; Wed, 11 Aug 2021 05:28:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233389AbhHKF3B (ORCPT ); Wed, 11 Aug 2021 01:29:01 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43678 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229730AbhHKF3B (ORCPT ); Wed, 11 Aug 2021 01:29:01 -0400 Received: from mail-lf1-x12d.google.com (mail-lf1-x12d.google.com [IPv6:2a00:1450:4864:20::12d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BB9E0C061765 for ; Tue, 10 Aug 2021 22:28:37 -0700 (PDT) Received: by mail-lf1-x12d.google.com with SMTP id x27so3128712lfu.5 for ; Tue, 10 Aug 2021 22:28:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=2RXbu2o75IfkBPqzArkAlf82epBagmbRna3iSceAioc=; b=Hw7+G+b4gQByoehKjSo6pgK1X3/mH69A2uxNc3F0zkB9EiqsyPv7H4sIHCtbenoNxS WwYTRzIRVBCjHDPgebeiCbtneMWGXXrnyZDXe6HeDP9QOBVNTK+AiATamskl+IFKMXMe ieYeVroeSPZ0K+YvbxAS2b2i2ousGa1y8m5BmT9/sJ/eGI3/GSDVn2O+Ty4ZHXAbfa0M /Vj/sI0iaTVdViYKKD9jPCbM48J22Tsh+bgUcFg1VKr5skLOAx6/xRZxuHSoEa4bv57S dOJH/QCqovAYnSTRsoKo8g0pN2np0ZJ2zs8QoWJZwb149Rs0pCviYTwBe6VgRH1QfZVJ Mp5A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=2RXbu2o75IfkBPqzArkAlf82epBagmbRna3iSceAioc=; b=tm4lC/2pItJWhNDtCb0eEBRpVYwMgOGrZg1OdHQZH7cYfEo/hIioAsADO3ZE7n1q+K 70nsAE2fxGuDhKTIDd3te3ezE18a8HaM0hTlVPVUmYR29HFrUWtb92F8IQlQE1EjLX6W uzkki+ZOwBxhoERUxaGEd+8b/65IbPB9ro5NZperqpNL/5WcvNs6Kgwvf1jdDRjFqvEn 5o6+qG56FagFkcaQ0tb3B/IauGkvylVtQbQ1hsAAuUy2ehimtwhMBWtqMX0ZelB0zosi UBFdIHIOwIfPe2ndoGRdebrlu9zsLZ5BYgxZZ+rBGtnf0HHfl3pgV9dvka0oqbd8yQP4 5g+A== X-Gm-Message-State: AOAM531jbvHRc0PelqU+8aSwrLKfnTd5rqegXfkNufEe+GNhUl2W1zSZ 2BskZPuC/TRIpxSgRUVdJei33dEh4he2Djuh3IA= X-Google-Smtp-Source: ABdhPJw9sC1dPs8Ga9hV3+ruWM+zPMBizaLRSNhA9vAhGBAKkwnpknKnPSdx6GcBdbd3Hk2yEUlzQFmwXEsa8gtZO+k= X-Received: by 2002:ac2:559c:: with SMTP id v28mr2442969lfg.133.1628659715914; Tue, 10 Aug 2021 22:28:35 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Steve French Date: Wed, 11 Aug 2021 00:28:25 -0500 Message-ID: Subject: Re: [PATCH][CIFS]Call close synchronously during unlink/rename/lease break. To: Rohith Surabattula Cc: Shyam Prasad N , linux-cifs , ronnie sahlberg Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-cifs@vger.kernel.org I added Cc: stable@vger.kernel.org # 5.13 Sound correct? On Tue, Aug 10, 2021 at 2:48 PM Rohith Surabattula wrote: > > Hi All, > > I have attached an old version of patch by mistake. > Have attached the correct one. > > Sorry for the confusion. > > Regards. > Rohith > > On Wed, Aug 11, 2021 at 1:02 AM Steve French wrote: > > > > This is in http://smb3-test-rhel-75.southcentralus.cloudapp.azure.com/#/builders/2/builds/766 > > > > On Tue, Aug 10, 2021 at 2:28 PM Steve French wrote: > > > > > > I saw some test failures in > > > > > > and looking in /var/log/messages I see e.g. > > > > > > Aug 10 14:00:22 fedora29 systemd[1]: Started /usr/bin/bash -c exit 77. > > > Aug 10 14:00:25 fedora29 kernel: kmemleak: 10 new suspected memory > > > leaks (see /sys/kernel/debug/kmemleak) > > > Aug 10 14:00:28 fedora29 kernel: kmemleak: 1 new suspected memory > > > leaks (see /sys/kernel/debug/kmemleak) > > > Aug 10 14:00:28 fedora29 kernel: CIFS: Attempting to mount > > > \\win16.vm.test\Scratch > > > Aug 10 14:00:28 fedora29 kernel: CIFS: Attempting to mount > > > \\win16.vm.test\Scratch > > > Aug 10 14:00:29 fedora29 root[6460]: run xfstest cifs/100 > > > Aug 10 14:00:29 fedora29 journal: run fstests cifs/100 at 2021-08-10 14:00:29 > > > Aug 10 14:00:29 fedora29 systemd[1]: Started /usr/bin/bash -c test -w > > > /proc/self/oom_score_adj && echo 250 > /proc/self/oom_score_adj; exec > > > ./tests/cifs/100. > > > Aug 10 14:00:30 fedora29 kernel: general protection fault, probably > > > for non-canonical address 0xdead000000000110: 0000 [#4] SMP PTI > > > Aug 10 14:00:30 fedora29 kernel: CPU: 0 PID: 6624 Comm: rm Tainted: G > > > D 5.14.0-rc4 #1 > > > Aug 10 14:00:30 fedora29 kernel: Hardware name: Red Hat KVM, BIOS > > > 0.5.1 01/01/2011 > > > Aug 10 14:00:30 fedora29 kernel: RIP: > > > 0010:cifs_close_deferred_file+0x77/0x140 [cifs] > > > Aug 10 14:00:30 fedora29 kernel: Code: 85 de 00 00 00 4c 89 ef e8 06 > > > 96 2e e8 48 8b 1c 24 4c 39 e3 74 55 49 bd 00 01 00 00 00 00 ad de 48 > > > bd 22 01 00 00 00 00 ad de <48> 8b 7b 10 31 d2 be 01 00 00 00 e8 a9 c5 > > > fe ff 48 89 df e8 61 46 > > > Aug 10 14:00:30 fedora29 kernel: RSP: 0018:ffffb121806dbdd0 EFLAGS: 00010287 > > > Aug 10 14:00:30 fedora29 kernel: RAX: ffff88dc55b8ca01 RBX: > > > dead000000000100 RCX: 00000000000034d9 > > > Aug 10 14:00:30 fedora29 kernel: RDX: 00000000000034d8 RSI: > > > 67739b6f43694caa RDI: 0000000000032080 > > > > > > and > > > > > > Aug 10 14:00:30 fedora29 kernel: ---[ end trace b062b12a095f4dd5 ]--- > > > Aug 10 14:00:30 fedora29 kernel: RIP: > > > 0010:cifs_close_deferred_file+0x77/0x140 [cifs] > > > Aug 10 14:00:30 fedora29 kernel: Code: 85 de 00 00 00 4c 89 ef e8 06 > > > 96 2e e8 48 8b 1c 24 4c 39 e3 74 55 49 bd 00 01 00 00 00 00 ad de 48 > > > bd 22 01 00 00 00 00 ad de <48> 8b 7b 10 31 d2 be 01 00 00 00 e8 a9 c5 > > > fe ff 48 89 df e8 61 46 > > > Aug 10 14:00:30 fedora29 kernel: RSP: 0018:ffffb12180f0fc80 EFLAGS: 00010283 > > > Aug 10 14:00:30 fedora29 kernel: RAX: ffff88dc5b2ca201 RBX: > > > dead000000000100 RCX: 0000000000002367 > > > Aug 10 14:00:30 fedora29 kernel: RDX: 0000000000002366 RSI: > > > 67100f614dfd246a RDI: 0000000000032080 > > > Aug 10 14:00:30 fedora29 kernel: RBP: dead000000000122 R08: > > > 0000000000000001 R09: 0000000000000001 > > > Aug 10 14:00:30 fedora29 kernel: R10: ffffb12180f0fba0 R11: > > > 0000000000000001 R12: ffffb12180f0fc80 > > > > > > On Tue, Aug 10, 2021 at 12:29 PM Rohith Surabattula > > > wrote: > > > > > > > > Hi All, > > > > > > > > Have updated the patch to fix the memleak. > > > > > > > > Regards, > > > > Rohith > > > > > > > > On Tue, Aug 10, 2021 at 5:18 PM Shyam Prasad N wrote: > > > > > > > > > > Already gave my review comments along with this one to the other email. > > > > > > > > > > Reviewed-by: Shyam Prasad N > > > > > > > > > > On Mon, Aug 9, 2021 at 3:28 PM Rohith Surabattula > > > > > wrote: > > > > > > > > > > > > Hi Steve/All, > > > > > > > > > > > > During unlink/rename/lease break, deferred work for close is > > > > > > scheduled immediately but in asynchronous manner which might > > > > > > lead to race with actual(unlink/rename) commands. > > > > > > > > > > > > This change will schedule close synchronously which will avoid > > > > > > the race conditions with other commands. > > > > > > > > > > > > Regards, > > > > > > Rohith > > > > > > > > > > > > > > > > > > > > -- > > > > > Regards, > > > > > Shyam > > > > > > > > > > > > -- > > > Thanks, > > > > > > Steve > > > > > > > > -- > > Thanks, > > > > Steve -- Thanks, Steve