Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Waldemar Brodkorb <wbx@openadk.org>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/4] package/openobex: disable on Blackfin
Date: Thu, 2 Nov 2017 03:14:34 +0100	[thread overview]
Message-ID: <20171102021434.GJ1829@waldemar-brodkorb.de> (raw)
In-Reply-To: <20171101232343.764c231a@windsurf>

Hi Thomas,
Thomas Petazzoni wrote,

> Hello,
> 
> On Wed,  1 Nov 2017 21:11:34 +0100, Samuel Martin wrote:
> > Openobex uses accept4 syscall, which is not available on Blackfin flavored uclibc.
> > So, diasble openobex on blackfin (do not add extra checks on the c-library since
> > uclibc is the only choice for Blackfin).
> > 
> > Fixed:
> >   http://autobuild.buildroot.net/results/78e033fe9f43845581a5d87b21a8451f67520e44
> > 
> > Signed-off-by: Samuel Martin <s.martin49@gmail.com>
> > ---
> >  package/openobex/Config.in | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/package/openobex/Config.in b/package/openobex/Config.in
> > index e611b8d803..4a4dc214ef 100644
> > --- a/package/openobex/Config.in
> > +++ b/package/openobex/Config.in
> > @@ -1,5 +1,6 @@
> >  config BR2_PACKAGE_OPENOBEX
> >  	bool "openobex"
> > +	depends on !BR2_bfin
> >  	help
> >  	  Free open source implementation of the Object Exchange (OBEX)
> >  	  protocol.
> 
> What about the Config.in comment, and the reverse dependencies of
> BR2_PACKAGE_OPENOBEX ?
> 
> Also, I'd like to hear from Waldemar about accept4 availability on
> Blackfin/uClibc-ng.

For me it looks like a uClibc bug.
When socketcalls.c was converted to use cancel.h for syscall
cancellation (19607f1113ef9916a0a0ac2bf99b5bc32526f0de)
it seems accept4 was left out for no specific reason.

For Linuxthreads the syscall wrapper for accept4 isn't compiled in.
I think for NPTL we do not see the issue.

I am going to sent out a patch soon.

best regards
 Waldemar

  reply	other threads:[~2017-11-02  2:14 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-01 20:11 [Buildroot] [PATCH 1/4] package/openobex: disable on Blackfin Samuel Martin
2017-11-01 20:11 ` [Buildroot] [PATCH 2/4] package/opencv: fix build failure Samuel Martin
2017-11-02 12:32   ` Thomas Petazzoni
2017-11-01 20:11 ` [Buildroot] [PATCH 3/4] package/opencv: add hashes for patch and license files Samuel Martin
2017-11-02 12:33   ` Thomas Petazzoni
2017-11-01 20:11 ` [Buildroot] [PATCH 4/4] package/ussp-push: removed deprecated/no longer needed deps on openobex-bluez Samuel Martin
2017-11-02 13:18   ` Thomas Petazzoni
2017-11-02 21:34   ` Thomas Petazzoni
2017-11-01 22:23 ` [Buildroot] [PATCH 1/4] package/openobex: disable on Blackfin Thomas Petazzoni
2017-11-02  2:14   ` Waldemar Brodkorb [this message]
2017-11-02  8:07     ` Thomas Petazzoni
2017-11-02  8:01   ` Samuel Martin
2017-11-02 10:38     ` Thomas Petazzoni
2017-11-02 10:42       ` Samuel Martin

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=20171102021434.GJ1829@waldemar-brodkorb.de \
    --to=wbx@openadk.org \
    --cc=buildroot@busybox.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox