Minor cleanups when parsing the "iommu" option
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v3 Latest2 patches 4 repliesIn review
- v22 patches 2 repliesSuperseded
- v12 patches 2 repliesSuperseded
Changes in v3 from the cover letter
error_report does not need a \n!
Patches in v3 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/2 | q35: Check propery to determine if iommu is sethw/ hw/ include/ | +1 | 1 reply | |
| 2/2 | i440fx: print an error message if user tries to enable iommuhw/ | +5 | 3 replies |
Reviews trailers per patch
All 2 patches have a Reviewed-by or Acked-by.
| Patch | Eric Blake | Markus Armbruster |
|---|---|---|
| 1/2 | R-b | |
| 2/2 | R-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [Qemu-devel] [PATCH v3 0/2] Minor cleanups when parsing the "iommu" option
Small cleanup changes. The first removes the helper function by directly checking the property and the second adds a error message if user tries to use "-machine iommu=on" with i440fx.
v3:error_report does not need a \n!
v2:
2/2: use error_report for the warning message
Bandan Das (2):
q35: Check propery to determine if iommu is set
i440fx: print an error message if user tries to enable iommu
hw/core/machine.c | 5 ----- hw/pci-host/piix.c | 5 +++++ hw/pci-host/q35.c | 2 +- include/hw/boards.h | 1 - 4 files changed, 6 insertions(+), 7 deletions(-)
Signature · 1 line
2.5.0