All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Anton Ivanov (antivano)" <antivano@cisco.com>
To: Stefan Hajnoczi <stefanha@gmail.com>,
	"arei.gonglei@huawei.com" <arei.gonglei@huawei.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	"Huangweidong (C)" <weidong.huang@huawei.com>,
	"chenliang88@huawei.com" <chenliang88@huawei.com>,
	Luonengjun <luonengjun@huawei.com>,
	qemu-devel <qemu-devel@nongnu.org>,
	Stefan Hajnoczi <stefanha@redhat.com>,
	Paolo Bonzini <pbonzini@redhat.com>
Subject: Re: [Qemu-devel] [PATCH v2] net: add mmsghdr struct check for L2TPV3
Date: Tue, 1 Jul 2014 15:06:14 +0000	[thread overview]
Message-ID: <53B2CE60.7030607@cisco.com> (raw)
In-Reply-To: <CAJSP0QVi7rJg_PVLhbQQ6fi82M97TcFTsUEQJpWTDh6Wo3=bmg@mail.gmail.com>

On 01/07/14 15:35, Stefan Hajnoczi wrote:
> On Tue, Jul 1, 2014 at 2:58 PM,  <arei.gonglei@huawei.com> wrote:
>> From: Gonglei <arei.gonglei@huawei.com>
>>
>> because mmsghdr struct is introduced on Linux kernel 3.3+.
>> add check for mmsghdr struct, it only gets built on hosts that
>> support mmsghdr.

Incorrect.

mmsghdr itself is Linux 2.6.32 and so is recvmmsg. Sendmmsg which will
be introduced in the next version of the patch is 3.0. None of them is 3.3+

Please see release notes for either kernel version.

2.6.32 is now 5 years old it is about time to update to relevant headers
on whatever embedded platform you are using to try to build on.

As far as general purpose linuxes, the relevant libc headers are present
in Ubuntul LTS 12.x and onwards, Debian 7.x, Fedora, and all other major
distros. I do not have a RHEL handy to check, but looking at release
notes that the kernel supporting recmmsg is in 6.0 in the first place
and so are the headers for glibc from certain point onwards.

What appears in 3.3+ is kernel support for l2tpv3 tunnels which is what
would allow a linux host to interop.

A.

>>
>> Reported-by: chenliang <chenliang88@huawei.com>
>> Signed-off-by: Gonglei <arei.gonglei@huawei.com>
>> ---
>> v2->v1:
>>    delete $l2tpv3 variable initialization and
>>    'if' false check suggested by PMM.
>> ---
>>  configure         | 16 ++++++++++++++++
>>  net/Makefile.objs |  2 +-
>>  net/net.c         |  4 ++--
>>  3 files changed, 19 insertions(+), 3 deletions(-)
> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

  reply	other threads:[~2014-07-01 15:06 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-01 12:58 [Qemu-devel] [PATCH v2] net: add mmsghdr struct check for L2TPV3 arei.gonglei
2014-07-01 14:35 ` Stefan Hajnoczi
2014-07-01 15:06   ` Anton Ivanov (antivano) [this message]
2014-07-01 15:14     ` Peter Maydell
2014-07-01 15:26       ` Anton Ivanov (antivano)
2014-07-01 15:34         ` Peter Maydell
2014-07-01 15:58           ` Anton Ivanov (antivano)
2014-07-01 16:01             ` Peter Maydell
2014-07-02  6:21     ` Gonglei (Arei)
2014-07-01 17:27 ` Peter Maydell
2014-07-02  1:04   ` Gonglei (Arei)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53B2CE60.7030607@cisco.com \
    --to=antivano@cisco.com \
    --cc=arei.gonglei@huawei.com \
    --cc=chenliang88@huawei.com \
    --cc=luonengjun@huawei.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@gmail.com \
    --cc=stefanha@redhat.com \
    --cc=weidong.huang@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.