Re: [PULL 0/1] slirp update

[Peter Maydell]

On Mon, 7 Oct 2019 at 00:55, Samuel Thibault
 wrote:
>
> The following changes since commit 860d9048c78ce59c5903c3d5209df56f38400986:
>
>   Merge remote-tracking branch 
> 'remotes/kraxel/tags/audio-20190924-pull-request' into staging (2019-09-24 
> 13:51:51 +0100)
>
> are available in the Git repository at:
>
>   https://people.debian.org/~sthibault/qemu.git tags/samuel-thibault
>
> for you to fetch changes up to 120b721f5b590393971673a315f02969ec89cccb:
>
>   slirp: Allow non-local DNS address when restrict is off (2019-10-01 
> 19:03:08 +0200)
>
> 
> slirp: Allow non-local DNS address when restrict is off
>
> 
> Samuel Thibault (1):
>   slirp: Allow non-local DNS address when restrict is off
>
>  net/slirp.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)



Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/4.2
for any user-visible changes.

-- PMM

[PULL 0/1] slirp update

[Samuel Thibault]

The following changes since commit 860d9048c78ce59c5903c3d5209df56f38400986:

  Merge remote-tracking branch 
'remotes/kraxel/tags/audio-20190924-pull-request' into staging (2019-09-24 
13:51:51 +0100)

are available in the Git repository at:

  https://people.debian.org/~sthibault/qemu.git tags/samuel-thibault

for you to fetch changes up to 120b721f5b590393971673a315f02969ec89cccb:

  slirp: Allow non-local DNS address when restrict is off (2019-10-01 19:03:08 
+0200)


slirp: Allow non-local DNS address when restrict is off


Samuel Thibault (1):
  slirp: Allow non-local DNS address when restrict is off

 net/slirp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

[Qemu-devel] [PULL 0/1] slirp update

[Samuel Thibault]

The following changes since commit cd82b1e170019c4b722ed53116ee9346315d7791:

  slirp: remove empty state.h (2019-03-13 22:12:23 +0100)

are available in the Git repository at:

  https://people.debian.org/~sthibault/qemu.git tags/samuel-thibault

for you to fetch changes up to cd82b1e170019c4b722ed53116ee9346315d7791:

  slirp: remove empty state.h (2019-03-13 22:12:23 +0100)


Slirp updates

Marc-André Lureau (1):
  slirp: remove empty state.h

Re: [Qemu-devel] [PULL 0/1] slirp update

[Samuel Thibault]

Samuel Thibault, le mer. 13 mars 2019 22:14:42 +0100, a ecrit:
> The following changes since commit cd82b1e170019c4b722ed53116ee9346315d7791:
> 
>   slirp: remove empty state.h (2019-03-13 22:12:23 +0100)

Oops, sorry, there was another patch to commit, you can ignore this pull
request.

Samuel

Re: [Qemu-devel] [PULL 0/1] slirp update

[Peter Maydell]

On 9 November 2017 at 18:01, Samuel Thibault
 wrote:
> warning: redirection vers https://people.debian.org/~sthibault/qemu.git/
> The following changes since commit b0fbe46ad82982b289a44ee2495b59b0bad8a842:
>
>   Update version for v2.11.0-rc0 release (2017-11-07 16:05:28 +)
>
> are available in the git repository at:
>
>   http://people.debian.org/~sthibault/qemu.git tags/samuel-thibault
>
> for you to fetch changes up to 990132cda9baa27bdc558df6c9c15aacb0322d2c:
>
>   slirp: don't zero the whole ti_i when m == NULL (2017-11-09 18:59:22 +0100)
>
> 
> slirp updates
>
> 
> Tao Wu (1):
>   slirp: don't zero the whole ti_i when m == NULL
>
> This fixes the qemu slirp behavior is some rare cases (some RST cases,
> keep alive probes), packets would be sent to 0.0.0.0.
>
>  slirp/tcp_subr.c | 11 ++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
>

Applied, thanks.

-- PMM

Re: [Qemu-devel] [PULL 0/1] slirp update

[no-reply]

Hi,

This series seems to have some coding style problems. See output below for
more information:

Subject: [Qemu-devel] [PULL 0/1] slirp update
Type: series
Message-id: 20171109180142.13923-1-samuel.thiba...@ens-lyon.org

=== TEST SCRIPT BEGIN ===
#!/bin/bash

BASE=base
n=1
total=$(git log --oneline $BASE.. | wc -l)
failed=0

git config --local diff.renamelimit 0
git config --local diff.renames True

commits="$(git log --format=%H --reverse $BASE..)"
for c in $commits; do
echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..."
if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then
failed=1
echo
fi
n=$((n+1))
done

exit $failed
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
 * [new tag]   
patchew/20171109180142.13923-1-samuel.thiba...@ens-lyon.org -> 
patchew/20171109180142.13923-1-samuel.thiba...@ens-lyon.org
 * [new tag]   patchew/20171109181741.31318-1-lep...@google.com -> 
patchew/20171109181741.31318-1-lep...@google.com
Switched to a new branch 'test'
9f23f4da08 slirp: don't zero the whole ti_i when m == NULL

=== OUTPUT BEGIN ===
Checking PATCH 1/1: slirp: don't zero the whole ti_i when m == NULL...
ERROR: code indent should never use tabs
#29: FILE: slirp/tcp_subr.c:151:
+^I^Iswitch (af) {$

ERROR: code indent should never use tabs
#30: FILE: slirp/tcp_subr.c:152:
+^I^Icase AF_INET:$

ERROR: code indent should never use tabs
#31: FILE: slirp/tcp_subr.c:153:
+^I^Iti->ti.ti_i4.ih_x1 = 0;$

ERROR: code indent should never use tabs
#32: FILE: slirp/tcp_subr.c:154:
+^I^Ibreak;$

ERROR: code indent should never use tabs
#33: FILE: slirp/tcp_subr.c:155:
+^I^Icase AF_INET6:$

ERROR: code indent should never use tabs
#34: FILE: slirp/tcp_subr.c:156:
+^I^Iti->ti.ti_i6.ih_x1 = 0;$

ERROR: code indent should never use tabs
#35: FILE: slirp/tcp_subr.c:157:
+^I^Ibreak;$

ERROR: code indent should never use tabs
#36: FILE: slirp/tcp_subr.c:158:
+^I^Idefault:$

ERROR: code indent should never use tabs
#37: FILE: slirp/tcp_subr.c:159:
+^I^Ig_assert_not_reached();$

ERROR: code indent should never use tabs
#38: FILE: slirp/tcp_subr.c:160:
+^I^I}$

total: 10 errors, 0 warnings, 17 lines checked

Your patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

=== OUTPUT END ===

Test command exited with code: 1


---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-de...@freelists.org

[Qemu-devel] [PULL 0/1] slirp update

[Samuel Thibault]

warning: redirection vers https://people.debian.org/~sthibault/qemu.git/
The following changes since commit b0fbe46ad82982b289a44ee2495b59b0bad8a842:

  Update version for v2.11.0-rc0 release (2017-11-07 16:05:28 +)

are available in the git repository at:

  http://people.debian.org/~sthibault/qemu.git tags/samuel-thibault

for you to fetch changes up to 990132cda9baa27bdc558df6c9c15aacb0322d2c:

  slirp: don't zero the whole ti_i when m == NULL (2017-11-09 18:59:22 +0100)


slirp updates


Tao Wu (1):
  slirp: don't zero the whole ti_i when m == NULL

This fixes the qemu slirp behavior is some rare cases (some RST cases,
keep alive probes), packets would be sent to 0.0.0.0.

 slirp/tcp_subr.c | 11 ++-
 1 file changed, 10 insertions(+), 1 deletion(-)