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=-0.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS 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 BBCC7C2D0E5 for ; Mon, 30 Mar 2020 17:08:22 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 9265820780 for ; Mon, 30 Mar 2020 17:08:22 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=blockbridge-com.20150623.gappssmtp.com header.i=@blockbridge-com.20150623.gappssmtp.com header.b="Ad+nBNSp" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730233AbgC3RIW (ORCPT ); Mon, 30 Mar 2020 13:08:22 -0400 Received: from mail-ot1-f65.google.com ([209.85.210.65]:34661 "EHLO mail-ot1-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730228AbgC3RIV (ORCPT ); Mon, 30 Mar 2020 13:08:21 -0400 Received: by mail-ot1-f65.google.com with SMTP id m2so4130413otr.1 for ; Mon, 30 Mar 2020 10:08:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=blockbridge-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=KzOIcpQMGry/S7KX/3i7FgRktSaDHF+E1Qb4eqLRCuI=; b=Ad+nBNSpjurCoWqj//saej4OFOKjZlYLa0jKuodco+te+2JVG/IANMC0ikGvaZGOg9 ehwwvhE5XhgZpWyO0m9adpKNM4PYIQaxkbNUEHZf9Sy3sOzUQ88RnOSIl5hXYIu3symh 2Yzv1I0vyFWGAEfDUG0zNQuf98O4BdjfDYJTUE7jFws0nGCYTn19Is7qyYIL5hrqRWgt rDx0G/o8YABjNrEhlFc6aY35c65jRO4QjO7+cJG871TLgXoTAxgN9C6wlTwv1MsBDXlg 15QEJSRz69OeHuBm6MfshBiN30xhgVlJOeAU0T4Ra+XlpoI1qPuQCkRPGdDJyx4JBLRA whtg== 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=KzOIcpQMGry/S7KX/3i7FgRktSaDHF+E1Qb4eqLRCuI=; b=JIeqH9MYLWJa/uTlWUVIrz6zgykAUPDBU9sO7fpAIo9Pi0PynxKjyErYvmJQJlg8lO 3rJnaCoQ1vLNaJnTiDCCYRJTB2/CJsNOhTHI0LSPV/TLIvJW69kUJIrDjTRqHp6q9LOz R6InTxkJGyG7YMsjLIqTthA6U7JAc6YnGbI1WxuGprtrh6vUCbZEmQ06gDLjQuiu4dQs U119N+Dkmqq7mkcVUqVyw8WVViIrXumqaraxQNpqNB0yDvvg45LG//USH4UFJaZ4dWHe 5bfjOKz0sDxdA9O1RbakugzFQ+z/8fEd09CSJu/xTgOlj3O30bT52C8gavl5mKaKxeh/ fFnw== X-Gm-Message-State: ANhLgQ3x0EBRIR0Utw2ZHF5BtBNwyOXiOYh/yyjWYY7KQ1zJkFynhul1 k2DwCUVwKBs7Cmavx+/wGZOWU0t1Vv7pIsIN8rFGqQ== X-Google-Smtp-Source: ADFU+vsZduY51eDd7FS9ubS+PNYEHKDob0M//Wghi65nlZ5LqUBQIcY9bjjm+Kk1UD2LVIpEBTaw7W/SLGM8xpAsD6U= X-Received: by 2002:a9d:6ad8:: with SMTP id m24mr126508otq.66.1585588099575; Mon, 30 Mar 2020 10:08:19 -0700 (PDT) MIME-Version: 1.0 References: <20191203005849.GB25002@ming.t460p> <20191203031444.GB6245@ming.t460p> <20191203124558.GA22805@ming.t460p> <20191204010529.GA3910@ming.t460p> <20191204230225.GA26189@ming.t460p> In-Reply-To: From: Stephen Rust Date: Mon, 30 Mar 2020 13:08:37 -0400 Message-ID: Subject: Re: Data corruption in kernel 5.1+ with iSER attached ramdisk To: Sagi Grimberg Cc: Ming Lei , Rob Townley , Christoph Hellwig , Jens Axboe , linux-block@vger.kernel.org, linux-rdma@vger.kernel.org, linux-scsi@vger.kernel.org, martin.petersen@oracle.com, target-devel@vger.kernel.org, Doug Ledford , Jason Gunthorpe , Max Gurtovoy Content-Type: text/plain; charset="UTF-8" Sender: linux-rdma-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-rdma@vger.kernel.org Sagi, > Sorry for the late reply, lost track of this. No problem! > Can you try attached patch and see if it solves your issue? > WARNING: very lightly tested... I have run our tests against this patch and it is working well for our "basic" testing as well. The test case that previously failed, now passes with this patch. So that's encouraging! Thanks for the quick response and quick patch. One question we had is regarding the hard coded header length: What happens if the initiator sends an extended CDB, like a WRITE32? Are there any concerns with an additional header segment (AHS)? Thanks again, Steve