From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f181.google.com (mail-pf1-f181.google.com [209.85.210.181]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2366C806 for ; Mon, 27 Nov 2023 06:04:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="BPWSCJl4" Received: by mail-pf1-f181.google.com with SMTP id d2e1a72fcca58-6cbda959062so3206585b3a.1 for ; Sun, 26 Nov 2023 22:04:44 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1701065084; x=1701669884; darn=lists.linux.dev; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date:from:to :cc:subject:date:message-id:reply-to; bh=+OTyuBSj00HpTGxowPCMZHFCFh0gtzlFv4DUNhcakBU=; b=BPWSCJl4ohP81SSh0FdhkIp6EWWbCNROX4Rwpwn3pPvXgC157paU3xw6pdjf5GlTln YvoM8MPara8cCqzVodvadX606O85UeuVRTIlvQsH2lmrY/Y1uT+vZgiO22xBflu6U+wi n7W8gAcW+LvaJNB2qtAe38hKMJC3nQZ+9IG4EWn7/CGYTNYrVLCeu1D4IqUstfvickm4 +Pv3KfE9gPTVWQVeBma9ubOGTp981HIFrAhWiT1Sty25D4nnxcL++YsOmUnrU5EKISDx jGWH4Y5FbUNN/mNv1U9ZWXuKpjrN8ITudMKJlQXFL2/E+L+bjIGVrsPoHyQybXqhlNrr 7cRQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1701065084; x=1701669884; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=+OTyuBSj00HpTGxowPCMZHFCFh0gtzlFv4DUNhcakBU=; b=bGPUo2yJtqvRRz/f/amiQbR3IQrMxUmx6ELa96gbpQc8Utz1bDPufeyfY7rLbi/i85 FFs9ShxPdlcegik8oGEjEpZ88OX5+2pHmGMoAIcrLeByMNimTzgbs5MUvZZefoR2cFuW YRwwA4WL+IeIe+dp/GMOEw2Ay/oNYNAo0DbB4TgWCU5I6W9bKUR3FCjLnhWWAkQBX9zY gzxQ+FTLil1pKCfgEe6lUc/MHyFNPis4Wp61bmkDYZMutfqrPj2HBJmnXqHAdcUg4QE1 +Un/UVHBO/TcFlEWkQJX4bChj5j07FoxLWjQRB6t7+89EdkbrFIEREe0UE55lzDWOjDc zocQ== X-Gm-Message-State: AOJu0Yx/SBX/OyXPmy/Cm3tYyZXIwU+l4Xw3CdW7S6QtXc91LEAqwkc4 aHHWp2Td6fHBH52GmN9iDzOJvFuD0GwuavT6Wg== X-Google-Smtp-Source: AGHT+IG/i+PnOVkzEH5tF9cNXhmaOkhMuwMasrzxUa0Df7G1z+bvbR/u7fIVg0ojsgJH6yM2ZEehSA== X-Received: by 2002:aa7:88ce:0:b0:6cb:6ab1:564c with SMTP id k14-20020aa788ce000000b006cb6ab1564cmr10324719pff.10.1701065084286; Sun, 26 Nov 2023 22:04:44 -0800 (PST) Received: from thinkpad ([103.28.246.157]) by smtp.gmail.com with ESMTPSA id c16-20020aa78c10000000b006905f6bfc37sm6460850pfd.31.2023.11.26.22.04.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 26 Nov 2023 22:04:43 -0800 (PST) Date: Mon, 27 Nov 2023 11:34:39 +0530 From: Manivannan Sadhasivam To: Jakub Kicinski Cc: davem@davemloft.net, edumazet@google.com, pabeni@redhat.com, mhi@lists.linux.dev, linux-arm-msm@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, loic.poulain@linaro.org Subject: Re: [PATCH v2 0/2] Add MHI Endpoint network driver Message-ID: <20231127060439.GA2505@thinkpad> References: <20230607152427.108607-1-manivannan.sadhasivam@linaro.org> <20230607094922.43106896@kernel.org> <20230607171153.GA109456@thinkpad> <20230607104350.03a51711@kernel.org> <20230608123720.GC5672@thinkpad> <20231117070602.GA10361@thinkpad> <20231117162638.7cdb3e7d@kernel.org> Precedence: bulk X-Mailing-List: mhi@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20231117162638.7cdb3e7d@kernel.org> On Fri, Nov 17, 2023 at 04:26:38PM -0800, Jakub Kicinski wrote: > On Fri, 17 Nov 2023 12:36:02 +0530 Manivannan Sadhasivam wrote: > > Sorry to revive this old thread, this discussion seems to have fell through the > > cracks... > > It did not fall thru the cracks, you got a nack. Here it is in a more > official form: > > Nacked-by: Jakub Kicinski > > Please make sure you keep this tag and CC me if you ever post any form > of these patches again. Thanks for the NACK. Could you please respond to my reply justifying this driver (the part you just snipped)? I'm posting it below: > As I explained above, other interfaces also expose this kind of functionality > between host and the device. One of the credible usecase with this driver is > sharing the network connectivity available in either host or the device with the > other end. > > To make it clear, we have 2 kind of channels exposed by MHI for networking. > > 1. IP_SW0 > 2. IP_HW0 > > IP_SW0 is useful in scenarios I explained above and IP_HW0 is purely used to > provide data connectivity to the host machines with the help of modem IP in the > device. And the host side stack is already well supported in mainline. With the > proposed driver, Linux can run on the device itself and it will give Qcom a > chance to get rid of their proprietary firmware used on the PCIe endpoint > devices like modems, etc... I think you made up your mind that this driver is exposing the network interface to the firmware on the device. I ought to clearify that the device running this driver doesn't necessarily be a modem but a PCIe endpoint instance that uses the netdev exposed by this driver to share data connectivity with another device. This concept is not new and being supported by other protocols such as Virtio etc... - Mani -- மணிவண்ணன் சதாசிவம்