Skip to content

xl: convert exit codes to EXIT_[SUCCESS|FAILURE]

Harmandeep Kaur xen-devel 6 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v6 Latest5 patches 2 repliesIn review
  2. v55 patches 1 replySuperseded
  3. v45 patches 2 repliesSuperseded
Earlier versions · 3
VersionPostedPatchesRepliesStatus
v35 patches6 repliesSuperseded
v25 patches8 repliesSuperseded
v13 patches8 repliesSuperseded

Patches in v6 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5xl: convert main() exit codes to EXIT_[SUCCESS|FAILURE]tools/libxl/xl.c tools/libxl/xl.h+13 -60 replies
2/5xl: improve return and exit codes of scheduling related functionstools/libxl/xl_cmdimpl.c+73 -780 replies
3/5xl: improve return and exit codes of vcpu related functionstools/libxl/xl_cmdimpl.c+11 -100 replies
4/5xl: improve return and exit codes of parse related functionstools/libxl/xl_cmdimpl.c+26 -260 replies
5/5xl: improve return and exit codes of parse related functionstools/libxl/xl_cmdimpl.c+23 -230 replies

Reviews trailers per patch

All 5 patches have a Reviewed-by or Acked-by.

PatchDario FaggioliWei Liu
1/5R-b A-b
2/5R-b A-b
3/5R-b A-b
4/5R-b A-b
5/5R-b A-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v6 thread

Cover letter [PATCH v6 0/5] xl: convert exit codes to EXIT_[SUCCESS|FAILURE]

*Its my "bite size contribution" that is required for applying to the Outreachy program.

*main_foo() is treated somewhat as a regular main(), it is changed to return EXIT_SUCCESS or EXIT_FAILURE.

*Functions that are not main_foo(), are changed to do 'return 0' or 'return 1', and then 0/1 is taken care in the main_foo() functions that calls them.

*Functions in xl.c and xl_cmdimpl.c are changed.

*Functions related to scheduling, vcpu, cpupool and parsing are currently changed excluding parse_config_data() which is big enough to deserve its own patch.

*Some discussions about this patch:
https://www.mail-archive.com/[email protected]/msg42850.html

*v1 of this patch:
http://lists.xenproject.org/archives/html/xen-devel/2015-10/msg02497.html

*v2 of this patch:
http://lists.xenproject.org/archives/html/xen-devel/2015-10/msg02623.html

*v3 of this patch:
http://lists.xenproject.org/archives/html/xen-devel/2015-10/msg02872.html

*v4 of this patch:
http://lists.xenproject.org/archives/html/xen-devel/2015-10/msg02947.html

*v5 of this patch:
http://lists.xenproject.org/archives/html/xen-devel/2015-10/msg02990.html

---