Mercurial > archived > louis > epitech > mq > rathaxes
annotate rathaxes_correctly_use_chunk_and_template_sequences_parameters_in_e1000.patch @ 69:8bb53638585b
WIP on the e1000 sample: fix build
| author | Louis Opter <louis@lse.epitech.net> |
|---|---|
| date | Sat, 18 Feb 2012 18:13:36 +0100 |
| parents | ef460b936b5f |
| children | c22a9acf8cd7 |
| rev | line source |
|---|---|
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
1 # HG changeset patch |
|
66
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
2 # Parent 0197bb30a5d0fe40c5c67c510a9f076d2272d45d |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
3 # User Louis Opter <louis@lse.epitech.net>, David Pineau <dav.pineau@gmail.com> |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
4 rathaxes: correctly use chunk and template sequences parameters in the e1000 sample |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
5 |
| 63 | 6 diff --git a/rathaxes/samples/e1000/e1000.blt b/rathaxes/samples/e1000/e1000.blt |
| 7 --- a/rathaxes/samples/e1000/e1000.blt | |
| 8 +++ b/rathaxes/samples/e1000/e1000.blt | |
| 9 @@ -15,18 +15,20 @@ | |
| 10 | |
| 11 chunk ::decl() | |
| 12 { | |
| 13 - struct rtx_e1000_ctx | |
| 14 + /* | |
| 15 + * Yes, this typedef looks ugly but read the remark about | |
| 16 + * Ethernet::Device in ethernet.blt. | |
| 17 + */ | |
| 18 + typedef struct rtx_e1000_ctx | |
| 19 { | |
| 20 int bars; | |
| 21 unsigned char /* __iomem */ *ioaddr; | |
| 22 int irq; | |
| 23 - }; | |
| 24 + } *rtx_e1000_ctx_p; | |
| 25 } | |
| 26 | |
| 27 - chunk ::init(bars, ioaddr) | |
| 28 + chunk ::init() | |
| 29 { | |
| 30 - ${self}.bars = ${bars}; | |
| 31 - ${self}.ioaddr = ${ioaddr}; | |
| 32 } | |
| 33 | |
| 34 map | |
| 35 @@ -136,58 +138,74 @@ | |
| 36 | |
| 37 template sequence e1000::create_device() | |
| 38 { | |
| 39 - chunk Ethernet::create_device() | |
| 40 + chunk Ethernet::create_device(/* PCI::Device */ pdev, /* Ethernet::Device */ rtx_ether_ctx) | |
| 41 { | |
| 42 - rtx_ether_ctx->hw_ctx.irq = pdev->irq; | |
| 43 - rtx_ether_ctx->hw_ctx.bars = pci_select_bars(pdev, IORESOURCE_MEM); | |
| 44 - if (pci_enable_device_mem(pdev)) | |
| 45 + /* | |
| 46 + * PCI init stuff: | |
| 47 + * | |
| 48 + * Some of that code should be moved in the PCI blts, also at some | |
| 49 + * point maybe we could do that completely automatically in the PCI | |
| 50 + * blts. | |
| 51 + */ | |
| 52 + | |
| 53 + /* | |
| 54 + * We could have used an init function here but since we can't init | |
| 55 + * all the fields at once (see, ioaddr) and cannot call a C | |
| 56 + * function within a placeholder (${}), it wasn't really worth it. | |
| 57 + */ | |
| 64 | 58 + ${rtx_ether_ctx}->hw_ctx.bars = pci_select_bars(${pdev}, IORESOURCE_MEM); |
| 59 + ${rtx_ether_ctx}->hw_ctx.irq = ${pdev}->irq; | |
| 63 | 60 + |
| 61 + if (pci_enable_device_mem(${pdev})) | |
| 62 { | |
| 63 ${Log::info("e1000::create: pci_enable_device_mem failed")}; | |
| 64 } | |
| 65 - | |
| 66 - if (pci_request_selected_regions(pdev, rtx_ether_ctx->hw_ctx.bars, ${config.name})) | |
| 67 + if (pci_request_selected_regions(${pdev}, ${rtx_ether_ctx}->hw_ctx.bars, ${config.name})) | |
| 68 { | |
| 69 ${Log::info("e1000::create: pci_request_selected_regions failed")}; | |
| 70 } | |
| 71 - | |
| 72 if (${config.set_master}) | |
| 73 { | |
| 74 - pci_set_master(pdev); | |
| 75 + pci_set_master(${pdev}); | |
| 76 } | |
| 77 | |
| 78 /* 0 here is for BAR_0: */ | |
| 79 - rtx_ether_ctx->hw_ctx.ioaddr = pci_ioremap_bar(pdev, 0); | |
| 80 - if (!rtx_ether_ctx->hw_ctx.ioaddr) | |
| 81 + ${rtx_ether_ctx}->hw_ctx.ioaddr = pci_ioremap_bar(${pdev}, 0); | |
| 82 + if (!${rtx_ether_ctx}->hw_ctx.ioaddr) | |
| 83 { | |
| 84 ${Log::info("e1000::create: pci_ioremap_bar failed")}; | |
| 85 } | |
| 86 | |
| 87 + /* | |
| 88 + * The really device specific algorithm starts here (so it should | |
| 89 + * certainly be written in the frontend): | |
| 90 + */ | |
| 91 + | |
| 92 /* Reset the card */ | |
| 93 - rtx_e1000_register_write32(&rtx_ether_ctx->hw_ctx, E1000_CTRL, E1000_CMD_RST); | |
| 94 + rtx_e1000_register_write32(&${rtx_ether_ctx}->hw_ctx, E1000_CTRL, E1000_CMD_RST); | |
| 95 udelay(10); | |
| 96 | |
| 97 - /* Now we can load its mac address */ | |
| 98 + /* Now we can load its mac address (thanks minix code) */ | |
| 99 int i = 0; | |
| 100 - for (i = 0 /* < this is not generated! */; i < 3; ++i) | |
| 101 + for (i = 0 /* < this is not generated! (cnorm bug) */; i < 3; ++i) | |
| 102 { | |
| 103 - rtx_e1000_register_write32(&rtx_ether_ctx->hw_ctx, E1000_EEPROM_READ, (i << 8) | 1); | |
| 104 + rtx_e1000_register_write32(&${rtx_ether_ctx}->hw_ctx, E1000_EEPROM_READ, (i << 8) | 1); | |
| 105 | |
| 106 int value; | |
| 107 - /* Should be a do { } while(); but the compiler doesn't do { } while(); yet. */ | |
| 108 - value = rtx_e1000_register_read32(&rtx_ether_ctx->hw_ctx, E1000_EEPROM_READ); | |
| 109 - while ((value & (1 << 4)) == 0) | |
| 110 - value = rtx_e1000_register_read32(&rtx_ether_ctx->hw_ctx, E1000_EEPROM_READ); | |
| 111 + do | |
| 112 + value = rtx_e1000_register_read32(&${rtx_ether_ctx}->hw_ctx, E1000_EEPROM_READ); | |
| 113 + while ((value & (1 << 4)) == 0); | |
| 114 value >>= 16; | |
| 115 | |
| 116 - rtx_ether_ctx->net_dev->dev_addr[i * 2] = value & 0xff; | |
| 117 - rtx_ether_ctx->net_dev->dev_addr[i * 2 + 1] = (value >> 8) & 0xff; | |
| 118 + ${rtx_ether_ctx}->net_dev->dev_addr[i * 2] = value & 0xff; | |
| 119 + ${rtx_ether_ctx}->net_dev->dev_addr[i * 2 + 1] = (value >> 8) & 0xff; | |
| 120 } | |
| 121 | |
| 122 - memcpy(rtx_ether_ctx->net_dev->perm_addr, | |
| 123 - rtx_ether_ctx->net_dev->dev_addr, | |
| 124 - rtx_ether_ctx->net_dev->addr_len); | |
| 125 + memcpy(${rtx_ether_ctx}->net_dev->perm_addr, | |
| 126 + ${rtx_ether_ctx}->net_dev->dev_addr, | |
| 127 + ${rtx_ether_ctx}->net_dev->addr_len); | |
| 128 | |
| 129 - { /* < mais lol. */ | |
| 130 + { /* < See #10 */ | |
| 131 ${Log::info("e1000::create: mac address loaded from the EEPROM")}; | |
| 132 } | |
| 133 } | |
| 134 @@ -199,15 +217,14 @@ | |
| 135 | |
| 136 template sequence e1000::destroy_device() | |
| 137 { | |
| 138 - chunk Ethernet::destroy_device | |
| 139 + chunk Ethernet::destroy_device(/* PCI::Device */ pdev, /* Ethernet::Device */ rtx_ether_ctx) | |
| 140 { | |
| 141 /* | |
| 142 * Here, we should have some checks to avoid to free resources that | |
| 143 * haven't been allocated. (e.g: in case of previous errors). | |
| 144 */ | |
| 145 - struct rtx_ethernet_dev* rtx_ether_ctx = netdev_priv(net_dev); | |
| 146 - iounmap(rtx_ether_ctx->hw_ctx.ioaddr); | |
| 147 - pci_release_selected_regions(pdev, rtx_ether_ctx->hw_ctx.bars); | |
| 148 + iounmap(${rtx_ether_ctx}->hw_ctx.ioaddr); | |
| 149 + pci_release_selected_regions(${pdev}, ${rtx_ether_ctx}->hw_ctx.bars); | |
| 150 } | |
| 151 | |
| 152 chunk ::CALL | |
| 65 | 153 @@ -227,7 +244,7 @@ |
| 154 static void rtx_e1000_print_status(struct rtx_e1000_ctx *ctx) | |
| 155 { | |
| 156 unsigned int status = rtx_e1000_register_read32(ctx, E1000_STATUS); | |
| 157 - pr_info("rtx_e1000 status: \n"); | |
| 158 + pr_info("card status: \n"); | |
| 159 pr_info("\tRegister value: 0x%x\n", status); | |
| 160 pr_info("\tMode: %s\n", (status & 1) ? "Full": "Half"); | |
| 161 pr_info("\tLink: %s\n", (status & 2) ? "Up" : "Down"); | |
| 63 | 162 @@ -275,7 +292,7 @@ |
| 163 | |
| 164 chunk ::CALL() | |
| 165 { | |
| 166 - rtx_e1000_register_read32(&${ctx}, ${reg_offset}); | |
| 167 + rtx_e1000_register_read32(${ctx}, ${reg_offset}); | |
| 168 } | |
| 169 } | |
| 170 | |
| 171 @@ -296,7 +313,7 @@ | |
| 172 | |
| 173 chunk ::CALL() | |
| 174 { | |
| 175 - rtx_e1000_register_write32(&${ctx}, ${reg_offset}); | |
| 176 + rtx_e1000_register_write32(${ctx}, ${reg_offset}); | |
| 177 } | |
| 178 } | |
| 179 | |
|
69
8bb53638585b
WIP on the e1000 sample: fix build
Louis Opter <louis@lse.epitech.net>
parents:
68
diff
changeset
|
180 @@ -320,12 +337,13 @@ |
| 63 | 181 } |
| 182 } | |
| 183 | |
| 184 - template sequence e1000::setup_interrupt_handler() | |
| 185 + template sequence e1000::setup_interrupt_handler(Ethernet::Device ctx) | |
| 186 { | |
| 187 chunk LKM::includes() | |
| 188 { | |
|
69
8bb53638585b
WIP on the e1000 sample: fix build
Louis Opter <louis@lse.epitech.net>
parents:
68
diff
changeset
|
189 #include <linux/interrupt.h> |
|
8bb53638585b
WIP on the e1000 sample: fix build
Louis Opter <louis@lse.epitech.net>
parents:
68
diff
changeset
|
190 } |
|
8bb53638585b
WIP on the e1000 sample: fix build
Louis Opter <louis@lse.epitech.net>
parents:
68
diff
changeset
|
191 + |
|
8bb53638585b
WIP on the e1000 sample: fix build
Louis Opter <louis@lse.epitech.net>
parents:
68
diff
changeset
|
192 chunk LKM::prototypes() |
|
8bb53638585b
WIP on the e1000 sample: fix build
Louis Opter <louis@lse.epitech.net>
parents:
68
diff
changeset
|
193 { |
|
8bb53638585b
WIP on the e1000 sample: fix build
Louis Opter <louis@lse.epitech.net>
parents:
68
diff
changeset
|
194 static int e1000_setup_interrupt_handler(struct rtx_ethernet_dev *); |
|
8bb53638585b
WIP on the e1000 sample: fix build
Louis Opter <louis@lse.epitech.net>
parents:
68
diff
changeset
|
195 @@ -345,20 +363,20 @@ |
| 65 | 196 |
| 197 if (error) | |
| 198 { | |
| 199 - ${Log::info("Cannot register the interruption")}; | |
| 200 + ${Log::info("cannot register the interrupt handler")}; | |
| 201 } | |
| 202 | |
| 203 return error; | |
| 63 | 204 } |
| 205 } | |
| 206 | |
| 207 - chunk ::CALL() | |
| 208 + chunk ::CALL | |
| 209 { | |
| 210 // this is an hack for the scope | |
| 211 (void)1; | |
| 212 { | |
| 213 int error; | |
| 214 - error = e1000_setup_interrupt_handler(rtx_ether_dev); | |
| 215 + error = e1000_setup_interrupt_handler(${ctx}); | |
| 216 if (error) | |
| 217 { | |
| 218 return error; | |
|
69
8bb53638585b
WIP on the e1000 sample: fix build
Louis Opter <louis@lse.epitech.net>
parents:
68
diff
changeset
|
219 @@ -423,7 +441,7 @@ |
| 65 | 220 intr = rtx_e1000_register_read32(ctx, E1000_ICR); |
| 221 if (intr & E1000_INTR_LSC) | |
| 222 { | |
| 223 - ${Log::info("Link status changed")}; | |
| 224 + ${Log::info("cable link status changed")}; | |
| 225 } | |
| 226 | |
| 227 if (intr) | |
| 63 | 228 diff --git a/rathaxes/samples/e1000/e1000.rti b/rathaxes/samples/e1000/e1000.rti |
| 229 --- a/rathaxes/samples/e1000/e1000.rti | |
| 230 +++ b/rathaxes/samples/e1000/e1000.rti | |
| 231 @@ -1,25 +1,22 @@ | |
| 232 interface e1000 : Socket, Ethernet, PCI, LKM | |
| 233 { | |
| 234 provided type e1000::Context; | |
| 235 + /* | |
| 236 + * These two types should actually be registers definitions in the frontend: | |
| 237 + */ | |
| 238 provided type e1000::Register; | |
| 239 provided type e1000::Commands; | |
| 240 | |
| 241 - /* | |
| 242 - * This sequence should receive an argument like Ethernet::Device, but it is | |
| 243 - * unclear about how this argument should be bound to a variable/argument in | |
| 244 - * the instrumented C code. | |
| 245 - * | |
| 246 - * Here again, we rely on the fact that *we* wrote the parent context and | |
| 247 - * named the C variables we need/use with the same name everywhere. | |
| 248 - */ | |
| 249 provided sequence e1000::create_device() | |
| 250 { | |
| 251 + /* should take PCI::Device and Ethernet::Device args: */ | |
| 252 provided chunk Ethernet::create_device; | |
| 253 provided chunk ::CALL; | |
| 254 } | |
| 255 | |
| 256 provided sequence e1000::destroy_device() | |
| 257 { | |
| 258 + /* should take PCI::Device and Ethernet::Device args: */ | |
| 259 provided chunk Ethernet::destroy_device; | |
| 260 provided chunk ::CALL; | |
| 261 } | |
| 262 @@ -35,9 +32,9 @@ | |
| 263 provided chunk ::CALL; | |
| 264 } | |
| 265 | |
| 266 - provided sequence e1000::setup_interrupt_handler() | |
| 267 + provided sequence e1000::setup_interrupt_handler(Ethernet::Device) | |
| 268 { | |
| 269 - provided chunk LKM::includes; // work without this one | |
| 270 + provided chunk LKM::includes; // works without this one | |
| 271 provided chunk LKM::prototypes; | |
| 272 provided chunk LKM::code; | |
| 273 provided chunk ::CALL; | |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
274 diff --git a/rathaxes/samples/e1000/ethernet.blt b/rathaxes/samples/e1000/ethernet.blt |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
275 --- a/rathaxes/samples/e1000/ethernet.blt |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
276 +++ b/rathaxes/samples/e1000/ethernet.blt |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
277 @@ -1,5 +1,10 @@ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
278 with Ethernet, PCI, LKM, Log |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
279 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
280 + /* |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
281 + * Unlike PCI::Device, Ethernet::Device doesn't match the struct net_device |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
282 + * from Linux. Ethernet::Device is the type that we use in the private |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
283 + * field of the struct net_device. |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
284 + */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
285 template type Ethernet::Device() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
286 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
287 chunk LKM::includes() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
288 @@ -10,7 +15,14 @@ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
289 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
290 chunk ::decl() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
291 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
292 - struct rtx_ethernet_dev |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
293 + /* |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
294 + * So, at first sight, it sucks to typedef it as pointer but (for |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
295 + * now) it makes sense for two reasons: |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
296 + * - This structure will always be used through a pointer; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
297 + * - This remove the ambiguity of pointer/not-pointer in the |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
298 + * ::init() chunk. |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
299 + */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
300 + typedef struct rtx_ethernet_dev |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
301 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
302 /* |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
303 * I think it's useless to use the ${PCI::Device} "abstraction" |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
304 @@ -21,12 +33,18 @@ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
305 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
306 /* while waiting on issue #8 */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
307 struct rtx_e1000_ctx hw_ctx; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
308 - }; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
309 + } *rtx_ethernet_dev_p; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
310 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
311 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
312 - chunk ::init(net_dev) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
313 + chunk ::init(net_dev, pci_dev) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
314 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
315 - ${self} = ${net_dev}; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
316 + ${self} = netdev_priv(${net_dev}); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
317 + /* |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
318 + * We can use -> because we know that ${self} will be always a |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
319 + * pointer ("thanks" to the typedef) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
320 + */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
321 + ${self}->pci_dev = ${pci_dev}; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
322 + ${self}->net_dev = ${net_dev}; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
323 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
324 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
325 map |
| 64 | 326 @@ -45,10 +63,10 @@ |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
327 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
328 static int rtx_ethernet_open(struct net_device *dev) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
329 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
330 - struct rtx_ethernet_dev* rtx_ether_dev = netdev_priv(dev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
331 + struct rtx_ethernet_dev* rtx_ether_dev = netdev_priv(dev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
332 struct rtx_e1000_ctx* ctx = &rtx_ether_dev->hw_ctx; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
333 |
| 64 | 334 - ${pointcut ::IMPLEMENTATION}; |
|
69
8bb53638585b
WIP on the e1000 sample: fix build
Louis Opter <louis@lse.epitech.net>
parents:
68
diff
changeset
|
335 + ${pointcut ::IMPLEMENTATION(local.rtx_ether_dev)}; |
| 64 | 336 |
| 337 return 0; | |
| 338 } | |
|
66
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
339 @@ -66,7 +84,7 @@ |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
340 { |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
341 static int rtx_ethernet_xmit(struct sk_buff* skb, struct net_device *dev) |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
342 { |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
343 - ${pointcut ::IMPLEMENTATION}; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
344 + ${pointcut ::IMPLEMENTATION(local.skb, local.dev)}; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
345 |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
346 return 0; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
347 } |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
348 @@ -87,7 +105,7 @@ |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
349 struct rtx_ethernet_dev* rtx_ether_dev = netdev_priv(dev); |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
350 struct rtx_e1000_ctx* ctx = &rtx_ether_dev->hw_ctx; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
351 |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
352 - ${pointcut ::IMPLEMENTATION}; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
353 + ${pointcut ::IMPLEMENTATION(local.rtx_ether_dev)}; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
354 |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
355 return 0; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
356 } |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
357 @@ -114,14 +132,14 @@ |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
358 |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
359 rtx_ether_dev = dev_id; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
360 ctx = &rtx_ether_dev->hw_ctx; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
361 - ${pointcut ::IMPLEMENTATION}; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
362 + ${pointcut ::IMPLEMENTATION(local.rtx_ether_dev)}; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
363 |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
364 return IRQ_NONE; |
|
901af221334b
Adding implementation poinctuts parameters to e1000 sample
David Pineau <dav.pineau@gmail.com>
parents:
64
diff
changeset
|
365 } |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
366 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
367 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
368 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
369 - template sequence Ethernet::init(PCI::Device pdev) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
370 + template sequence Ethernet::init() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
371 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
372 chunk LKM::data() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
373 { |
| 65 | 374 @@ -133,83 +151,75 @@ |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
375 }; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
376 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
377 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
378 - chunk PCI::pci_probe_hook() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
379 + /* For now the type is not handled, so we just omit it (see #17) */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
380 + chunk PCI::pci_probe_hook(/* PCI::Device */ pdev) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
381 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
382 /* |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
383 * This typedef is needed to workaround a bug in CNorm __std__ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
384 * dialect. |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
385 */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
386 typedef int ${Ethernet::Device}; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
387 - ${Ethernet::Device} *rtx_ether_ctx; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
388 - struct net_device *net_dev; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
389 + |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
390 + ${Ethernet::Device} rtx_ether_ctx; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
391 + struct net_device *net_dev; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
392 int error; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
393 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
394 - error = 0; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
395 + /* Initialize the net_device structure */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
396 net_dev = alloc_etherdev(sizeof(*rtx_ether_ctx)); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
397 if (net_dev == 0) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
398 { |
| 65 | 399 - ${Log::info("Cannot allocate memory")}; |
| 400 + ${Log::info("cannot allocate memory")}; | |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
401 /* |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
402 * Again, the error should be "raised" in the parent context. |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
403 * |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
404 - * Here we know that we should return ENOMEM because *we* wrote |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
405 + * Here we know that we can return ENOMEM because *we* wrote |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
406 * the parent context. |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
407 */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
408 return -ENOMEM; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
409 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
410 + SET_NETDEV_DEV(net_dev, &${pdev}->dev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
411 strlcpy(net_dev->name, ${config.ifname}, sizeof(net_dev->name)); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
412 - net_dev->irq = pdev->irq; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
413 - // Maybe we should try ${rtx_ether_ctx.init()} here: |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
414 - rtx_ether_ctx = netdev_priv(net_dev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
415 - //rtx_ether_ctx->pci_dev = ${pdev}; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
416 - rtx_ether_ctx->pci_dev = pdev; // In the meantime do it directly |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
417 - rtx_ether_ctx->net_dev = net_dev; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
418 + net_dev->irq = ${pdev}->irq; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
419 + net_dev->netdev_ops = &rtx_ether_ops; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
420 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
421 - /* |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
422 - * The substitution of ${pdev} fails here. I also tried to add a |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
423 - * "substitute method" to the PCI::Device that was just doing |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
424 - * "${self}" but it didn't work either (it was subsituted by a |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
425 - * placeholder, e.g: _1). |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
426 - * |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
427 - * That's why we cheated a bit and named all the arguments pdev. |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
428 - */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
429 - //SET_NETDEV_DEV(net_dev, &${pdev}->dev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
430 - SET_NETDEV_DEV(net_dev, &pdev->dev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
431 - net_dev->netdev_ops = &rtx_ether_ops; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
432 - if ((error = register_netdev(net_dev))) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
433 + error = register_netdev(net_dev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
434 + if (error) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
435 { |
| 65 | 436 - ${Log::info("Cannot register the driver")}; |
| 437 + ${Log::info("cannot register the driver in the net subsystem")}; | |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
438 return error; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
439 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
440 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
441 - /* same problem as above with ${pdev} */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
442 - //pci_set_drvdata(${pdev}, net_dev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
443 - pci_set_drvdata(pdev, net_dev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
444 + /* Initialize our context held by the net_device structure */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
445 + ${rtx_ether_ctx.init(local.net_dev, pdev)}; |
| 63 | 446 |
| 447 - ${pointcut Ethernet::create_device}; | |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
448 + pci_set_drvdata(${pdev}, net_dev); |
| 63 | 449 + |
| 450 + ${pointcut Ethernet::create_device(pdev, local.rtx_ether_ctx)}; | |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
451 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
452 |
| 63 | 453 + /* This chunk should be removed (see #26) */ |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
454 chunk ::CALL |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
455 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
456 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
457 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
458 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
459 - template sequence Ethernet::exit(PCI::Device pdev) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
460 + template sequence Ethernet::exit() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
461 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
462 - chunk PCI::pci_remove_hook() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
463 + chunk PCI::pci_remove_hook(/* PCI::Device */ pdev) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
464 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
465 - struct net_device *net_dev = pci_get_drvdata(pdev); |
| 63 | 466 + /* workaround for cnorm unstrict */ |
| 64 | 467 + typedef int ${Ethernet::Device}; |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
468 |
| 63 | 469 - ${pointcut Ethernet::destroy_device}; |
| 470 + struct net_device *net_dev = pci_get_drvdata(${pdev}); | |
| 64 | 471 + ${Ethernet::Device} rtx_ether_ctx = netdev_priv(net_dev); |
| 63 | 472 + |
| 473 + ${pointcut Ethernet::destroy_device(pdev, local.rtx_ether_ctx)}; | |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
474 |
| 63 | 475 unregister_netdev(net_dev); |
| 476 - /* | |
| 477 - * If we had some cleanup todo with struct rtx_ether_ctx we would | |
| 478 - * do a netdev_priv(net_dev) here and do it. | |
| 479 - */ | |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
480 free_netdev(net_dev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
481 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
482 |
| 63 | 483 + /* This chunk should be removed (see #26) */ |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
484 chunk ::CALL |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
485 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
486 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
487 diff --git a/rathaxes/samples/e1000/ethernet.rti b/rathaxes/samples/e1000/ethernet.rti |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
488 --- a/rathaxes/samples/e1000/ethernet.rti |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
489 +++ b/rathaxes/samples/e1000/ethernet.rti |
| 63 | 490 @@ -28,19 +28,17 @@ |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
491 provided chunk LKM::code; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
492 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
493 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
494 - provided sequence Ethernet::init(PCI::Device) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
495 + provided sequence Ethernet::init() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
496 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
497 provided chunk LKM::data; |
| 63 | 498 - provided chunk PCI::pci_probe_hook; |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
499 - provided chunk ::CALL; |
| 63 | 500 + provided chunk PCI::pci_probe_hook; /* should take a PCI::Device arg */ |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
501 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
502 provided pointcut Ethernet::create_device; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
503 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
504 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
505 - provided sequence Ethernet::exit(PCI::Device) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
506 + provided sequence Ethernet::exit() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
507 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
508 - provided chunk ::CALL; |
| 63 | 509 - provided chunk PCI::pci_remove_hook; |
| 510 + provided chunk PCI::pci_remove_hook; /* should take a PCI::Device arg */ | |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
511 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
512 provided pointcut Ethernet::destroy_device; |
| 63 | 513 } |
| 514 diff --git a/rathaxes/samples/e1000/lkm.rtx b/rathaxes/samples/e1000/lkm.rtx | |
| 515 --- a/rathaxes/samples/e1000/lkm.rtx | |
| 516 +++ b/rathaxes/samples/e1000/lkm.rtx | |
|
68
ef460b936b5f
Merge + remove patch from David
Louis Opter <louis@lse.epitech.net>
parents:
67
diff
changeset
|
517 @@ -2,50 +2,50 @@ |
| 65 | 518 { |
| 63 | 519 Ethernet::open(Ethernet::Device dev) |
| 520 { | |
| 65 | 521 - Log::info("Open the device"); |
| 63 | 522 - e1000::setup_interrupt_handler(); |
| 65 | 523 - Log::info("Interrupt handler installed"); |
| 524 + Log::info("open the device"); | |
| 63 | 525 + e1000::setup_interrupt_handler(dev); |
| 65 | 526 + Log::info("interrupt handler installed"); |
| 63 | 527 e1000::set_up_device(); |
| 528 e1000::activate_device_interruption(); | |
| 65 | 529 } |
| 530 | |
| 531 Ethernet::close(Ethernet::Device dev) | |
| 532 { | |
| 533 - Log::info("Close the device"); | |
| 534 + Log::info("close the device"); | |
| 535 e1000::free_interrupt_handler(); | |
| 536 } | |
| 537 | |
| 538 Ethernet::interrupt_handler(Ethernet::Device dev) | |
| 539 { | |
| 540 - Log::info("Got an interruption"); | |
| 541 + Log::info("got an interruption"); | |
| 542 e1000::handle_intr(); | |
| 543 } | |
| 544 | |
| 545 Ethernet::send(Ethernet::Device dev, Socket::SKBuff skb) | |
| 546 { | |
| 547 - Log::info("We have one packet to transmit!"); | |
| 548 + Log::info("we have one packet to transmit!"); | |
| 549 } | |
| 550 | |
| 551 LKM::init() | |
| 552 { | |
| 553 - Log::info("Hello this is LKM"); | |
| 554 + Log::info("loading module"); | |
| 555 } | |
| 556 | |
| 557 LKM::exit() | |
| 558 { | |
| 559 - Log::info("Good bye this was LKM"); | |
| 560 + Log::info("unloading module"); | |
| 561 } | |
| 562 } | |
| 563 | |
| 564 configuration | |
| 565 { | |
| 566 - LKM::name = "hello"; | |
| 567 + LKM::name = "rtx_e1k"; | |
| 568 LKM::author = "Rathaxes"; | |
| 569 LKM::description = "Hello World Loadable Kernel Module (LKM)"; | |
| 570 LKM::license = "GPL"; | |
|
68
ef460b936b5f
Merge + remove patch from David
Louis Opter <louis@lse.epitech.net>
parents:
67
diff
changeset
|
571 |
|
ef460b936b5f
Merge + remove patch from David
Louis Opter <louis@lse.epitech.net>
parents:
67
diff
changeset
|
572 PCI::vendor_id = 0x8086; |
|
ef460b936b5f
Merge + remove patch from David
Louis Opter <louis@lse.epitech.net>
parents:
67
diff
changeset
|
573 - PCI::product_id = 0x100f; |
|
ef460b936b5f
Merge + remove patch from David
Louis Opter <louis@lse.epitech.net>
parents:
67
diff
changeset
|
574 + PCI::product_id = 0x100e; |
|
ef460b936b5f
Merge + remove patch from David
Louis Opter <louis@lse.epitech.net>
parents:
67
diff
changeset
|
575 PCI::set_master = true; |
|
ef460b936b5f
Merge + remove patch from David
Louis Opter <louis@lse.epitech.net>
parents:
67
diff
changeset
|
576 |
|
ef460b936b5f
Merge + remove patch from David
Louis Opter <louis@lse.epitech.net>
parents:
67
diff
changeset
|
577 Ethernet::ifname = "rtx%d"; |
| 65 | 578 diff --git a/rathaxes/samples/e1000/log.blt b/rathaxes/samples/e1000/log.blt |
| 579 --- a/rathaxes/samples/e1000/log.blt | |
| 580 +++ b/rathaxes/samples/e1000/log.blt | |
| 581 @@ -4,7 +4,7 @@ | |
| 582 { | |
| 583 chunk ::CALL | |
| 584 { | |
| 585 - pr_info("%s\n", ${msg}); | |
| 586 + pr_info("%s: %s\n", ${config.name}, ${msg}); | |
| 587 } | |
| 588 } | |
| 589 } | |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
590 diff --git a/rathaxes/samples/e1000/pci.blt b/rathaxes/samples/e1000/pci.blt |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
591 --- a/rathaxes/samples/e1000/pci.blt |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
592 +++ b/rathaxes/samples/e1000/pci.blt |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
593 @@ -22,7 +22,7 @@ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
594 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
595 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
596 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
597 - template sequence PCI::probe(PCI::Device pdev) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
598 + template sequence PCI::probe() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
599 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
600 chunk LKM::prototypes() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
601 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
602 @@ -41,7 +41,8 @@ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
603 if (err < 0) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
604 goto fail; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
605 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
606 - ${pointcut PCI::pci_probe_hook}; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
607 + /* Use local. to reference a local C variable: */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
608 + ${pointcut PCI::pci_probe_hook(local.pdev)}; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
609 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
610 return 0; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
611 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
612 @@ -50,12 +51,13 @@ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
613 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
614 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
615 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
616 + /* This chunk should be remove (see #26) */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
617 chunk ::CALL |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
618 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
619 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
620 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
621 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
622 - template sequence PCI::remove(PCI::Device pdev) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
623 + template sequence PCI::remove() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
624 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
625 chunk LKM::prototypes() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
626 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
627 @@ -66,12 +68,13 @@ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
628 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
629 static void rtx_pci_remove(struct pci_dev *pdev) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
630 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
631 - ${pointcut PCI::pci_remove_hook}; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
632 + ${pointcut PCI::pci_remove_hook(local.pdev)}; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
633 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
634 pci_disable_device(pdev); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
635 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
636 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
637 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
638 + /* This chunk should be remove (see #26) */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
639 chunk ::CALL() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
640 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
641 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
642 @@ -125,6 +128,8 @@ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
643 * This sequence is just "intermediate" code that will just inject |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
644 * itself in the hook LKM::init_bus_hook for which this sequence |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
645 * has a chunk (see above chunk). |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
646 + * |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
647 + * -> Should be removed see #26 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
648 */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
649 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
650 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
651 @@ -136,6 +141,7 @@ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
652 pci_unregister_driver(&rtx_pci_driver); |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
653 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
654 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
655 + /* This chunk should be removed */ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
656 chunk ::CALL |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
657 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
658 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
659 diff --git a/rathaxes/samples/e1000/pci.rti b/rathaxes/samples/e1000/pci.rti |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
660 --- a/rathaxes/samples/e1000/pci.rti |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
661 +++ b/rathaxes/samples/e1000/pci.rti |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
662 @@ -18,7 +18,7 @@ |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
663 provided chunk LKM::deinit_bus_hook; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
664 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
665 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
666 - provided sequence PCI::probe(PCI::Device) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
667 + provided sequence PCI::probe() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
668 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
669 provided chunk LKM::prototypes; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
670 provided chunk LKM::code; |
| 63 | 671 @@ -26,11 +26,11 @@ |
|
62
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
672 provided pointcut PCI::pci_probe_hook; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
673 } |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
674 |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
675 - provided sequence PCI::remove(PCI::Device) |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
676 + provided sequence PCI::remove() |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
677 { |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
678 provided chunk LKM::prototypes; |
|
b43bed449cc2
Start a series on the e1000 sample
Louis Opter <louis@lse.epitech.net>
parents:
diff
changeset
|
679 provided chunk LKM::code; |
| 63 | 680 |
| 681 - provided pointcut PCI::pci_remove_hook; | |
| 682 + provided pointcut PCI::pci_remove_hook; /* Should take a PCI::Device arg */ | |
| 683 } | |
| 684 } |