Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 1 | /// Find functions that refer to GFP_KERNEL but are called with locks held. |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 2 | //# The proposed change of converting the GFP_KERNEL is not necessarily the |
| 3 | //# correct one. It may be desired to unlock the lock, or to not call the |
| 4 | //# function under the lock in the first place. |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 5 | /// |
| 6 | // Confidence: Moderate |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 7 | // Copyright: (C) 2012 Nicolas Palix. GPLv2. |
| 8 | // Copyright: (C) 2012 Julia Lawall, INRIA/LIP6. GPLv2. |
| 9 | // Copyright: (C) 2012 Gilles Muller, INRIA/LiP6. GPLv2. |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 10 | // URL: http://coccinelle.lip6.fr/ |
| 11 | // Comments: |
| 12 | // Options: -no_includes -include_headers |
| 13 | |
| 14 | virtual patch |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 15 | virtual context |
| 16 | virtual org |
| 17 | virtual report |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 18 | |
| 19 | @gfp exists@ |
| 20 | identifier fn; |
| 21 | position p; |
| 22 | @@ |
| 23 | |
| 24 | fn(...) { |
| 25 | ... when != read_unlock_irq(...) |
| 26 | when != write_unlock_irq(...) |
| 27 | when != read_unlock_irqrestore(...) |
| 28 | when != write_unlock_irqrestore(...) |
| 29 | when != spin_unlock(...) |
| 30 | when != spin_unlock_irq(...) |
| 31 | when != spin_unlock_irqrestore(...) |
| 32 | when != local_irq_enable(...) |
| 33 | when any |
| 34 | GFP_KERNEL@p |
| 35 | ... when any |
| 36 | } |
| 37 | |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 38 | @locked exists@ |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 39 | identifier gfp.fn; |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 40 | position p1,p2; |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 41 | @@ |
| 42 | |
| 43 | ( |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 44 | read_lock_irq@p1 |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 45 | | |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 46 | write_lock_irq@p1 |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 47 | | |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 48 | read_lock_irqsave@p1 |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 49 | | |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 50 | write_lock_irqsave@p1 |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 51 | | |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 52 | spin_lock@p1 |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 53 | | |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 54 | spin_trylock@p1 |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 55 | | |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 56 | spin_lock_irq@p1 |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 57 | | |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 58 | spin_lock_irqsave@p1 |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 59 | | |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 60 | local_irq_disable@p1 |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 61 | ) |
| 62 | (...) |
| 63 | ... when != read_unlock_irq(...) |
| 64 | when != write_unlock_irq(...) |
| 65 | when != read_unlock_irqrestore(...) |
| 66 | when != write_unlock_irqrestore(...) |
| 67 | when != spin_unlock(...) |
| 68 | when != spin_unlock_irq(...) |
| 69 | when != spin_unlock_irqrestore(...) |
| 70 | when != local_irq_enable(...) |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 71 | fn@p2(...) |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 72 | |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 73 | @depends on locked && patch@ |
Nicolas Palix | e105007 | 2010-08-24 17:38:58 +0200 | [diff] [blame] | 74 | position gfp.p; |
| 75 | @@ |
| 76 | |
| 77 | - GFP_KERNEL@p |
| 78 | + GFP_ATOMIC |
Julia Lawall | 29a36d4 | 2012-01-14 23:41:54 +0100 | [diff] [blame] | 79 | |
| 80 | @depends on locked && !patch@ |
| 81 | position gfp.p; |
| 82 | @@ |
| 83 | |
| 84 | * GFP_KERNEL@p |
| 85 | |
| 86 | @script:python depends on !patch && org@ |
| 87 | p << gfp.p; |
| 88 | fn << gfp.fn; |
| 89 | p1 << locked.p1; |
| 90 | p2 << locked.p2; |
| 91 | @@ |
| 92 | |
| 93 | cocci.print_main("lock",p1) |
| 94 | cocci.print_secs("call",p2) |
| 95 | cocci.print_secs("GFP_KERNEL",p) |
| 96 | |
| 97 | @script:python depends on !patch && report@ |
| 98 | p << gfp.p; |
| 99 | fn << gfp.fn; |
| 100 | p1 << locked.p1; |
| 101 | p2 << locked.p2; |
| 102 | @@ |
| 103 | |
| 104 | msg = "ERROR: function %s called on line %s inside lock on line %s but uses GFP_KERNEL" % (fn,p2[0].line,p1[0].line) |
| 105 | coccilib.report.print_report(p[0], msg) |