RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Parav Pandit
; net...@vger.kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > On Sat, 24 Aug 2019 03:56:08 + > Parav Pandit wrote: > > > > -Original Message- > > > From: Alex Williamson > > > Sent: Saturday, August 24, 2019

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Alex Williamson
; ; Kirti Wankhede ; Cornelia > > Huck ; k...@vger.kernel.org; linux- > > ker...@vger.kernel.org; cjia ; net...@vger.kernel.org > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > On Fri, 23 Aug 2019 18:00:30 + > > Parav Pandit wrote: > > &g

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Parav Pandit
; > Cornelia Huck ; k...@vger.kernel.org; linux- > ker...@vger.kernel.org; cjia ; net...@vger.kernel.org > Subject: RE: [PATCH v2 0/2] Simplify mtty driver and mdev core > > I don't understand this logic. I'm simply asking that we have a way > > to test the collision behavior withou

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Parav Pandit
ia ; net...@vger.kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > On Fri, 23 Aug 2019 18:00:30 + > Parav Pandit wrote: > > > > -Original Message- > > > From: Alex Williamson > > > Sent: Friday, August 23, 20

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Alex Williamson
; ; Kirti Wankhede ; Cornelia > > Huck ; k...@vger.kernel.org; linux- > > ker...@vger.kernel.org; cjia ; net...@vger.kernel.org > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > On Fri, 23 Aug 2019 16:14:04 + > > Parav Pandit wrote: > > >

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Parav Pandit
ia ; net...@vger.kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > On Fri, 23 Aug 2019 16:14:04 + > Parav Pandit wrote: > > > > > Idea is to have mdev alias as optional. > > > > Each mdev_parent says whether it wants mdev_c

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Alex Williamson
; Cornelia > > Huck ; k...@vger.kernel.org; linux- > > ker...@vger.kernel.org; cjia ; net...@vger.kernel.org > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > On Fri, 23 Aug 2019 14:53:06 + > > Parav Pandit wrote: > > &g

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Parav Pandit
ia ; net...@vger.kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > On Fri, 23 Aug 2019 14:53:06 + > Parav Pandit wrote: > > > > -Original Message- > > > From: Alex Williamson > > > Sent: Friday, August 23, 2

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Alex Williamson
; Cornelia > > Huck ; k...@vger.kernel.org; linux- > > ker...@vger.kernel.org; cjia ; net...@vger.kernel.org > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > On Fri, 23 Aug 2019 08:14:39 + > > Parav Pandit wrote: > > >

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Jiri Pirko
>> ; Kirti Wankhede ; Cornelia >> Huck ; k...@vger.kernel.org; linux- >> ker...@vger.kernel.org; cjia ; net...@vger.kernel.org >> Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core >> >> On Fri, 23 Aug 2019 08:14:39 + >> Parav Pandit wrote: >> >

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Parav Pandit
ia ; net...@vger.kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > On Fri, 23 Aug 2019 08:14:39 + > Parav Pandit wrote: > > > Hi Alex, > > > > > > > -Original Message- > > > From: Jiri Pirko > > >

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Alex Williamson
; Kirti > > Wankhede ; Cornelia Huck ; > > k...@vger.kernel.org; linux-kernel@vger.kernel.org; cjia ; > > net...@vger.kernel.org > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > Thu, Aug 22, 2019 at 03:33:30PM CEST, pa...@mellanox.com

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Parav Pandit
> net...@vger.kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > Thu, Aug 22, 2019 at 03:33:30PM CEST, pa...@mellanox.com wrote: > > > > > >> -Original Message- > >> From: Jiri Pirko > >> Sent: Thursday, Aug

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-23 Thread Jiri Pirko
; Kirti >> Wankhede ; Cornelia Huck ; >> k...@vger.kernel.org; linux-kernel@vger.kernel.org; cjia ; >> net...@vger.kernel.org >> Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core >> >> Thu, Aug 22, 2019 at 12:04:02PM CEST, pa...@mellanox.com wrote: >>

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-22 Thread Parav Pandit
> net...@vger.kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > Thu, Aug 22, 2019 at 12:04:02PM CEST, pa...@mellanox.com wrote: > > > > > >> -Original Message- > >> From: Jiri Pirko > >> Sent: Thursday, August 22

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-22 Thread Jiri Pirko
; Kirti >> Wankhede ; Cornelia Huck ; >> k...@vger.kernel.org; linux-kernel@vger.kernel.org; cjia ; >> net...@vger.kernel.org >> Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core >> >> Thu, Aug 22, 2019 at 11:42:13AM CEST, pa...@mellanox.com wrote: >>

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-22 Thread Parav Pandit
> net...@vger.kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > Thu, Aug 22, 2019 at 11:42:13AM CEST, pa...@mellanox.com wrote: > > > > > >> -Original Message- > >> From: Jiri Pirko > >> Sent: Thursday, August 22

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-22 Thread Jiri Pirko
; Kirti >> Wankhede ; Cornelia Huck ; >> k...@vger.kernel.org; linux-kernel@vger.kernel.org; cjia ; >> net...@vger.kernel.org >> Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core >> >> Wed, Aug 21, 2019 at 08:23:17AM CEST, pa...@mellanox.com wrote: >> > &

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-22 Thread Parav Pandit
> net...@vger.kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > Wed, Aug 21, 2019 at 08:23:17AM CEST, pa...@mellanox.com wrote: > > > > > >> -Original Message- > >> From: Alex Williamson > >> Sent: Wednesday, Au

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-22 Thread Jiri Pirko
elia Huck >> ; k...@vger.kernel.org; linux-kernel@vger.kernel.org; >> cjia ; net...@vger.kernel.org >> Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core >> >> > > > > Just an example of the alias, not proposing how it's set. In >>

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-21 Thread Parav Pandit
kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > > > Just an example of the alias, not proposing how it's set. In > > > > > fact, proposing that the user does not set it, mdev-core > > > > > provides one >

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Alex Williamson
> ; k...@vger.kernel.org; linux-kernel@vger.kernel.org; > > cjia ; net...@vger.kernel.org > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > On Wed, 21 Aug 2019 04:40:15 + > > Parav Pandit wrote: > > > > > > --

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Parav Pandit
gt; To: Parav Pandit > > > Cc: Jiri Pirko ; David S . Miller > > > ; Kirti Wankhede ; > > > Cornelia Huck ; k...@vger.kernel.org; > > > linux-kernel@vger.kernel.org; cjia ; > > > net...@vger.kernel.org > > > Subject: Re: [PATCH v2 0/2] Simplify

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Alex Williamson
> ; k...@vger.kernel.org; linux-kernel@vger.kernel.org; > > cjia ; net...@vger.kernel.org > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > On Wed, 21 Aug 2019 03:42:25 + > > Parav Pandit wrote: > > > > > > -O

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Parav Pandit
gt; To: Parav Pandit > > > Cc: Jiri Pirko ; David S . Miller > > > ; Kirti Wankhede ; > > > Cornelia Huck ; k...@vger.kernel.org; > > > linux-kernel@vger.kernel.org; cjia ; > > > net...@vger.kernel.org > > > Subject: Re: [PATCH v2 0/2]

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Alex Williamson
> ; k...@vger.kernel.org; linux-kernel@vger.kernel.org; > > cjia ; net...@vger.kernel.org > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > On Tue, 20 Aug 2019 08:58:02 + > > Parav Pandit wrote: > > > > > + Dave. >

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Parav Pandit
kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > On Tue, 20 Aug 2019 11:19:04 -0600 > Alex Williamson wrote: > > > What about an alias based on the uuid? For example, we use 160-bit > > sha1s daily with git (uuids are only 128-bit), but we gene

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Parav Pandit
kernel.org > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > On Tue, 20 Aug 2019 08:58:02 + > Parav Pandit wrote: > > > + Dave. > > > > Hi Jiri, Dave, Alex, Kirti, Cornelia, > > > > Please provide your feedback on it, how shall we

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Parav Pandit
> -Original Message- > From: Cornelia Huck > Sent: Tuesday, August 20, 2019 10:01 PM > > > > Option-1: mdev index > > > > Introduce an optional mdev index/handle as u32 during mdev create > time. > > > > User passes mdev index/handle as input. > > > > > > > > phys_port_name=mIndex=m%u

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Cornelia Huck
On Tue, 20 Aug 2019 11:19:04 -0600 Alex Williamson wrote: > What about an alias based on the uuid? For example, we use 160-bit > sha1s daily with git (uuids are only 128-bit), but we generally don't > reference git commits with the full 20 character string. Generally 12 > characters is

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Alex Williamson
On Tue, 20 Aug 2019 08:58:02 + Parav Pandit wrote: > + Dave. > > Hi Jiri, Dave, Alex, Kirti, Cornelia, > > Please provide your feedback on it, how shall we proceed? > > Short summary of requirements. > For a given mdev (mediated device [1]), there is one representor > netdevice and

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Cornelia Huck
On Tue, 20 Aug 2019 11:25:05 + Parav Pandit wrote: > > -Original Message- > > From: Christophe de Dinechin > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > > > Parav Pandit writes: > > > > > + Dav

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Parav Pandit
> -Original Message- > From: Christophe de Dinechin > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > Parav Pandit writes: > > > + Dave. > > > > Hi Jiri, Dave, Alex, Kirti, Cornelia, > > > > Please provide your

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Christophe de Dinechin
Parav Pandit writes: > + Dave. > > Hi Jiri, Dave, Alex, Kirti, Cornelia, > > Please provide your feedback on it, how shall we proceed? > > Hence, I would like to discuss below options. > > Option-1: mdev index > Introduce an optional mdev index/handle as u32 during mdev create time. > User

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-20 Thread Parav Pandit
> ker...@vger.kernel.org; c...@nvidia.com; Jiri Pirko ; > net...@vger.kernel.org > Subject: RE: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > > -Original Message- > > From: Alex Williamson > > Sent: Wednesday, August 14, 2019 8:28 PM >

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-14 Thread Parav Pandit
> To: Parav Pandit > > > Cc: Alex Williamson ; Kirti Wankhede > > > ; k...@vger.kernel.org; linux- > > > ker...@vger.kernel.org; c...@nvidia.com; Jiri Pirko > > > ; net...@vger.kernel.org > > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-14 Thread Alex Williamson
linux- > > ker...@vger.kernel.org; c...@nvidia.com; Jiri Pirko ; > > net...@vger.kernel.org > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > On Wed, 14 Aug 2019 12:27:01 + > > Parav Pandit wrote: > > > > > + Jiri,

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-14 Thread Parav Pandit
nelia, > > > > > -Original Message- > > > From: Cornelia Huck > > > Sent: Wednesday, August 14, 2019 1:32 PM > > > To: Parav Pandit > > > Cc: Alex Williamson ; Kirti Wankhede > > > ; k...@vger.kernel.org; linux- > > > ker..

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-14 Thread Cornelia Huck
ent: Wednesday, August 14, 2019 1:32 PM > > To: Parav Pandit > > Cc: Alex Williamson ; Kirti Wankhede > > ; k...@vger.kernel.org; linux- > > ker...@vger.kernel.org; c...@nvidia.com > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > On Wed, 14

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-14 Thread Parav Pandit
k...@vger.kernel.org; linux- > ker...@vger.kernel.org; c...@nvidia.com > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > On Wed, 14 Aug 2019 05:54:36 + > Parav Pandit wrote: > > > > > I get that part. I prefer to remove the UUID itself from th

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-14 Thread Cornelia Huck
On Wed, 14 Aug 2019 05:54:36 + Parav Pandit wrote: > > > I get that part. I prefer to remove the UUID itself from the structure > > > and therefore removing this API makes lot more sense? > > > > Mdev and support tools around mdev are based on UUIDs because it's defined > > in the

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-13 Thread Parav Pandit
> -Original Message- > From: Alex Williamson > Sent: Tuesday, August 13, 2019 10:42 PM > To: Parav Pandit > Cc: Kirti Wankhede ; k...@vger.kernel.org; linux- > ker...@vger.kernel.org; coh...@redhat.com; c...@nvidia.com > Subject: Re: [PATCH v2 0/2] Simplify mtty

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-13 Thread Parav Pandit
hat.com; c...@nvidia.com > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > On Tue, Aug 13, 2019 at 09:37:21AM -0700, Christoph Hellwig wrote: > > On Tue, Aug 13, 2019 at 02:40:02PM +, Parav Pandit wrote: > > > We need to ask Greg or Linus on the kernel

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-13 Thread Greg Kroah-Hartman
On Tue, Aug 13, 2019 at 09:37:21AM -0700, Christoph Hellwig wrote: > On Tue, Aug 13, 2019 at 02:40:02PM +, Parav Pandit wrote: > > We need to ask Greg or Linus on the kernel policy on whether an API should > > exist without in-kernel driver. I "love" it when people try to ask a question of

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-13 Thread Alex Williamson
; coh...@redhat.com; c...@nvidia.com > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > On Tue, 13 Aug 2019 14:40:02 + > > Parav Pandit wrote: > > > > > > -Original Message- > > > > From: Kirti Wankhede

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-13 Thread Christoph Hellwig
On Tue, Aug 13, 2019 at 02:40:02PM +, Parav Pandit wrote: > We need to ask Greg or Linus on the kernel policy on whether an API should > exist without in-kernel driver. > We don't add such API in netdev, rdma and possibly other subsystem. > Where can we find this mdev driver in-tree? The

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-13 Thread Cornelia Huck
On Tue, 13 Aug 2019 16:28:53 + Parav Pandit wrote: > In bigger objective, I wanted to discuss post this cleanup patch, is to > expand mdev to have more user friendly device names. Uh, what is unfriendly about uuids? > > Before we reach there, I should include a patch that eliminates

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-13 Thread Parav Pandit
> -Original Message- > From: Alex Williamson > Sent: Tuesday, August 13, 2019 8:23 PM > To: Parav Pandit > Cc: Kirti Wankhede ; k...@vger.kernel.org; linux- > ker...@vger.kernel.org; coh...@redhat.com; c...@nvidia.com > Subject: Re: [PATCH v2 0/2] Simplify mtty

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-13 Thread Alex Williamson
h...@redhat.com; > > c...@nvidia.com > > Subject: Re: [PATCH v2 0/2] Simplify mtty driver and mdev core > > > > > > > > On 8/9/2019 4:32 AM, Alex Williamson wrote: > > > On Thu, 8 Aug 2019 09:12:53 -0500 > > > Parav Pandit wrote: >

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-13 Thread Parav Pandit
Hi Alex, > -Original Message- > From: Alex Williamson > Sent: Friday, August 9, 2019 4:33 AM > To: Parav Pandit > Cc: k...@vger.kernel.org; kwankh...@nvidia.com; linux- > ker...@vger.kernel.org; coh...@redhat.com; c...@nvidia.com > Subject: Re: [PATCH v2 0/2]

RE: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-13 Thread Parav Pandit
> -Original Message- > From: Kirti Wankhede > Sent: Monday, August 12, 2019 5:06 PM > To: Alex Williamson ; Parav Pandit > > Cc: k...@vger.kernel.org; linux-kernel@vger.kernel.org; coh...@redhat.com; > c...@nvidia.com > Subject: Re: [PATCH v2 0/2] Simplify m

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-12 Thread Kirti Wankhede
On 8/9/2019 4:32 AM, Alex Williamson wrote: > On Thu, 8 Aug 2019 09:12:53 -0500 > Parav Pandit wrote: > >> Currently mtty sample driver uses mdev state and UUID in convoluated way to >> generate an interrupt. >> It uses several translations from mdev_state to mdev_device to mdev uuid. >>

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-09 Thread Cornelia Huck
On Thu, 8 Aug 2019 17:02:47 -0600 Alex Williamson wrote: > On Thu, 8 Aug 2019 09:12:53 -0500 > Parav Pandit wrote: > > > Currently mtty sample driver uses mdev state and UUID in convoluated way to > > generate an interrupt. > > It uses several translations from mdev_state to mdev_device to

Re: [PATCH v2 0/2] Simplify mtty driver and mdev core

2019-08-08 Thread Alex Williamson
On Thu, 8 Aug 2019 09:12:53 -0500 Parav Pandit wrote: > Currently mtty sample driver uses mdev state and UUID in convoluated way to > generate an interrupt. > It uses several translations from mdev_state to mdev_device to mdev uuid. > After which it does linear search of long uuid comparision