2010-02-05 04:59:12

by FUJITA Tomonori

[permalink] [raw]
Subject: [PATCH] mlx4: replace the dma_sync_single_range_for_cpu/device API

There are only two users of the dma_sync_single_range_for_cpu/device
API in mainline (mlx4 and ssb). The
dma_sync_single_range_for_cpu/device API has never been documented and
the dma_sync_single_for_cpu/device API also support a partial sync.

This converts mlx4 to use the dma_sync_single_for_cpu/device API
(preparations for the removal of the dma_sync_single_range_for_cpu/device API).

Signed-off-by: FUJITA Tomonori <[email protected]>
---
drivers/net/mlx4/en_rx.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/net/mlx4/en_rx.c b/drivers/net/mlx4/en_rx.c
index 829b9ec..6439464 100644
--- a/drivers/net/mlx4/en_rx.c
+++ b/drivers/net/mlx4/en_rx.c
@@ -508,11 +508,11 @@ static struct sk_buff *mlx4_en_rx_skb(struct mlx4_en_priv *priv,
/* We are copying all relevant data to the skb - temporarily
* synch buffers for the copy */
dma = be64_to_cpu(rx_desc->data[0].addr);
- dma_sync_single_range_for_cpu(&mdev->pdev->dev, dma, 0,
- length, DMA_FROM_DEVICE);
+ dma_sync_single_for_cpu(&mdev->pdev->dev, dma, length,
+ DMA_FROM_DEVICE);
skb_copy_to_linear_data(skb, va, length);
- dma_sync_single_range_for_device(&mdev->pdev->dev, dma, 0,
- length, DMA_FROM_DEVICE);
+ dma_sync_single_for_device(&mdev->pdev->dev, dma, length,
+ DMA_FROM_DEVICE);
skb->tail += length;
} else {

--
1.5.6.5


2010-02-05 05:09:27

by David Miller

[permalink] [raw]
Subject: Re: [PATCH] mlx4: replace the dma_sync_single_range_for_cpu/device API

From: FUJITA Tomonori <[email protected]>
Date: Fri, 5 Feb 2010 13:57:42 +0900

> There are only two users of the dma_sync_single_range_for_cpu/device
> API in mainline (mlx4 and ssb). The
> dma_sync_single_range_for_cpu/device API has never been documented and
> the dma_sync_single_for_cpu/device API also support a partial sync.
>
> This converts mlx4 to use the dma_sync_single_for_cpu/device API
> (preparations for the removal of the dma_sync_single_range_for_cpu/device API).
>
> Signed-off-by: FUJITA Tomonori <[email protected]>

Acked-by: David S. Miller <[email protected]>

2010-02-11 22:31:28

by Roland Dreier

[permalink] [raw]
Subject: Re: [PATCH] mlx4: replace the dma_sync_single_range_for_cpu/device API

> > There are only two users of the dma_sync_single_range_for_cpu/device
> > API in mainline (mlx4 and ssb). The
> > dma_sync_single_range_for_cpu/device API has never been documented and
> > the dma_sync_single_for_cpu/device API also support a partial sync.
> >
> > This converts mlx4 to use the dma_sync_single_for_cpu/device API
> > (preparations for the removal of the dma_sync_single_range_for_cpu/device API).
> >
> > Signed-off-by: FUJITA Tomonori <[email protected]>
>
> Acked-by: David S. Miller <[email protected]>

Dave, please go ahead and merge this -- this affects the mlx4 ethernet
driver, so it's your area anyway.
--
Roland Dreier <[email protected]>
For corporate legal information go to:
http://www.cisco.com/web/about/doing_business/legal/cri/index.html

2010-02-19 07:45:12

by FUJITA Tomonori

[permalink] [raw]
Subject: Re: [PATCH] mlx4: replace the dma_sync_single_range_for_cpu/device API

On Thu, 11 Feb 2010 14:31:22 -0800
Roland Dreier <[email protected]> wrote:

> > > There are only two users of the dma_sync_single_range_for_cpu/device
> > > API in mainline (mlx4 and ssb). The
> > > dma_sync_single_range_for_cpu/device API has never been documented and
> > > the dma_sync_single_for_cpu/device API also support a partial sync.
> > >
> > > This converts mlx4 to use the dma_sync_single_for_cpu/device API
> > > (preparations for the removal of the dma_sync_single_range_for_cpu/device API).
> > >
> > > Signed-off-by: FUJITA Tomonori <[email protected]>
> >
> > Acked-by: David S. Miller <[email protected]>
>
> Dave, please go ahead and merge this -- this affects the mlx4 ethernet
> driver, so it's your area anyway.

Seems that net-next still doesn't have this. Can you please merge
this?

Thanks,

2010-02-19 20:39:18

by David Miller

[permalink] [raw]
Subject: Re: [PATCH] mlx4: replace the dma_sync_single_range_for_cpu/device API

From: FUJITA Tomonori <[email protected]>
Date: Fri, 19 Feb 2010 16:44:29 +0900

> On Thu, 11 Feb 2010 14:31:22 -0800
> Roland Dreier <[email protected]> wrote:
>
>> > > There are only two users of the dma_sync_single_range_for_cpu/device
>> > > API in mainline (mlx4 and ssb). The
>> > > dma_sync_single_range_for_cpu/device API has never been documented and
>> > > the dma_sync_single_for_cpu/device API also support a partial sync.
>> > >
>> > > This converts mlx4 to use the dma_sync_single_for_cpu/device API
>> > > (preparations for the removal of the dma_sync_single_range_for_cpu/device API).
>> > >
>> > > Signed-off-by: FUJITA Tomonori <[email protected]>
>> >
>> > Acked-by: David S. Miller <[email protected]>
>>
>> Dave, please go ahead and merge this -- this affects the mlx4 ethernet
>> driver, so it's your area anyway.
>
> Seems that net-next still doesn't have this. Can you please merge
> this?

I'll take care of this later today.