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=-2.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_SANE_1 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 CA02AC43331 for ; Mon, 11 Nov 2019 20:37:50 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 9FC1A20659 for ; Mon, 11 Nov 2019 20:37:50 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="menPzIoi" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9FC1A20659 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=lst.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-nvme-bounces+linux-nvme=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=N5w0/WcizVevvTv1tpnmDkLQbB+HvnTVxNhx92wnzWI=; b=menPzIoi9g8Uq8 X7T92qaLZjxxssbSOGX0R1IIMf//rIXP75BrwaKj8TAZZMUz2BXOCL8afcFMhwOD6P1qV6tMP59m4 J26UF0LM8+fujkSD0zq4U5sqtwjbC+vE6cUDuPwFH/04R+2HTzIApBtInmDKza484hGKVlONNnG0t WleV18HGsPIwX806Qsw2jaHncOWoNuOyZlSdih1eLNsuMrZbDjjsOvGZU7XVIXlceY4Sf8VDdUQSf 9ytQMmQGnjr2SOTXujgqN4uDJLd71mGmze1QDJ19djHC3aDhmr/H3La52dkHvYkKbQ2F4quGtecvT +cNBTW8LrZYFcmEqXJow==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1iUGRZ-0004fD-KW; Mon, 11 Nov 2019 20:37:49 +0000 Received: from verein.lst.de ([213.95.11.211]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1iUGRW-0004eP-LE; Mon, 11 Nov 2019 20:37:48 +0000 Received: by verein.lst.de (Postfix, from userid 2407) id 45C7368B05; Mon, 11 Nov 2019 21:37:43 +0100 (CET) Date: Mon, 11 Nov 2019 21:37:43 +0100 From: Christoph Hellwig To: Alan Mikhak Subject: Re: [PATCH RFC] PCI: endpoint: Add NVMe endpoint function driver Message-ID: <20191111203743.GA25876@lst.de> References: <1573493889-22336-1-git-send-email-alan.mikhak@sifive.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1573493889-22336-1-git-send-email-alan.mikhak@sifive.com> User-Agent: Mutt/1.5.17 (2007-11-01) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20191111_123746_841597_D96DF7AF X-CRM114-Status: UNSURE ( 6.24 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-nvme@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: lorenzo.pieralisi@arm.com, linux-kernel@vger.kernel.org, linux-nvme@lists.infradead.org, kishon@ti.com, palmer@dabbelt.com, paul.walmsley@sifive.com, bhelgaas@google.com, linux-riscv@lists.infradead.org, hch@lst.de Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-nvme" Errors-To: linux-nvme-bounces+linux-nvme=archiver.kernel.org@lists.infradead.org On Mon, Nov 11, 2019 at 09:38:09AM -0800, Alan Mikhak wrote: > A design goal is to not modify the Linux NVMe target driver > at all. As I told you before that is not a "goal" but a fundamental mistake and against the design philosophy of all major Linux subsystems. Please fix your series to move all command parsing to the code based on flags for fabrics vs PCIe in the few places where they significantly differ. _______________________________________________ Linux-nvme mailing list Linux-nvme@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-nvme 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=-2.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_SANE_1 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 F364EC43331 for ; Mon, 11 Nov 2019 20:37:53 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id C548820659 for ; Mon, 11 Nov 2019 20:37:53 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="n5t2ngxH" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C548820659 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=lst.de Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-riscv-bounces+infradead-linux-riscv=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=lxhB5Lr8hAsPyjxNt5mVZAH1tHP23BkOg+tAfoUFJv4=; b=n5t2ngxHJJS7Qf u9Np3zoWzB0OJJSL2PjHueyuuSVlLPKm2fhlPc6Ai5w1vboS7af/O6ESQgAKWVhzv+MaRPwutr/4l dzQAVo5LNRektev1KroGlIvDaynlYmp2XCF+N98dV55dc8zSrQCiP2zjGByuVhUtaS7NgvtYmHsiK Uc0UXfBKGWccCPt/reHdYIDvKLF8ChP1OavuLAXOWjpAySK5antVcpF7Cwbss7FYkkFGX3orlPahj rE7zAchitEzYNAhUtBD502mUe+1ZtjJdPqdA2rVeOcb8dFiSmtY+Lc75Q9Hv7vutGZULKsGx6BDzs PpC+MnfUaNR2z7E1qAPQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1iUGRc-0004j1-H4; Mon, 11 Nov 2019 20:37:52 +0000 Received: from verein.lst.de ([213.95.11.211]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1iUGRW-0004eP-LE; Mon, 11 Nov 2019 20:37:48 +0000 Received: by verein.lst.de (Postfix, from userid 2407) id 45C7368B05; Mon, 11 Nov 2019 21:37:43 +0100 (CET) Date: Mon, 11 Nov 2019 21:37:43 +0100 From: Christoph Hellwig To: Alan Mikhak Subject: Re: [PATCH RFC] PCI: endpoint: Add NVMe endpoint function driver Message-ID: <20191111203743.GA25876@lst.de> References: <1573493889-22336-1-git-send-email-alan.mikhak@sifive.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1573493889-22336-1-git-send-email-alan.mikhak@sifive.com> User-Agent: Mutt/1.5.17 (2007-11-01) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20191111_123746_841597_D96DF7AF X-CRM114-Status: UNSURE ( 6.24 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: lorenzo.pieralisi@arm.com, linux-kernel@vger.kernel.org, linux-nvme@lists.infradead.org, kishon@ti.com, palmer@dabbelt.com, paul.walmsley@sifive.com, bhelgaas@google.com, linux-riscv@lists.infradead.org, hch@lst.de Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-riscv" Errors-To: linux-riscv-bounces+infradead-linux-riscv=archiver.kernel.org@lists.infradead.org On Mon, Nov 11, 2019 at 09:38:09AM -0800, Alan Mikhak wrote: > A design goal is to not modify the Linux NVMe target driver > at all. As I told you before that is not a "goal" but a fundamental mistake and against the design philosophy of all major Linux subsystems. Please fix your series to move all command parsing to the code based on flags for fabrics vs PCIe in the few places where they significantly differ. _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv 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=-2.3 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 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 91428C43331 for ; Mon, 11 Nov 2019 20:37:47 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6D1CF20659 for ; Mon, 11 Nov 2019 20:37:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727401AbfKKUhq (ORCPT ); Mon, 11 Nov 2019 15:37:46 -0500 Received: from verein.lst.de ([213.95.11.211]:51892 "EHLO verein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726939AbfKKUhq (ORCPT ); Mon, 11 Nov 2019 15:37:46 -0500 Received: by verein.lst.de (Postfix, from userid 2407) id 45C7368B05; Mon, 11 Nov 2019 21:37:43 +0100 (CET) Date: Mon, 11 Nov 2019 21:37:43 +0100 From: Christoph Hellwig To: Alan Mikhak Cc: linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, linux-nvme@lists.infradead.org, kishon@ti.com, lorenzo.pieralisi@arm.com, bhelgaas@google.com, hch@lst.de, palmer@dabbelt.com, paul.walmsley@sifive.com Subject: Re: [PATCH RFC] PCI: endpoint: Add NVMe endpoint function driver Message-ID: <20191111203743.GA25876@lst.de> References: <1573493889-22336-1-git-send-email-alan.mikhak@sifive.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1573493889-22336-1-git-send-email-alan.mikhak@sifive.com> User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Nov 11, 2019 at 09:38:09AM -0800, Alan Mikhak wrote: > A design goal is to not modify the Linux NVMe target driver > at all. As I told you before that is not a "goal" but a fundamental mistake and against the design philosophy of all major Linux subsystems. Please fix your series to move all command parsing to the code based on flags for fabrics vs PCIe in the few places where they significantly differ.