Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: tJLXIR6sVvJ5TSbINON6vcoHPEaSNuCm2wbmG+CCeD474ISR4BvJc9R2KLcn3sDI9/DXxJbwEI
 //jQSzmJmCDw==
IronPort-SDR: iOk6x44I2R/IVHg4tM8RX2trIK1ZUNOMMwudDTHe7d4698zCtrGS7+WSe5ckriz8+Cme2knqd2
 DVNm2ytiwTiw==
Message-ID: <bcf8bad08a5f586093a151126aba2127eee44c02.camel@linux.intel.com>
Subject: Re: [PATCH 03/22] keembay-ipc: Add Keem Bay IPC module
From:   Daniele Alessandrelli <daniele.alessandrelli@linux.intel.com>
To:     Greg KH <gregkh@linuxfoundation.org>, mgross@linux.intel.com
Cc:     markgross@kernel.org, arnd@arndb.de, bp@suse.de,
        damien.lemoal@wdc.com, dragan.cvetic@xilinx.com, corbet@lwn.net,
        leonard.crestez@nxp.com, palmerdabbelt@google.com,
        paul.walmsley@sifive.com, peng.fan@nxp.com, robh+dt@kernel.org,
        shawnguo@kernel.org, linux-kernel@vger.kernel.org
Date:   Tue, 08 Dec 2020 18:59:09 +0000
In-Reply-To: <X8cyA1qKCwrayEOp@kroah.com>
References: <20201201223511.65542-1-mgross@linux.intel.com>
         <20201201223511.65542-4-mgross@linux.intel.com>
         <X8cyA1qKCwrayEOp@kroah.com>
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.36.5 (3.36.5-1.fc32) 
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Greg,

Thanks for your feedback.

On Wed, 2020-12-02 at 07:19 +0100, Greg KH wrote:
> On Tue, Dec 01, 2020 at 02:34:52PM -0800, mgross@linux.intel.com wrote:
> > From: Daniele Alessandrelli <daniele.alessandrelli@intel.com>
> > 
> > On the Intel Movidius SoC code named Keem Bay, communication between the
> > Computing Sub-System (CSS), i.e., the CPU, and the Multimedia Sub-System
> > (MSS), i.e., the VPU is enabled by the Keem Bay Inter-Processor
> > Communication (IPC) mechanism.
> > 
> > Add the driver for using Keem Bay IPC from within the Linux Kernel.
> > 
> > Keem Bay IPC uses the following terminology:
> > 
> > - Node:    A processing entity that can use the IPC to communicate;
> > 	   currently, we just have two nodes, CPU (CSS) and VPU (MSS).
> > 
> > - Link:    Two nodes that can communicate over IPC form an IPC link
> > 	   (currently, we just have one link, the one between the CPU
> > 	   and VPU).
> > 
> > - Channel: An IPC link can provide multiple IPC channels. IPC channels
> > 	   allow communication multiplexing, i.e., the same IPC link can
> > 	   be used by different applications for different
> > 	   communications. Each channel is identified by a channel ID,
> > 	   which must be unique within a single IPC link. Channels are
> > 	   divided in two categories, High-Speed (HS) channels and
> > 	   General-Purpose (GP) channels. HS channels have higher
> > 	   priority over GP channels.
> > 
> > Keem Bay IPC mechanism is based on shared memory and hardware FIFOs.
> > Both the CPU and the VPU have their own hardware FIFO. When the CPU
> > wants to send an IPC message to the VPU, it writes to the VPU FIFO (MSS
> > FIFO); similarly, when MSS wants to send an IPC message to the CPU, it
> > writes to the CPU FIFO (CSS FIFO).
> > 
> > A FIFO entry is simply a pointer to an IPC buffer (aka IPC header)
> > stored in a portion of memory shared between the CPU and the VPU.
> > Specifically, the FIFO entry contains the (VPU) physical address of the
> > IPC buffer being transferred.
> > 
> > In turn, the IPC buffer contains the (VPU) physical address of the
> > payload (which must be located in shared memory too) as well as other
> > information (payload size, IPC channel ID, etc.).
> > 
> > Each IPC node instantiates a pool of IPC buffers from its own IPC buffer
> > memory region. When instantiated, IPC buffers are marked as free. When
> > the node needs to send an IPC message, it gets the first free buffer it
> > finds (from its own pool), marks it as allocated (used), and puts its
> > physical address into the IPC FIFO of the destination node. The
> > destination node (which is notified by an interrupt when there are
> > entries pending in its FIFO) extract the physical address from the FIFO
> > and process the IPC buffer, marking it as free once done (so that the
> > sender can reuse the buffer).
> 
> Any reason you can't use the dmabuf interface for these memory buffers
> you are creating and having to manage "by hand"?  I thought that was
> what the kernel was wanting to unify on such that individual
> drivers/subsystems didn't have to do this on their own.

My understanding is that the dmabuf interface is used to share DMA
buffers across different drivers, while these buffers are used only
internally to the IPC driver (and exchanged only with the VPU
firmware). They basically are packet headers that are sent to the VPU.

So, unless I'm missing something, dmabuf is probably not a good fit
here.

However, it could make sense to use dmabuf to share buffers across the
drivers in the xLink stack. We'll look into that.