[Qemu-commits] [qemu/qemu] 0193c6: xenfb: map framebuffer read-only and handle unmap ...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 0193c62c94643a837832f2b5ccc133434ee740cb
      
https://github.com/qemu/qemu/commit/0193c62c94643a837832f2b5ccc133434ee740cb
  Author: Stefano Stabellini <address@hidden>
  Date:   2014-01-17 (Fri, 17 Jan 2014)

  Changed paths:
    M hw/display/xenfb.c

  Log Message:
  -----------
  xenfb: map framebuffer read-only and handle unmap errors

The framebuffer is needlessly mapped (PROT_READ | PROT_WRITE), map it
PROT_READ instead.

The framebuffer is unmapped by replacing the framebuffer pages with
anonymous shared memory, calling mmap. Check for return errors and print
a warning.

Signed-off-by: Stefano Stabellini <address@hidden>


  Commit: fc33b9004c49f21f8188df5c4cd079a9a3d08de8
      
https://github.com/qemu/qemu/commit/fc33b9004c49f21f8188df5c4cd079a9a3d08de8
  Author: Anthony PERARD <address@hidden>
  Date:   2014-01-17 (Fri, 17 Jan 2014)

  Changed paths:
    M hw/xen/xen_pt.c

  Log Message:
  -----------
  xen_pt: Fix debug output.

Signed-off-by: Anthony PERARD <address@hidden>
Signed-off-by: Stefano Stabellini <address@hidden>
Reviewed-by: Konrad Rzeszutek Wilk <address@hidden>


  Commit: 794798e36eda77802ce7cc7d7d6b1c65751e8a76
      
https://github.com/qemu/qemu/commit/794798e36eda77802ce7cc7d7d6b1c65751e8a76
  Author: Anthony PERARD <address@hidden>
  Date:   2014-01-17 (Fri, 17 Jan 2014)

  Changed paths:
    M hw/xen/xen_pt.c

  Log Message:
  -----------
  xen_pt: Fix passthrough of device with ROM.

QEMU does not need and should not allocate memory for the ROM of a
passthrough PCI device. So this patch initialize the particular region
like any other PCI BAR of a passthrough device.

When a guest will access the ROM, Xen will take care of the IO, QEMU
will not be involved in it.

Xen set a limit of memory available for each guest, allocating memory
for a ROM can hit this limit.

Signed-off-by: Anthony PERARD <address@hidden>
Signed-off-by: Stefano Stabellini <address@hidden>
Reported-and-Tested-by: Konrad Rzeszutek Wilk <address@hidden>


  Commit: 4bf2c138ddefc6ff17f6c4b947320c60aa0c38a0
      
https://github.com/qemu/qemu/commit/4bf2c138ddefc6ff17f6c4b947320c60aa0c38a0
  Author: Hani Benhabiles <address@hidden>
  Date:   2014-01-27 (Mon, 27 Jan 2014)

  Changed paths:
    M net/net.c

  Log Message:
  -----------
  net: Use g_strdup_printf instead of snprintf.

assign_name() in net/net.c is using snprintf + g_strdup to get the same
result as g_strdup_printf.

Signed-off-by: Hani Benhabiles <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 2ad657e3f3af66def47554186a58f1748787a527
      
https://github.com/qemu/qemu/commit/2ad657e3f3af66def47554186a58f1748787a527
  Author: Roy Franz <address@hidden>
  Date:   2014-01-27 (Mon, 27 Jan 2014)

  Changed paths:
    M hw/net/lan9118.c

  Log Message:
  -----------
  Fix lan9118 TX "CMD A" handling

The 9118 ethernet controller supports transmission of multi-buffer packets
with arbitrary byte alignment of the start and end bytes.  All writes to
the packet fifo are 32 bits, so the controller discards bytes at the beginning
and end of each buffer based on the 'Data start offset' and 'Buffer size'
of the TX command 'A' format.

This patch changes the buffer size and offset internal state variables to be
updated on every "TX command A" write.  Previously they were only updated for
the first segment, which resulted incorrect behavior for packets with more
than one segment. Each segment of the packet has its own CMD A command, with
its own buffer size and start offset.

Also update extraction of fields from the CMD A word to use extract32().

Signed-off-by: Roy Franz <address@hidden>
Reviewed-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: c444dfabfc21cb5f093862100e333b808eea32e4
      
https://github.com/qemu/qemu/commit/c444dfabfc21cb5f093862100e333b808eea32e4
  Author: Roy Franz <address@hidden>
  Date:   2014-01-27 (Mon, 27 Jan 2014)

  Changed paths:
    M hw/net/lan9118.c

  Log Message:
  -----------
  Fix lan9118 buffer length handling

The 9118 ethernet controller supports transmission of multi-buffer packets
with arbitrary byte alignment of the start and end bytes.  All writes to
the packet fifo are 32 bits, so the controller discards bytes at the beginning
and end of each buffer based on the 'Data start offset' and 'Buffer size'
of the TX command 'A' format.

This patch uses the provided buffer length to limit the bytes transmitted.
Previously all the bytes of the last 32-bit word written to the TX fifo
were added to the internal transmit buffer structure resulting in more bytes
being transmitted than were submitted to the hardware in the command.  This
resulted in extra bytes being inserted into the middle of multi-buffer
packets when the non-final buffers had non-32bit aligned ending addresses.

Signed-off-by: Roy Franz <address@hidden>
Reviewed-by: Peter Crosthwaite <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 1f149e721feb617d14a3ee4f5911c47b29866a54
      
https://github.com/qemu/qemu/commit/1f149e721feb617d14a3ee4f5911c47b29866a54
  Author: Kusanagi Kouichi <address@hidden>
  Date:   2014-01-27 (Mon, 27 Jan 2014)

  Changed paths:
    M net/tap-linux.c

  Log Message:
  -----------
  tap-linux: Get features once and use it many times

Signed-off-by: Kusanagi Kouichi <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 8e02b359267f8919de799525a4ccd4d37ef2d127
      
https://github.com/qemu/qemu/commit/8e02b359267f8919de799525a4ccd4d37ef2d127
  Author: Peter Maydell <address@hidden>
  Date:   2014-01-30 (Thu, 30 Jan 2014)

  Changed paths:
    M hw/net/lan9118.c
    M net/net.c
    M net/tap-linux.c

  Log Message:
  -----------
  Merge remote-tracking branch 'stefanha/tags/net-pull-request' into staging

Net patches

# gpg: Signature made Mon 27 Jan 2014 14:45:35 GMT using RSA key ID 81AB73C8
# gpg: Can't check signature: public key not found

* stefanha/tags/net-pull-request:
  tap-linux: Get features once and use it many times
  Fix lan9118 buffer length handling
  Fix lan9118 TX "CMD A" handling
  net: Use g_strdup_printf instead of snprintf.

Message-id: address@hidden
Signed-off-by: Peter Maydell <address@hidden>


  Commit: 97374ce538883af677fd94803b71df2d55a9a4de
      
https://github.com/qemu/qemu/commit/97374ce538883af677fd94803b71df2d55a9a4de
  Author: Peter Maydell <address@hidden>
  Date:   2014-01-30 (Thu, 30 Jan 2014)

  Changed paths:
    M hw/display/xenfb.c
    M hw/xen/xen_pt.c

  Log Message:
  -----------
  Merge remote-tracking branch 'sstabellini/xen-170114' into staging

* sstabellini/xen-170114:
  xen_pt: Fix passthrough of device with ROM.
  xen_pt: Fix debug output.
  xenfb: map framebuffer read-only and handle unmap errors

Message-id: address@hidden
Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/dc08f85188b5...97374ce53888