qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 229d33: linux-user: fix statfs


From: GitHub
Subject: [Qemu-commits] [qemu/qemu] 229d33: linux-user: fix statfs
Date: Fri, 19 Oct 2012 12:30:13 -0700

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 229d3376a38bf97aa09b6f73a957c5389badcd06
      
https://github.com/qemu/qemu/commit/229d3376a38bf97aa09b6f73a957c5389badcd06
  Author: Alexander Graf <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M linux-user/syscall.c

  Log Message:
  -----------
  linux-user: fix statfs

The statfs syscall should always memset(0) its full struct extent before
writing to it. Newer versions of the syscall use one of the reserved fields
for flags, which would otherwise get stale values from uncleaned memory.

This fixes libarchive for me, which got confused about the return value of
pathconf("/", _PC_REC_XFER_ALIGN) otherwise, as it some times gave old pointers
as return value.

Signed-off-by: Alexander Graf <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: 1bdd7c7ea8a711efcb5141663865cc1f7e4e824d
      
https://github.com/qemu/qemu/commit/1bdd7c7ea8a711efcb5141663865cc1f7e4e824d
  Author: Alexander Graf <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M linux-user/syscall.c

  Log Message:
  -----------
  linux-user: fix multi-threaded /proc/self/maps

When reading our faked /proc/self/maps from a secondary thread,
we get an invalid stack entry. This is because ts->stack_base is not
initialized in non-primary threads.

However, ts->info is, and the stack layout information we're looking
for is there too. So let's use that one instead!

Signed-off-by: Alexander Graf <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: f287b2c2d4d20d35880ab6dca44bda0476e67dce
      
https://github.com/qemu/qemu/commit/f287b2c2d4d20d35880ab6dca44bda0476e67dce
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M linux-user/syscall.c

  Log Message:
  -----------
  linux-user: Perform more checks on iovec lists

Validate count between 0 and IOV_MAX.  Limit total length of
operation in the same way the kernel does.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: 3d21d29c32380384e5ee5b804d0b0bf720469d97
      
https://github.com/qemu/qemu/commit/3d21d29c32380384e5ee5b804d0b0bf720469d97
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M linux-user/syscall.c

  Log Message:
  -----------
  linux-user: Implement gethostname

Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: b7fb2310136090aab86004363f7c031b30845f2f
      
https://github.com/qemu/qemu/commit/b7fb2310136090aab86004363f7c031b30845f2f
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M linux-user/alpha/target_signal.h

  Log Message:
  -----------
  alpha-linux-user: Fix sigaltstack structure definition

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: a05c64091509056b7e321537196db967f2545601
      
https://github.com/qemu/qemu/commit/a05c64091509056b7e321537196db967f2545601
  Author: Richard Henderson <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M linux-user/qemu.h
    M linux-user/signal.c
    M linux-user/syscall.c

  Log Message:
  -----------
  linux-user: Fix siginfo handling

Compare signal numbers in the proper domain.
Convert all of the fields for SIGIO and SIGCHLD.

Signed-off-by: Richard Henderson <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: 885c1d10b803fc37e6656e733ba916c702b6f515
      
https://github.com/qemu/qemu/commit/885c1d10b803fc37e6656e733ba916c702b6f515
  Author: Peter Maydell <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M linux-user/linuxload.c
    M linux-user/main.c

  Log Message:
  -----------
  linux-user: If loading fails, print error as string, not number

If the attempt to load the guest executable fails, print the
error message as a string, not a number. This requires us to
fix a couple of places in loader_exec() where we were returning
-1 instead of a valid negative errno.

The change allows us to drop the "Unknown binary format" message
because the strerror-enhanced message is now a more self-explanatory
"Error while loading $guest-binary: Exec format error".

Signed-off-by: Peter Maydell <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: 30163d89953e2ec3e5fc53918682c8bc4b1b3b8d
      
https://github.com/qemu/qemu/commit/30163d89953e2ec3e5fc53918682c8bc4b1b3b8d
  Author: Peter Maydell <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: Remove unnecessary host_guest_base code

All TCG hosts now support guest-base functionality, so we can
remove the setting of host_guest_base to 'yes' in every arm
of the case "$cpu" statement, and simply set guest_base to
default to 'yes'.

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: 07e10e5de1470bdf1d1ed97c85cb7ed9e4826775
      
https://github.com/qemu/qemu/commit/07e10e5de1470bdf1d1ed97c85cb7ed9e4826775
  Author: Peter Maydell <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M tcg/arm/tcg-target.h
    M tcg/hppa/tcg-target.h
    M tcg/i386/tcg-target.h
    M tcg/ia64/tcg-target.h
    M tcg/mips/tcg-target.h
    M tcg/ppc/tcg-target.h
    M tcg/ppc64/tcg-target.h
    M tcg/s390/tcg-target.h
    M tcg/sparc/tcg-target.h
    M tcg/tcg.c
    M tcg/tci/tcg-target.h

  Log Message:
  -----------
  tcg: Remove TCG_TARGET_HAS_GUEST_BASE define

GUEST_BASE support is now supported by all TCG backends, and is
now mandatory. Drop the now-pointless TCG_TARGET_HAS_GUEST_BASE
define (set by every backend) and the error if it is unset.

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Richard Henderson <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: 4a1def4e4ec2f0eb72b15596a04a030cdc889370
      
https://github.com/qemu/qemu/commit/4a1def4e4ec2f0eb72b15596a04a030cdc889370
  Author: Alexander Graf <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M linux-user/syscall.c

  Log Message:
  -----------
  linux-user: ppc: mark as long long aligned

The SysV PPC32 ABI dictates that long long (64bit) parameters are pass in 
odd/even
register pairs. Because unlike ARM and MIPS we start at an odd register number,
we can reuse the same aligning code that ARM and MIPS use.

Clarified inline comment that it is SysV ABI that requires long long aligned
parameters - Riku

Signed-off-by: Alexander Graf <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: ae017a5b95962f68ece21065376cd3266998fd02
      
https://github.com/qemu/qemu/commit/ae017a5b95962f68ece21065376cd3266998fd02
  Author: Alexander Graf <address@hidden>
  Date:   2012-10-12 (Fri, 12 Oct 2012)

  Changed paths:
    M linux-user/syscall.c

  Log Message:
  -----------
  linux-user: register align p{read, write}64

pread64 and pwrite64 pass 64bit parameters which for some architectures need
to be aligned to special argument pairs, creating a gap argument.

Handle this special case the same way we handle it in other places of the code.

Reported-by: Alex Barcelo <address@hidden>
Signed-off-by: Alexander Graf <address@hidden>
Tested-by: Alex Barcelo <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>


  Commit: fe3e7f2dc05225cdd2ba40defcd4e2581bebc5e0
      
https://github.com/qemu/qemu/commit/fe3e7f2dc05225cdd2ba40defcd4e2581bebc5e0
  Author: Joel Martin <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M ui/vnc.c

  Log Message:
  -----------
  ui/vnc: Only report/use TIGHT_PNG encoding if enabled.

If TIGHT_PNG is not enabled by the --enable-vnc-png configure flag
then do not report to the client that it is supported.

Also, since TIGHT_PNG is the same as the TIGHT encoding but with the
filter/copy replaced with PNG data, adding it to the supported
encodings list when it is disabled will cause the TIGHT encoding to be
used even though the client requested TIGHT_PNG.

Signed-off-by: Joel Martin <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 301592ea6f9c23d233fadc73124301d1305af13c
      
https://github.com/qemu/qemu/commit/301592ea6f9c23d233fadc73124301d1305af13c
  Author: Peter Maydell <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M targphys.h

  Log Message:
  -----------
  targphys.h: Don't define target_phys_addr_t for user-mode emulators

Commit 4be403c accidentally defined the target_phys_addr_t type when
building user-mode emulators. Since the type doesn't really make
any sense except for system emulators, avoid defining it when building
in user mode.

Signed-off-by: Peter Maydell <address@hidden>
Acked-by: Andreas Färber <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 02c7ac0c0f00ce7fb259a0d68c2590a09adb73f4
      
https://github.com/qemu/qemu/commit/02c7ac0c0f00ce7fb259a0d68c2590a09adb73f4
  Author: Peter Maydell <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M target-arm/neon_helper.c

  Log Message:
  -----------
  target-arm/neon_helper: Remove obsolete FIXME comment

Commit 33ebc29 fixed the bugs in the implementation of VQRSHL,
but forgot to remove the FIXME comment...

Signed-off-by: Peter Maydell <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: abab1a0f32e44cb4e6c317e24d1705f182b88cd0
      
https://github.com/qemu/qemu/commit/abab1a0f32e44cb4e6c317e24d1705f182b88cd0
  Author: Stefan Weil <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: Remove unused parameters from main function

This modification is required if compiler option -Wunused-parameter is 
activated.

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


  Commit: 08778b398328c4978e6e6bed023e37a3141dba84
      
https://github.com/qemu/qemu/commit/08778b398328c4978e6e6bed023e37a3141dba84
  Author: Stefan Weil <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M net/tap-win32.c

  Log Message:
  -----------
  net/tap-win32: Fix compiler warning caused by missing include statement

The include file for net_init_tap was missing:

net/tap-win32.c:703:
 warning: no previous prototype for ‘net_init_tap’

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


  Commit: 11f66978618b542986172a62fcd6acfa36c0d178
      
https://github.com/qemu/qemu/commit/11f66978618b542986172a62fcd6acfa36c0d178
  Author: Peter Maydell <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M ui/vnc-jobs.c

  Log Message:
  -----------
  ui/vnc-jobs.c: Fix minor typos in comments

Fix some minor typos/grammar errors in comments.

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Stefan Weil <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 3e50da65255bf4026b15e71299f4a5a40382a057
      
https://github.com/qemu/qemu/commit/3e50da65255bf4026b15e71299f4a5a40382a057
  Author: Aurelien Jarno <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M configure
    M net/tap-win32.c
    M target-arm/neon_helper.c
    M targphys.h
    M ui/vnc-jobs.c
    M ui/vnc.c

  Log Message:
  -----------
  Merge branch 'trivial-patches' of git://github.com/stefanha/qemu

* 'trivial-patches' of git://github.com/stefanha/qemu:
  ui/vnc-jobs.c: Fix minor typos in comments
  net/tap-win32: Fix compiler warning caused by missing include statement
  configure: Remove unused parameters from main function
  target-arm/neon_helper: Remove obsolete FIXME comment
  targphys.h: Don't define target_phys_addr_t for user-mode emulators
  ui/vnc: Only report/use TIGHT_PNG encoding if enabled.


  Commit: 41a05a457620539f84950c2f52293963388275b0
      
https://github.com/qemu/qemu/commit/41a05a457620539f84950c2f52293963388275b0
  Author: Aurelien Jarno <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M configure
    M linux-user/alpha/target_signal.h
    M linux-user/linuxload.c
    M linux-user/main.c
    M linux-user/qemu.h
    M linux-user/signal.c
    M linux-user/syscall.c
    M tcg/arm/tcg-target.h
    M tcg/hppa/tcg-target.h
    M tcg/i386/tcg-target.h
    M tcg/ia64/tcg-target.h
    M tcg/mips/tcg-target.h
    M tcg/ppc/tcg-target.h
    M tcg/ppc64/tcg-target.h
    M tcg/s390/tcg-target.h
    M tcg/sparc/tcg-target.h
    M tcg/tcg.c
    M tcg/tci/tcg-target.h

  Log Message:
  -----------
  Merge branch 'linux-user-for-upstream' of 
git://git.linaro.org/people/rikuvoipio/qemu

* 'linux-user-for-upstream' of git://git.linaro.org/people/rikuvoipio/qemu:
  linux-user: register align p{read, write}64
  linux-user: ppc: mark as long long aligned
  tcg: Remove TCG_TARGET_HAS_GUEST_BASE define
  configure: Remove unnecessary host_guest_base code
  linux-user: If loading fails, print error as string, not number
  linux-user: Fix siginfo handling
  alpha-linux-user: Fix sigaltstack structure definition
  linux-user: Implement gethostname
  linux-user: Perform more checks on iovec lists
  linux-user: fix multi-threaded /proc/self/maps
  linux-user: fix statfs


  Commit: 585f60368f23e6603cf86cfdaeceb89d1169f4b8
      
https://github.com/qemu/qemu/commit/585f60368f23e6603cf86cfdaeceb89d1169f4b8
  Author: Peter Maydell <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M qemu-options.hx

  Log Message:
  -----------
  qemu-options.hx: Change from recommending '?' to 'help'

Update the -help output and documentation so that it recommends
'help' rather than '?' for the various "list valid values for this
option" cases. '?' is deprecated (as it can fail confusingly if
not quoted), so it's better to steer users towards 'help'. ('?'
still works, for backwards compatibility.)

This is the -help option part of the change otherwise done in
commit c8057f9, since we are now past release 1.2 and free to
change our help text without worrying about breaking libvirt.

Signed-off-by: Peter Maydell <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Signed-off-by: Aurelien Jarno <address@hidden>


  Commit: 38f419f35225decdbaea9fe1fd00218f8924ce84
      
https://github.com/qemu/qemu/commit/38f419f35225decdbaea9fe1fd00218f8924ce84
  Author: Jan Kiszka <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M configure

  Log Message:
  -----------
  configure: Fix CONFIG_QEMU_HELPERDIR generation

We need to evaluate $libexecdir in configure, otherwise we literally end
up with "${prefix}/libexec" instead of the absolute path as
CONFIG_QEMU_HELPERDIR.

Signed-off-by: Jan Kiszka <address@hidden>
Signed-off-by: Aurelien Jarno <address@hidden>


  Commit: 1cd0f8ce731574afbf219b7ae2b938f305b77e86
      
https://github.com/qemu/qemu/commit/1cd0f8ce731574afbf219b7ae2b938f305b77e86
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M MAINTAINERS

  Log Message:
  -----------
  MAINTAINERS: Update email address for Stefan Hajnoczi

Switch to my new work email address from which I am contributing.

Signed-off-by: Stefan Hajnoczi <address@hidden>
Signed-off-by: Aurelien Jarno <address@hidden>


  Commit: 42a159284570b6fbb0b097e18c7590c094a5188c
      
https://github.com/qemu/qemu/commit/42a159284570b6fbb0b097e18c7590c094a5188c
  Author: Michael Roth <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M exec-all.h

  Log Message:
  -----------
  tci: fix build breakage for target-sparc

commit c28ae41 introduced GETPC() usage for sparc, which is currently
not defined when building with --enable-tcg-interpreter. Add sparc to
the list of targets we selectively define GETPC() for.

Signed-off-by: Michael Roth <address@hidden>
Reviewed-by: Stefan Weil <address@hidden>
Signed-off-by: Aurelien Jarno <address@hidden>


  Commit: 7748b8cb1d100105753a80976d2d02ab107d8107
      
https://github.com/qemu/qemu/commit/7748b8cb1d100105753a80976d2d02ab107d8107
  Author: Mike Frysinger <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M Makefile

  Log Message:
  -----------
  allow make {dist, }clean work w/out configure

There's no reason to require configure to run before running a clean
target, so check MAKECMDGOALS before.

Signed-off-by: Mike Frysinger <address@hidden>
Signed-off-by: Aurelien Jarno <address@hidden>


  Commit: f62cb1b6ddc2c82694abac23ab5eeddd85800074
      
https://github.com/qemu/qemu/commit/f62cb1b6ddc2c82694abac23ab5eeddd85800074
  Author: Catalin Patulea <address@hidden>
  Date:   2012-10-19 (Fri, 19 Oct 2012)

  Changed paths:
    M tests/tcg/Makefile
    M tests/tcg/linux-test.c
    M tests/tcg/test-i386.c
    M tests/tcg/test_path.c

  Log Message:
  -----------
  tests/tcg: fix build

This broke when the tests were moved from tests/ to tests/tcg/.

On x86_64 host/i386-linux-user non-kvm guest, test-i386 and test-mmap are 
broken, but at least they build.

To build/run the tests:
$ cd $BUILD_PATH/tests/tcg
$ SRC_PATH=path/to/qemu make <target>

Signed-off-by: Catalin Patulea <address@hidden>
Reviewed-by: Peter Maydell <address@hidden>
Signed-off-by: Aurelien Jarno <address@hidden>


Compare: https://github.com/qemu/qemu/compare/d7dce494769e...f62cb1b6ddc2

reply via email to

[Prev in Thread] Current Thread [Next in Thread]