08.05.2024 20:44, Philippe Mathieu-Daudé wrote:
From: Mattias Nissler <mniss...@rivosinc.com>

PCI config space is little-endian, so on a big-endian host we need to
perform byte swaps for values as they are passed to and received from
the generic PCI config space access machinery.

Is this a material for stable?

Thanks,

/mjt

  hw/remote/vfio-user-obj.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/hw/remote/vfio-user-obj.c b/hw/remote/vfio-user-obj.c
index d9b879e056..8dbafafb9e 100644
--- a/hw/remote/vfio-user-obj.c
+++ b/hw/remote/vfio-user-obj.c
@@ -281,7 +281,7 @@ static ssize_t vfu_object_cfg_access(vfu_ctx_t *vfu_ctx, 
char * const buf,
      while (bytes > 0) {
          len = (bytes > pci_access_width) ? pci_access_width : bytes;
          if (is_write) {
-            memcpy(&val, ptr, len);
+            val = ldn_le_p(ptr, len);
              pci_host_config_write_common(o->pci_dev, offset,
                                           pci_config_size(o->pci_dev),
                                           val, len);
@@ -289,7 +289,7 @@ static ssize_t vfu_object_cfg_access(vfu_ctx_t *vfu_ctx, 
char * const buf,
          } else {
              val = pci_host_config_read_common(o->pci_dev, offset,
                                                pci_config_size(o->pci_dev), 
len);
-            memcpy(ptr, &val, len);
+            stn_le_p(ptr, len, val);
              trace_vfu_cfg_read(offset, val);
          }
          offset += len;

--
GPG Key transition (from rsa2048 to rsa4096) since 2024-04-24.
New key: rsa4096/61AD3D98ECDF2C8E  9D8B E14E 3F2A 9DD7 9199  28F1 61AD 3D98 
ECDF 2C8E
Old key: rsa2048/457CE0A0804465C5  6EE1 95D1 886E 8FFB 810D  4324 457C E0A0 
8044 65C5
Transition statement: http://www.corpit.ru/mjt/gpg-transition-2024.txt


Reply via email to