Re: IB/mlx4: Use EAGAIN error code instead or EINVAL in function mlx4_ib_poll_one

[Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1604 bytes --]

On Wed, Jul 27, 2016 at 02:42:39PM +0300, Yuval Shaia wrote:
> On Wed, Jul 27, 2016 at 02:23:29PM +0300, Leon Romanovsky wrote:
> > On Wed, Jul 27, 2016 at 01:26:41PM +0300, Yuval Shaia wrote:
> > > On Wed, Jul 27, 2016 at 11:58:00AM +0300, Leon Romanovsky wrote:
> > > > On Wed, Jul 27, 2016 at 01:24:51AM -0700, Yuval Shaia wrote:
> > > > > Please review the following two patches.
> > > > > First patch makes int-return function to a void one as it anyway returns 0.
> > > > > Second patch replace all EINVAL error codes returned from mlx4_ib_poll_one
> > > > > to EAGAIN.
> > > > 
> > > > Thanks Yuval for taking time to do it.
> > > > After your's IPoIB patches, we prepared patch similar to these and
> > > > planned to send it after merge window will close.
> > > 
> > > Great new!
> > > So, will you also take care of making the function use_tunnel_data void?
> > 
> > No problem, I'll resubmit your use_tunnel_data patch.
> > Thanks
> 
> With the surrounding {} please :)
> I can send v1 if needed.

There is no need, I already added correct version to our verification
system.

Thanks

> 
> > 
> > > 
> > > > 
> > > > > 
> > > > > [PATCH 1/2] IB/mlx4: Make function use_tunnel_data return void
> > > > > [PATCH 2/2] IB/mlx4: Return EAGAIN for any error in mlx4_ib_poll_one
> > > > > --
> > > > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> > > > > the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> > > > > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > > 
> > > 
> 
> 

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]