mirror of
				https://github.com/qemu/qemu.git
				synced 2025-10-31 12:07:31 +00:00 
			
		
		
		
	 15377f6e79
			
		
	
	
		15377f6e79
		
	
	
	
	
		
			
			There were several different ways to deal with the situation where the vector specified for a msix function is out of bound: - early return a function and keep progresssing - propagate the error to the caller - mark msix unusable - assert it is in bound - just ignore An out-of-bound vector should not be specified if the device implementation is correct so let msix functions always assert that the specified vector is in range. An exceptional case is virtio-pci, which allows the guest to configure vectors. For virtio-pci, it is more appropriate to introduce its own checks because it is sometimes too late to check the vector range in msix functions. Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com> Message-Id: <20220829083524.143640-1-akihiko.odaki@daynix.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Yuval Shaia <yuval.shaia.ml@gmail.com> Signed-off-by: Akihiko Odaki <<a href="mailto:akihiko.odaki@daynix.com" target="_blank">akihiko.odaki@daynix.com</a>><br>
		
			
				
	
	
		
			66 lines
		
	
	
		
			2.5 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			66 lines
		
	
	
		
			2.5 KiB
		
	
	
	
		
			C
		
	
	
	
	
	
| #ifndef QEMU_MSIX_H
 | |
| #define QEMU_MSIX_H
 | |
| 
 | |
| #include "hw/pci/pci.h"
 | |
| 
 | |
| #define MSIX_CAP_LENGTH 12
 | |
| 
 | |
| void msix_set_message(PCIDevice *dev, int vector, MSIMessage msg);
 | |
| MSIMessage msix_get_message(PCIDevice *dev, unsigned int vector);
 | |
| int msix_init(PCIDevice *dev, unsigned short nentries,
 | |
|               MemoryRegion *table_bar, uint8_t table_bar_nr,
 | |
|               unsigned table_offset, MemoryRegion *pba_bar,
 | |
|               uint8_t pba_bar_nr, unsigned pba_offset, uint8_t cap_pos,
 | |
|               Error **errp);
 | |
| int msix_init_exclusive_bar(PCIDevice *dev, unsigned short nentries,
 | |
|                             uint8_t bar_nr, Error **errp);
 | |
| 
 | |
| void msix_write_config(PCIDevice *dev, uint32_t address, uint32_t val, int len);
 | |
| 
 | |
| void msix_uninit(PCIDevice *dev, MemoryRegion *table_bar,
 | |
|                  MemoryRegion *pba_bar);
 | |
| void msix_uninit_exclusive_bar(PCIDevice *dev);
 | |
| 
 | |
| unsigned int msix_nr_vectors_allocated(const PCIDevice *dev);
 | |
| 
 | |
| void msix_save(PCIDevice *dev, QEMUFile *f);
 | |
| void msix_load(PCIDevice *dev, QEMUFile *f);
 | |
| 
 | |
| int msix_enabled(PCIDevice *dev);
 | |
| int msix_present(PCIDevice *dev);
 | |
| 
 | |
| bool msix_is_masked(PCIDevice *dev, unsigned vector);
 | |
| void msix_set_pending(PCIDevice *dev, unsigned vector);
 | |
| void msix_clr_pending(PCIDevice *dev, int vector);
 | |
| 
 | |
| void msix_vector_use(PCIDevice *dev, unsigned vector);
 | |
| void msix_vector_unuse(PCIDevice *dev, unsigned vector);
 | |
| void msix_unuse_all_vectors(PCIDevice *dev);
 | |
| void msix_set_mask(PCIDevice *dev, int vector, bool mask);
 | |
| 
 | |
| void msix_notify(PCIDevice *dev, unsigned vector);
 | |
| 
 | |
| void msix_reset(PCIDevice *dev);
 | |
| 
 | |
| int msix_set_vector_notifiers(PCIDevice *dev,
 | |
|                               MSIVectorUseNotifier use_notifier,
 | |
|                               MSIVectorReleaseNotifier release_notifier,
 | |
|                               MSIVectorPollNotifier poll_notifier);
 | |
| void msix_unset_vector_notifiers(PCIDevice *dev);
 | |
| 
 | |
| extern const VMStateDescription vmstate_msix;
 | |
| 
 | |
| #define VMSTATE_MSIX_TEST(_field, _state, _test) {                   \
 | |
|     .name         = (stringify(_field)),                             \
 | |
|     .size         = sizeof(PCIDevice),                               \
 | |
|     .vmsd         = &vmstate_msix,                                   \
 | |
|     .flags        = VMS_STRUCT,                                      \
 | |
|     .offset       = vmstate_offset_value(_state, _field, PCIDevice), \
 | |
|     .field_exists = (_test)                                          \
 | |
| }
 | |
| 
 | |
| #define VMSTATE_MSIX(_f, _s)                                         \
 | |
|     VMSTATE_MSIX_TEST(_f, _s, NULL)
 | |
| 
 | |
| #endif
 |