lkml.org 
[lkml]   [2004]   [Aug]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRe: [PATCH 2.6] ibmveth bug fixes 2/4
From
Date
On Tue, 2004-08-10 at 11:06, Santiago Leon wrote:
> This patch fixes a race condition that would panic the kernel when
> replenishing a buffer pool. Please apply.

How about something like this that doesn't add more magic numbers?

I'm not so sure about the type, though. Is that (u16) cast OK?

-- Dave
--- veth/drivers/net/ibmveth.h.orig 2004-08-10 11:34:05.000000000 -0700
+++ veth/drivers/net/ibmveth.h 2004-08-10 11:35:01.000000000 -0700
@@ -77,6 +77,8 @@
#define IbmVethPool1DftCnt 256
#define IbmVethPool2DftCnt 256

+#define IBM_VETH_INVALID_MAP ((u16)0xffff)
+
struct ibmveth_buff_pool {
u32 size;
u32 index;
--- veth/drivers/net/ibmveth.c.orig 2004-08-10 11:35:17.000000000 -0700
+++ veth/drivers/net/ibmveth.c 2004-08-10 11:37:22.000000000 -0700
@@ -213,11 +213,12 @@ static void ibmveth_replenish_buffer_poo
free_index = pool->consumer_index++ % pool->size;
index = pool->free_map[free_index];

- ibmveth_assert(index != 0xffff);
+ ibmveth_assert(index != IBM_VETH_INVALID_MAP);
ibmveth_assert(pool->skbuff[index] == NULL);

dma_addr = vio_map_single(adapter->vdev, skb->data, pool->buff_size, DMA_FROM_DEVICE);

+ pool->free_map[free_index] = IBM_VETH_INVALID_MAP;
pool->dma_addr[index] = dma_addr;
pool->skbuff[index] = skb;

@@ -232,6 +233,7 @@ static void ibmveth_replenish_buffer_poo
lpar_rc = h_add_logical_lan_buffer(adapter->vdev->unit_address, desc.desc);

if(lpar_rc != H_Success) {
+ pool->free_map[free_index] = IBM_VETH_INVALID_MAP;
pool->skbuff[index] = NULL;
pool->consumer_index--;
vio_unmap_single(adapter->vdev, pool->dma_addr[index], pool->buff_size, DMA_FROM_DEVICE);
@@ -239,7 +241,6 @@ static void ibmveth_replenish_buffer_poo
adapter->replenish_add_buff_failure++;
break;
} else {
- pool->free_map[free_index] = 0xffff;
buffers_added++;
adapter->replenish_add_buff_success++;
}
\
 
 \ /
  Last update: 2005-03-22 14:05    [W:0.027 / U:0.056 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site