Skip to content

Commit

Permalink
Staging: hv: Remove typedef NETVSC_PACKET and PNETVSC_PACKET
Browse files Browse the repository at this point in the history
typedef NETVSC_PACKET and PNETVSC_PACKET are removed and their usages
are replace by the use of struct hv_netvsc_packet and
struct hv_netvsc_packet * respectively.

Here is the semantic patch generated to perform this transformation:
(http://coccinelle.lip6.fr/)

//<smpl>
@rm_NETVSC_PACKET@
@@
-typedef struct _NETVSC_PACKET
+struct hv_netvsc_packet
{...}
-NETVSC_PACKET
;

@rm_PNETVSC_PACKET@
@@
-typedef struct _NETVSC_PACKET *PNETVSC_PACKET;
+struct hv_netvsc_packet;

@fixtypedef_NETVSC_PACKET@
typedef NETVSC_PACKET;
@@
-NETVSC_PACKET
+struct hv_netvsc_packet

@fixstruct__NETVSC_PACKET@
@@
struct
-_NETVSC_PACKET
+hv_netvsc_packet

@fixtypedef_PNETVSC_PACKET@
typedef PNETVSC_PACKET;
@@
-PNETVSC_PACKET
+struct hv_netvsc_packet*
//</smpl>

Signed-off-by: Nicolas Palix <npalix@diku.dk>
Cc: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
  • Loading branch information
Nicolas Palix authored and Greg Kroah-Hartman committed Sep 15, 2009
1 parent 775ef25 commit 4193d4f
Show file tree
Hide file tree
Showing 5 changed files with 38 additions and 38 deletions.
32 changes: 16 additions & 16 deletions drivers/staging/hv/NetVsc.c
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ NetVscOnSendCompletion(
static int
NetVscOnSend(
struct hv_device *Device,
NETVSC_PACKET *Packet
struct hv_netvsc_packet *Packet
);

static void
Expand Down Expand Up @@ -239,8 +239,8 @@ NetVscInitialize(

DPRINT_ENTER(NETVSC);

DPRINT_DBG(NETVSC, "sizeof(NETVSC_PACKET)=%zd, sizeof(NVSP_MESSAGE)=%zd, sizeof(VMTRANSFER_PAGE_PACKET_HEADER)=%zd",
sizeof(NETVSC_PACKET), sizeof(NVSP_MESSAGE), sizeof(VMTRANSFER_PAGE_PACKET_HEADER));
DPRINT_DBG(NETVSC, "sizeof(struct hv_netvsc_packet)=%zd, sizeof(NVSP_MESSAGE)=%zd, sizeof(VMTRANSFER_PAGE_PACKET_HEADER)=%zd",
sizeof(struct hv_netvsc_packet), sizeof(NVSP_MESSAGE), sizeof(VMTRANSFER_PAGE_PACKET_HEADER));

/* Make sure we are at least 2 pages since 1 page is used for control */
ASSERT(driver->RingBufferSize >= (PAGE_SIZE << 1));
Expand Down Expand Up @@ -802,7 +802,7 @@ NetVscOnDeviceAdd(
int i;

struct NETVSC_DEVICE *netDevice;
NETVSC_PACKET* packet;
struct hv_netvsc_packet *packet;
LIST_ENTRY *entry;

NETVSC_DRIVER_OBJECT *netDriver = (NETVSC_DRIVER_OBJECT*) Device->Driver;;
Expand All @@ -828,7 +828,7 @@ NetVscOnDeviceAdd(

for (i=0; i < NETVSC_RECEIVE_PACKETLIST_COUNT; i++)
{
packet = kzalloc(sizeof(NETVSC_PACKET) + (NETVSC_RECEIVE_SG_COUNT* sizeof(PAGE_BUFFER)), GFP_KERNEL);
packet = kzalloc(sizeof(struct hv_netvsc_packet) + (NETVSC_RECEIVE_SG_COUNT* sizeof(PAGE_BUFFER)), GFP_KERNEL);
if (!packet)
{
DPRINT_DBG(NETVSC, "unable to allocate netvsc pkts for receive pool (wanted %d got %d)", NETVSC_RECEIVE_PACKETLIST_COUNT, i);
Expand Down Expand Up @@ -885,7 +885,7 @@ NetVscOnDeviceAdd(
while (!IsListEmpty(&netDevice->ReceivePacketList))
{
entry = REMOVE_HEAD_LIST(&netDevice->ReceivePacketList);
packet = CONTAINING_RECORD(entry, NETVSC_PACKET, ListEntry);
packet = CONTAINING_RECORD(entry, struct hv_netvsc_packet, ListEntry);
kfree(packet);
}

Expand Down Expand Up @@ -915,7 +915,7 @@ NetVscOnDeviceRemove(
)
{
struct NETVSC_DEVICE *netDevice;
NETVSC_PACKET *netvscPacket;
struct hv_netvsc_packet *netvscPacket;
int ret=0;
LIST_ENTRY *entry;

Expand Down Expand Up @@ -958,7 +958,7 @@ NetVscOnDeviceRemove(
while (!IsListEmpty(&netDevice->ReceivePacketList))
{
entry = REMOVE_HEAD_LIST(&netDevice->ReceivePacketList);
netvscPacket = CONTAINING_RECORD(entry, NETVSC_PACKET, ListEntry);
netvscPacket = CONTAINING_RECORD(entry, struct hv_netvsc_packet, ListEntry);

kfree(netvscPacket);
}
Expand Down Expand Up @@ -999,7 +999,7 @@ NetVscOnSendCompletion(
{
struct NETVSC_DEVICE *netDevice;
NVSP_MESSAGE *nvspPacket;
NETVSC_PACKET *nvscPacket;
struct hv_netvsc_packet *nvscPacket;

DPRINT_ENTER(NETVSC);

Expand All @@ -1026,7 +1026,7 @@ NetVscOnSendCompletion(
else if (nvspPacket->Header.MessageType == NvspMessage1TypeSendRNDISPacketComplete)
{
/* Get the send context */
nvscPacket = (NETVSC_PACKET *)(unsigned long)Packet->TransactionId;
nvscPacket = (struct hv_netvsc_packet *)(unsigned long)Packet->TransactionId;
ASSERT(nvscPacket);

/* Notify the layer above us */
Expand All @@ -1048,7 +1048,7 @@ NetVscOnSendCompletion(
static int
NetVscOnSend(
struct hv_device *Device,
NETVSC_PACKET *Packet
struct hv_netvsc_packet *Packet
)
{
struct NETVSC_DEVICE *netDevice;
Expand Down Expand Up @@ -1118,7 +1118,7 @@ NetVscOnReceive(
struct NETVSC_DEVICE *netDevice;
VMTRANSFER_PAGE_PACKET_HEADER *vmxferpagePacket;
NVSP_MESSAGE *nvspPacket;
NETVSC_PACKET *netvscPacket=NULL;
struct hv_netvsc_packet *netvscPacket=NULL;
LIST_ENTRY* entry;
unsigned long start;
unsigned long end, endVirtual;
Expand Down Expand Up @@ -1183,7 +1183,7 @@ NetVscOnReceive(
while (!IsListEmpty(&netDevice->ReceivePacketList))
{
entry = REMOVE_HEAD_LIST(&netDevice->ReceivePacketList);
netvscPacket = CONTAINING_RECORD(entry, NETVSC_PACKET, ListEntry);
netvscPacket = CONTAINING_RECORD(entry, struct hv_netvsc_packet, ListEntry);

INSERT_TAIL_LIST(&listHead, &netvscPacket->ListEntry);

Expand All @@ -1206,7 +1206,7 @@ NetVscOnReceive(
for (i=count; i != 0; i--)
{
entry = REMOVE_HEAD_LIST(&listHead);
netvscPacket = CONTAINING_RECORD(entry, NETVSC_PACKET, ListEntry);
netvscPacket = CONTAINING_RECORD(entry, struct hv_netvsc_packet, ListEntry);

INSERT_TAIL_LIST(&netDevice->ReceivePacketList, &netvscPacket->ListEntry);
}
Expand All @@ -1233,7 +1233,7 @@ NetVscOnReceive(
for (i=0; i < (count - 1); i++)
{
entry = REMOVE_HEAD_LIST(&listHead);
netvscPacket = CONTAINING_RECORD(entry, NETVSC_PACKET, ListEntry);
netvscPacket = CONTAINING_RECORD(entry, struct hv_netvsc_packet, ListEntry);

/* Initialize the netvsc packet */
netvscPacket->XferPagePacket = xferpagePacket;
Expand Down Expand Up @@ -1357,7 +1357,7 @@ static void
NetVscOnReceiveCompletion(
void * Context)
{
NETVSC_PACKET *packet = (NETVSC_PACKET*)Context;
struct hv_netvsc_packet *packet = (struct hv_netvsc_packet*)Context;
struct hv_device *device = (struct hv_device*)packet->Device;
struct NETVSC_DEVICE *netDevice;
u64 transactionId=0;
Expand Down
2 changes: 1 addition & 1 deletion drivers/staging/hv/NetVsc.h
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ struct NETVSC_DEVICE {
int RefCount;

int NumOutstandingSends;
/* List of free preallocated NETVSC_PACKET to represent receive packet */
/* List of free preallocated hv_netvsc_packet to represent receive packet */
LIST_ENTRY ReceivePacketList;
spinlock_t receive_packet_list_lock;

Expand Down
16 changes: 8 additions & 8 deletions drivers/staging/hv/RndisFilter.c
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ typedef struct _RNDIS_REQUEST {
RNDIS_MESSAGE ResponseMessage;

/* Simplify allocation by having a netvsc packet inline */
NETVSC_PACKET Packet;
struct hv_netvsc_packet Packet;
PAGE_BUFFER Buffer;
/* FIXME: We assumed a fixed size request here. */
RNDIS_MESSAGE RequestMessage;
Expand Down Expand Up @@ -107,13 +107,13 @@ static void
RndisFilterReceiveData(
RNDIS_DEVICE *Device,
RNDIS_MESSAGE *Message,
NETVSC_PACKET *Packet
struct hv_netvsc_packet *Packet
);

static int
RndisFilterOnReceive(
struct hv_device *Device,
NETVSC_PACKET *Packet
struct hv_netvsc_packet *Packet
);

static int
Expand Down Expand Up @@ -184,7 +184,7 @@ RndisFilterOnClose(
static int
RndisFilterOnSend(
struct hv_device *Device,
NETVSC_PACKET *Packet
struct hv_netvsc_packet *Packet
);

static void
Expand Down Expand Up @@ -345,7 +345,7 @@ RndisFilterSendRequest(
)
{
int ret=0;
NETVSC_PACKET *packet;
struct hv_netvsc_packet *packet;

DPRINT_ENTER(NETVSC);

Expand Down Expand Up @@ -458,7 +458,7 @@ static void
RndisFilterReceiveData(
RNDIS_DEVICE *Device,
RNDIS_MESSAGE *Message,
NETVSC_PACKET *Packet
struct hv_netvsc_packet *Packet
)
{
RNDIS_PACKET *rndisPacket;
Expand Down Expand Up @@ -491,7 +491,7 @@ RndisFilterReceiveData(
static int
RndisFilterOnReceive(
struct hv_device *Device,
NETVSC_PACKET *Packet
struct hv_netvsc_packet *Packet
)
{
struct NETVSC_DEVICE *netDevice = (struct NETVSC_DEVICE*)Device->Extension;
Expand Down Expand Up @@ -1080,7 +1080,7 @@ RndisFilterOnClose(
static int
RndisFilterOnSend(
struct hv_device *Device,
NETVSC_PACKET *Packet
struct hv_netvsc_packet *Packet
)
{
int ret=0;
Expand Down
10 changes: 5 additions & 5 deletions drivers/staging/hv/include/NetVscApi.h
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@

/* Fwd declaration */

typedef struct _NETVSC_PACKET *PNETVSC_PACKET;
struct hv_netvsc_packet;



Expand All @@ -48,10 +48,10 @@ typedef int (*PFN_ON_OPEN)(struct hv_device *Device);
typedef int (*PFN_ON_CLOSE)(struct hv_device *Device);

typedef void (*PFN_QUERY_LINKSTATUS)(struct hv_device *Device);
typedef int (*PFN_ON_SEND)(struct hv_device *dev, PNETVSC_PACKET packet);
typedef int (*PFN_ON_SEND)(struct hv_device *dev, struct hv_netvsc_packet *packet);
typedef void (*PFN_ON_SENDRECVCOMPLETION)(void * Context);

typedef int (*PFN_ON_RECVCALLBACK)(struct hv_device *dev, PNETVSC_PACKET packet);
typedef int (*PFN_ON_RECVCALLBACK)(struct hv_device *dev, struct hv_netvsc_packet *packet);
typedef void (*PFN_ON_LINKSTATUS_CHANGED)(struct hv_device *dev, u32 Status);

/* Represent the xfer page packet which contains 1 or more netvsc packet */
Expand All @@ -70,7 +70,7 @@ typedef struct _XFERPAGE_PACKET {
* Represent netvsc packet which contains 1 RNDIS and 1 ethernet frame
* within the RNDIS
*/
typedef struct _NETVSC_PACKET {
struct hv_netvsc_packet {
/* Bookkeeping stuff */
DLIST_ENTRY ListEntry;

Expand Down Expand Up @@ -104,7 +104,7 @@ typedef struct _NETVSC_PACKET {
u32 PageBufferCount;
PAGE_BUFFER PageBuffers[NETVSC_PACKET_MAXPAGE];

} NETVSC_PACKET;
};


/* Represents the net vsc driver */
Expand Down
16 changes: 8 additions & 8 deletions drivers/staging/hv/netvsc_drv.c
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ static int netvsc_remove(struct device *device);
static int netvsc_open(struct net_device *net);
static void netvsc_xmit_completion(void *context);
static int netvsc_start_xmit (struct sk_buff *skb, struct net_device *net);
static int netvsc_recv_callback(struct hv_device *device_obj, NETVSC_PACKET* Packet);
static int netvsc_recv_callback(struct hv_device *device_obj, struct hv_netvsc_packet *Packet);
static int netvsc_close(struct net_device *net);
static struct net_device_stats *netvsc_get_stats(struct net_device *net);
static void netvsc_linkstatus_callback(struct hv_device *device_obj, unsigned int status);
Expand Down Expand Up @@ -371,7 +371,7 @@ Desc: Send completion processing
--*/
static void netvsc_xmit_completion(void *context)
{
NETVSC_PACKET *packet = (NETVSC_PACKET *)context;
struct hv_netvsc_packet *packet = (struct hv_netvsc_packet *)context;
struct sk_buff *skb = (struct sk_buff *)(unsigned long)packet->Completion.Send.SendCompletionTid;
struct net_device* net;

Expand Down Expand Up @@ -412,7 +412,7 @@ static int netvsc_start_xmit (struct sk_buff *skb, struct net_device *net)
NETVSC_DRIVER_OBJECT *net_drv_obj = &net_drv_ctx->drv_obj;

int i=0;
NETVSC_PACKET* packet;
struct hv_netvsc_packet *packet;
int num_frags;
int retries=0;

Expand All @@ -428,14 +428,14 @@ static int netvsc_start_xmit (struct sk_buff *skb, struct net_device *net)
num_frags = skb_shinfo(skb)->nr_frags + 1 + net_drv_obj->AdditionalRequestPageBufferCount;

/* Allocate a netvsc packet based on # of frags. */
packet = kzalloc(sizeof(NETVSC_PACKET) + (num_frags * sizeof(PAGE_BUFFER)) + net_drv_obj->RequestExtSize, GFP_ATOMIC);
packet = kzalloc(sizeof(struct hv_netvsc_packet) + (num_frags * sizeof(PAGE_BUFFER)) + net_drv_obj->RequestExtSize, GFP_ATOMIC);
if (!packet)
{
DPRINT_ERR(NETVSC_DRV, "unable to allocate NETVSC_PACKET");
DPRINT_ERR(NETVSC_DRV, "unable to allocate hv_netvsc_packet");
return -1;
}

packet->Extension = (void*)(unsigned long)packet + sizeof(NETVSC_PACKET) + (num_frags * sizeof(PAGE_BUFFER)) ;
packet->Extension = (void*)(unsigned long)packet + sizeof(struct hv_netvsc_packet) + (num_frags * sizeof(PAGE_BUFFER)) ;

/* Setup the rndis header */
packet->PageBufferCount = num_frags;
Expand Down Expand Up @@ -549,7 +549,7 @@ Name: netvsc_recv_callback()
Desc: Callback when we receive a packet from the "wire" on the specify device
--*/
static int netvsc_recv_callback(struct hv_device *device_obj, NETVSC_PACKET* packet)
static int netvsc_recv_callback(struct hv_device *device_obj, struct hv_netvsc_packet *packet)
{
int ret=0;
struct device_context *device_ctx = to_device_context(device_obj);
Expand Down Expand Up @@ -581,7 +581,7 @@ static int netvsc_recv_callback(struct hv_device *device_obj, NETVSC_PACKET* pac
/* for kmap_atomic */
local_irq_save(flags);

/* Copy to skb. This copy is needed here since the memory pointed by NETVSC_PACKET */
/* Copy to skb. This copy is needed here since the memory pointed by hv_netvsc_packet */
/* cannot be deallocated */
for (i=0; i<packet->PageBufferCount; i++)
{
Expand Down

0 comments on commit 4193d4f

Please sign in to comment.