linux-kernel - RE: [PATCH v3 7/8] staging: fsl-mc: update TODO and README for restool driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <HE1PR04MB16415E8385BB2D516D78A4708DD60@HE1PR04MB1641.eurprd04.prod.outlook.com>
Date:	Tue, 9 Feb 2016 18:29:12 +0000
From:	Stuart Yoder <stuart.yoder@....com>
To:	Lijun Pan <Lijun.Pan@...escale.com>,
	"gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
	"arnd@...db.de" <arnd@...db.de>,
	"devel@...verdev.osuosl.org" <devel@...verdev.osuosl.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
CC:	"stuart.yoder@...escale.com" <stuart.yoder@...escale.com>,
	"itai.katz@...escale.com" <itai.katz@...escale.com>,
	"german.rivera@...escale.com" <german.rivera@...escale.com>,
	"leoli@...escale.com" <leoli@...escale.com>,
	"scottwood@...escale.com" <scottwood@...escale.com>,
	"agraf@...e.de" <agraf@...e.de>,
	"bhamciu1@...escale.com" <bhamciu1@...escale.com>,
	"R89243@...escale.com" <R89243@...escale.com>,
	"bhupesh.sharma@...escale.com" <bhupesh.sharma@...escale.com>,
	"nir.erez@...escale.com" <nir.erez@...escale.com>,
	"richard.schmitt@...escale.com" <richard.schmitt@...escale.com>,
	"dan.carpenter@...cle.com" <dan.carpenter@...cle.com>,
	"Lijun.Pan2000@...il.com" <Lijun.Pan2000@...il.com>
Subject: RE: [PATCH v3 7/8] staging: fsl-mc: update TODO and README for
 restool driver



> -----Original Message-----
> From: Lijun Pan [mailto:Lijun.Pan@...escale.com]
> Sent: Monday, February 08, 2016 5:40 PM
> To: gregkh@...uxfoundation.org; arnd@...db.de; devel@...verdev.osuosl.org; linux-
> kernel@...r.kernel.org
> Cc: stuart.yoder@...escale.com; itai.katz@...escale.com; german.rivera@...escale.com;
> leoli@...escale.com; scottwood@...escale.com; agraf@...e.de; bhamciu1@...escale.com;
> R89243@...escale.com; bhupesh.sharma@...escale.com; nir.erez@...escale.com;
> richard.schmitt@...escale.com; dan.carpenter@...cle.com; Lijun.Pan2000@...il.com; Lijun
> Pan <Lijun.Pan@...escale.com>
> Subject: [PATCH v3 7/8] staging: fsl-mc: update TODO and README for restool driver
> 
> Add more introduction of restool driver and state why
> restool driver is needed in helping moving fsl-mc bus
> out of staging tree.
> 
> Signed-off-by: Lijun Pan <Lijun.Pan@...escale.com>
> ---
>  drivers/staging/fsl-mc/README.txt |   11 ++++++++++-
>  drivers/staging/fsl-mc/TODO       |   18 ++++++++++++++++--
>  2 files changed, 26 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/staging/fsl-mc/README.txt b/drivers/staging/fsl-mc/README.txt
> index 8214102..e9ec507 100644
> --- a/drivers/staging/fsl-mc/README.txt
> +++ b/drivers/staging/fsl-mc/README.txt
> @@ -130,7 +130,16 @@ the objects involved in creating a network interfaces.
>      via a config file passed to the MC when firmware starts
>      it.  There is also a Linux user space tool called "restool"
>      that can be used to create/destroy containers and objects
> -    dynamically.
> +    dynamically. The kernel side restool driver communicates with
> +    user space restool via ioctl. Restool relies on allocator driver
> +    to allocate dpmcp resources, enumerates fsl-mc bus to find root dprc
> +    objects of interest. When the user space restool program sends a request
> +    to restool driver to create a dp* objects in MC firmware, an interrupt
> +    will be triggered by MC firmware and the dprc driver's interrupt handler
> +    shall process the interrupt (synchronizing the objects in MC firmware and
> +    objects in Linux kernel). Though small, restool driver helps verify all
> +    the functionality of fsl-mc bus, dprc driver, allocator driver,
> +    and MC flib API.
> 
>  -DPAA2 Objects for an Ethernet Network Interface
> 
> diff --git a/drivers/staging/fsl-mc/TODO b/drivers/staging/fsl-mc/TODO
> index 5065821..4892eb6 100644
> --- a/drivers/staging/fsl-mc/TODO
> +++ b/drivers/staging/fsl-mc/TODO
> @@ -5,10 +5,24 @@
>    fsl-mc bus out of staging.
> 
>  * Decide if multiple root fsl-mc buses will be supported per Linux instance,
> -  and if so add support for this.
> +  and if so add support for this. No matter fsl-mc bus support multiple root
> +  dprc or not, restool driver is designed to support multiple root if fsl-mc
> +  bus is ready some day later. If there is only one root dprc, restool driver
> +  works fine.
> +
> +* Add at least one driver utilizing fsl-mc bus. Restool driver is a very
> +  small and simple driver, which interacts with fsl-mc bus, dprc driver,
> +  allocator driver. Restool relies on allocator driver to allocate
> +  dpmcp resources, enumerates fsl-mc bus to find root dprc objects of interest.
> +  When the user space restool program sends a request to restool driver to
> +  create a dp* objects in MC firmware, an interrupt will be triggered by
> +  MC firmware and the dprc driver's interrupt handler shall process the
> +  interrupt. Though small, restool driver helps verify all the functionality
> +  of fsl-mc bus, dprc driver, allocator driver, and MC flib API. Restool
> +  driver helps in moving fsl-mc bus out of staging branch.

I don't agree with expanding the TODO list.  The discussion with Alex Graf
and Greg last year left us with the target of getting a object driver into
staging that uses the fsl-mc bus in a normal way, which is what the current
TODO list reflects.  Now that MSI support is in we are closer to being able
to do that.

The restool driver is a special case that I think can wait.

Stuart