annotate rathaxes_samples_e1000_use_the_dma_abstraction_in_socket.patch @ 119:d84bc9a46771

WIP, finally the alloc_skbuff method will be in the Ethernet subsystem since, the socket subsystem cannot depend on Ethernet (that would cause a circular dependency)
author Louis Opter <louis@lse.epita.fr>
date Fri, 26 Jul 2013 17:35:12 -0700
parents ad21d8a182ad
children cbd9b9a49b63
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
111
b072f682823d Start a new series to refactor e1000::set_up_device
Louis Opter <louis@lse.epita.fr>
parents:
diff changeset
1 # HG changeset patch
119
d84bc9a46771 WIP, finally the alloc_skbuff method will be in the Ethernet subsystem since, the socket subsystem cannot depend on Ethernet (that would cause a circular dependency)
Louis Opter <louis@lse.epita.fr>
parents: 118
diff changeset
2 # Parent c096b6162035042fbafb60b040180a6a5beb2a47
d84bc9a46771 WIP, finally the alloc_skbuff method will be in the Ethernet subsystem since, the socket subsystem cannot depend on Ethernet (that would cause a circular dependency)
Louis Opter <louis@lse.epita.fr>
parents: 118
diff changeset
3 rathaxes: remove some Linux DMA specific code in the socket abstraction in e1000
d84bc9a46771 WIP, finally the alloc_skbuff method will be in the Ethernet subsystem since, the socket subsystem cannot depend on Ethernet (that would cause a circular dependency)
Louis Opter <louis@lse.epita.fr>
parents: 118
diff changeset
4
d84bc9a46771 WIP, finally the alloc_skbuff method will be in the Ethernet subsystem since, the socket subsystem cannot depend on Ethernet (that would cause a circular dependency)
Louis Opter <louis@lse.epita.fr>
parents: 118
diff changeset
5 By using the new DMA abstraction.
111
b072f682823d Start a new series to refactor e1000::set_up_device
Louis Opter <louis@lse.epita.fr>
parents:
diff changeset
6
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
7 diff --git a/rathaxes/samples/e1000/socket.blt b/rathaxes/samples/e1000/socket.blt
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
8 --- a/rathaxes/samples/e1000/socket.blt
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
9 +++ b/rathaxes/samples/e1000/socket.blt
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
10 @@ -1,4 +1,4 @@
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
11 -with Socket, LKM, Device, Ethernet
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
12 +with Socket, LKM, Device, DMA, Ethernet
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
13 {
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
14 template type Socket::AbstractSKBuff()
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
15 {
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
16 @@ -14,7 +14,7 @@
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
17
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
18 map
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
19 {
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
20 - k_sk_buff: ((struct sk_buff *)${self});
118
ad21d8a182ad WIP: put up (hopefully the last) some touches on the dma abstraction
Louis Opter <louis@lse.epita.fr>
parents: 117
diff changeset
21 + k_sk_buff: ((struct sk_buff *)(${self}));
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
22 }
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
23 }
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
24
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
25 @@ -22,35 +22,34 @@
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
26 {
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
27 decl data_types()
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
28 {
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
29 - ${Socket::AbstractSKBuff.ref} skbuff;
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
30 - dma_addr_t dma_handle;
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
31 + ${Socket::AbstractSKBuff.ref} skbuff;
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
32 + ${DMA::AbstractDMAHandle.scalar} dma_handle;
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
33 }
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
34
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
35 chunk LKM::prototypes()
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
36 {
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
37 static void rtx_socket_skbuff_dump_infos(${Socket::SKBuff.ref});
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
38 - static int rtx_socket_skbuff_map(${Socket::SKBuff.ref}, struct device *, enum dma_data_direction);
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
39 - static void rtx_socket_skbuff_unmap_and_free(${Socket::SKBuff.ref}, struct device *, enum dma_data_direction);
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
40 + static int rtx_socket_skbuff_map(${Socket::SKBuff.ref}, ${Device::AbstractDevice.ref}, ${DMA::DMADirection.scalar});
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
41 + static void rtx_socket_skbuff_unmap_and_free(${Socket::SKBuff.ref}, ${Device::AbstractDevice.ref}, ${DMA::DMADirection.scalar});
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
42 }
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
43
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
44 chunk LKM::code()
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
45 {
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
46 static void rtx_socket_skbuff_dump_infos(${Socket::SKBuff.ref} self)
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
47 {
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
48 - WARN_ON(!self->skbuff);
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
49 + WARN_ON(!${local.self.sk_buff});
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
50 +
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
51 + ${Ethernet::ProtocolId} ethernet_proto = { .id = be16_to_cpu(${local.self.sk_buff.k_sk_buff}->protocol) };
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
52 + static const char * const ip_summed_values[] = {
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
53 + "none", "unnecessary", "complete", "partial"
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
54 + };
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
55 + struct skb_shared_info *shinfo = skb_shinfo(${local.self.sk_buff.k_sk_buff});
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
56
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
57 /*
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
58 * We should use a Rathaxes log abstraction instead of pr_info here,
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
59 * but Rathaxes doesn't support functions with a variable number of
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
60 * arguments yet.
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
61 */
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
62 - ${Socket::AbstractSKBuff.ref} skb = self->skbuff;
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
63 - ${Ethernet::ProtocolId} ethernet_proto = { .id = be16_to_cpu(${local.skb.k_sk_buff}->protocol) };
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
64 - static const char * const ip_summed_values[] = {
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
65 - "none", "unnecessary", "complete", "partial"
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
66 - };
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
67 - struct skb_shared_info *shinfo = skb_shinfo(${local.skb.k_sk_buff});
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
68 -
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
69 pr_info(
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
70 "\t protocol = %#-5x (%s) ip_summed = %d (%s)\n"
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
71 "\t len = %-5u data_len = %-5u head_len = %-5u\n"
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
72 @@ -58,53 +57,46 @@
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
73 "\t gso_size = %-5u gso_segs = %-5u gso_type = %-5u",
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
74 /* XXX: can't use ${local.ethernet_proto.id} here (issue #52): */
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
75 ethernet_proto.id, ${local.ethernet_proto.str},
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
76 - ${local.skb.k_sk_buff}->ip_summed, ip_summed_values[${local.skb.k_sk_buff}->ip_summed],
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
77 - ${local.skb.k_sk_buff}->len, ${local.skb.k_sk_buff}->data_len, skb_headlen(${local.skb.k_sk_buff}),
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
78 + ${local.self.sk_buff.k_sk_buff}->ip_summed, ip_summed_values[${local.self.sk_buff.k_sk_buff}->ip_summed],
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
79 + ${local.self.sk_buff.k_sk_buff}->len, ${local.self.sk_buff.k_sk_buff}->data_len, skb_headlen(${local.self.sk_buff.k_sk_buff}),
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
80 shinfo->nr_frags, shinfo->gso_size, shinfo->gso_segs, shinfo->gso_type
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
81 );
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
82 }
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
83
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
84 static int rtx_socket_skbuff_map(${Socket::SKBuff.ref} self,
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
85 - struct device *dev,
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
86 - enum dma_data_direction direction)
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
87 + ${Device::AbstractDevice.ref} dev,
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
88 + ${DMA::DMADirection.scalar} direction)
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
89 {
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
90 - ${Socket::AbstractSKBuff.ref} skb = self->skbuff;
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
91 + WARN_ON(!${local.self.sk_buff});
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
92 + WARN_ON(${local.self.dma_handle});
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
93
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
94 - WARN_ON(!${local.skb.k_sk_buff});
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
95 - WARN_ON(self->dma_handle);
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
96 -
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
97 - self->dma_handle = dma_map_single(
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
98 - dev,
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
99 - ${local.skb.k_sk_buff},
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
100 - skb_headlen(${local.skb.k_sk_buff}),
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
101 - direction);
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
102 - int err = dma_mapping_error(dev, self->dma_handle);
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
103 + unsigned int len = skb_headlen(${local.self.sk_buff.k_sk_buff});
115
5a663f8f0e54 WIP, fix another small bug in rathaxes sequence calls
Louis Opter <louis@lse.epita.fr>
parents: 114
diff changeset
104 + ${cast local.len as Builtin::number};
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
105 + ${local.self.dma_handle} = ${DMA::map(local.dev, local.self.sk_buff.k_sk_buff, local.len, local.direction)};
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
106 + int err = ${DMA::mapping_error(local.dev, local.self.dma_handle)};
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
107 if (err)
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
108 {
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
109 - self->dma_handle = 0;
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
110 + ${local.self.dma_handle} = 0;
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
111 return err;
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
112 }
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
113 return 0;
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
114 }
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
115
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
116 static void rtx_socket_skbuff_unmap_and_free(${Socket::SKBuff.ref} self,
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
117 - struct device *dev,
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
118 - enum dma_data_direction direction)
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
119 + ${Device::AbstractDevice.ref} dev,
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
120 + ${DMA::DMADirection} direction)
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
121 {
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
122 - ${Socket::AbstractSKBuff.ref} skb = self->skbuff;
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
123 + WARN_ON(!${local.self.sk_buff});
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
124
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
125 - WARN_ON(!${local.skb.k_sk_buff});
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
126 -
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
127 - if (self->dma_handle)
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
128 + if (${local.self.dma_handle})
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
129 {
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
130 - dma_unmap_single(dev,
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
131 - self->dma_handle,
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
132 - skb_headlen(${local.skb.k_sk_buff}),
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
133 - direction);
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
134 - self->dma_handle = 0;
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
135 + unsigned int len = skb_headlen(${local.self.sk_buff.k_sk_buff});
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
136 + ${cast local.len as Builtin::number};
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
137 + ${DMA::unmap(local.dev, local.self.dma_handle, local.len, local.direction)};
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
138 + ${local.self.dma_handle} = 0;
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
139 }
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
140 - dev_kfree_skb_any(${local.skb.k_sk_buff});
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
141 - self->skbuff = 0;
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
142 + dev_kfree_skb_any(${local.self.sk_buff.k_sk_buff});
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
143 + ${local.self.sk_buff} = NULL;
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
144 }
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
145 }
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
146
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
147 @@ -115,8 +107,8 @@
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
148 */
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
149 method init(Socket::AbstractSKBuff kernel_skb)
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
150 {
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
151 - ${self}.skbuff = ${kernel_skb};
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
152 - ${self}.dma_handle = 0;
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
153 + ${self.sk_buff} = ${kernel_skb};
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
154 + ${self.dma_handle} = 0;
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
155 }
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
156
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
157 method dump_infos()
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
158 @@ -126,30 +118,37 @@
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
159
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
160 method map_to(Device::AbstractDevice dev)
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
161 {
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
162 - rtx_socket_skbuff_map(${self}, ${dev.k_device}, DMA_TO_DEVICE);
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
163 + rtx_socket_skbuff_map(${self}, ${dev}, RTX_DMA_TO_DEVICE);
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
164 }
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
165
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
166 method map_from(Device::AbstractDevice dev)
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
167 {
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
168 - rtx_socket_skbuff_map(${self}, ${dev.k_device}, DMA_FROM_DEVICE);
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
169 + rtx_socket_skbuff_map(${self}, ${dev}, RTX_DMA_FROM_DEVICE);
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
170 }
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
171
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
172 method unmap_to_and_free(Device::AbstractDevice dev)
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
173 {
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
174 - rtx_socket_skbuff_unmap_and_free(${self}, ${dev.k_device}, DMA_TO_DEVICE);
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
175 + rtx_socket_skbuff_unmap_and_free(${self}, ${dev}, RTX_DMA_TO_DEVICE);
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
176 }
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
177
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
178 method unmap_from_and_free(Device::AbstractDevice dev)
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
179 {
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
180 - rtx_socket_skbuff_unmap_and_free(${self}, ${dev.k_device}, DMA_FROM_DEVICE);
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
181 + rtx_socket_skbuff_unmap_and_free(${self}, ${dev}, RTX_DMA_FROM_DEVICE);
112
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
182 }
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
183
Louis Opter <louis@lse.epita.fr>
parents: 111
diff changeset
184 map
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
185 {
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
186 - // some work may have to be done here in order
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
187 - // to access to some field of the sk_buff.
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
188 - // We should determine if all the sk_buff management
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
189 - // can be abstracted from the user.
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
190 + // Some work may have to be done here in order to access to some
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
191 + // field of the sk_buff. We should determine if all the sk_buff
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
192 + // management can be abstracted from the user. But this is at least
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
193 + // useful for internal use:
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
194 + sk_buff: (${self})->skbuff;
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
195 + // XXX: We need to cast here so we can do things like
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
196 + // var.dma_handle = 0; but the type shouldn't be hardcoded (at the
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
197 + // same time ${DMA:AbstractDMAHandle} couldn't be used because that
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
198 + // would yield to a struct type which you can't assign directly;
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
199 + // but maybe doing the ->data in that case would be acceptable).
118
ad21d8a182ad WIP: put up (hopefully the last) some touches on the dma abstraction
Louis Opter <louis@lse.epita.fr>
parents: 117
diff changeset
200 + dma_handle: (*((dma_addr_t *)&(${self})->dma_handle));
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
201 }
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
202 }
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
203 }
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
204 diff --git a/rathaxes/samples/e1000/socket.rti b/rathaxes/samples/e1000/socket.rti
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
205 --- a/rathaxes/samples/e1000/socket.rti
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
206 +++ b/rathaxes/samples/e1000/socket.rti
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
207 @@ -1,4 +1,4 @@
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
208 -interface Socket : LKM, Device
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
209 +interface Socket : LKM, Device, DMA
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
210 {
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
211 /* The SKBuff type from the kernel */
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
212 provided type AbstractSKBuff
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
213 @@ -19,5 +19,8 @@
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
214 method map_from(Device::AbstractDevice);
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
215 method unmap_to_and_free(Device::AbstractDevice);
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
216 method unmap_from_and_free(Device::AbstractDevice);
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
217 +
117
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
218 + attribute Socket::AbstractSKBuff.ref sk_buff;
f3c7e9b0c5cf WIP on the DMA API
Louis Opter <louis@lse.epita.fr>
parents: 115
diff changeset
219 + attribute DMA::AbstractDMAHandle.scalar dma_handle;
113
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
220 }
b22983cb9e3a WIP, thanks joa for some fixes on the compiler
Louis Opter <louis@lse.epita.fr>
parents: 112
diff changeset
221 }