[LU-5829] too many EXPORT_SYMBOL in code Created: 30/Oct/14 Updated: 02/Aug/16 Resolved: 19/Jul/15 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.8.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Frank Zago (Inactive) | Assignee: | Cliff White (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | patch | ||
| Issue Links: |
|
||||||||
| Severity: | 3 | ||||||||
| Rank (Obsolete): | 16350 | ||||||||
| Description |
|
Lustre currently exports 1809 symbols, compared to 25395 symbols for the whole linux kernel. A lot of these symbols don't need to be exported at all because they are only used in the module they belong to. |
| Comments |
| Comment by Frank Zago (Inactive) [ 30/Oct/14 ] |
|
Removing 565 unnecessary exports: http://review.whamcloud.com/12510 |
| Comment by Gerrit Updater [ 09/Jan/15 ] |
|
frank zago (fzago@cray.com) uploaded a new patch: http://review.whamcloud.com/13319 |
| Comment by Gerrit Updater [ 09/Jan/15 ] |
|
frank zago (fzago@cray.com) uploaded a new patch: http://review.whamcloud.com/13320 |
| Comment by Gerrit Updater [ 09/Jan/15 ] |
|
frank zago (fzago@cray.com) uploaded a new patch: http://review.whamcloud.com/13321 |
| Comment by Gerrit Updater [ 09/Jan/15 ] |
|
frank zago (fzago@cray.com) uploaded a new patch: http://review.whamcloud.com/13323 |
| Comment by Gerrit Updater [ 09/Jan/15 ] |
|
frank zago (fzago@cray.com) uploaded a new patch: http://review.whamcloud.com/13324 |
| Comment by Gerrit Updater [ 03/Feb/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13321/ |
| Comment by Gerrit Updater [ 03/Feb/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13323/ |
| Comment by Gerrit Updater [ 04/Feb/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12510/ |
| Comment by Gerrit Updater [ 18/Mar/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13320/ |
| Comment by Gerrit Updater [ 18/Mar/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13324/ |
| Comment by James A Simmons [ 15/Jul/15 ] |
|
One patch left that once landed we can close this ticket. |
| Comment by Gerrit Updater [ 19/Jul/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13319/ |
| Comment by Peter Jones [ 19/Jul/15 ] |
|
Landed for 2.8 |
| Comment by Gerrit Updater [ 18/Sep/15 ] |
|
frank zago (fzago@cray.com) uploaded a new patch: http://review.whamcloud.com/16482 |
| Comment by Frank Zago (Inactive) [ 18/Sep/15 ] |
|
Incorrect LU in commit. Please ignore that. |
| Comment by Andrew Perepechko [ 02/Aug/16 ] |
This patch is not correct. It removes e.g. lu_ref_add() export, but it is used from other modules (you can check with grep). |
| Comment by Frank Zago (Inactive) [ 02/Aug/16 ] |
|
Fix for that issue is in |