[V2] musl: Update to latest trunk

Message ID 20181129175214.3041-1-raj.khem@gmail.com
State Accepted
Commit e0564cbd6c824ba9ee1ed20a261261228fb594e3
Headers show
Series
  • [V2] musl: Update to latest trunk
Related show

Commit Message

Khem Raj Nov. 29, 2018, 5:52 p.m.
Complete changelogs are here
https://git.musl-libc.org/cgit/musl/log/?qt=range&q=c50985d5c8e316c5c464f352e79eeebfed1121a9..39ef612aa193cc6e954ac5a01574300ccd4b7ef9

Signed-off-by: Khem Raj <raj.khem@gmail.com>

---
V2: move to latest master ( 3 more commits for regressions )

 meta/recipes-core/musl/musl_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.19.2

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Comments

Richard Purdie Nov. 30, 2018, 5:04 p.m. | #1
On Thu, 2018-11-29 at 09:52 -0800, Khem Raj wrote:
> Complete changelogs are here

> https://git.musl-libc.org/cgit/musl/log/?qt=range&q=c50985d5c8e316c5c464f352e79eeebfed1121a9..39ef612aa193cc6e954ac5a01574300ccd4b7ef9

> 

> Signed-off-by: Khem Raj <raj.khem@gmail.com>

> ---

> V2: move to latest master ( 3 more commits for regressions )


Sorry, still fails :( :

https://autobuilder.yoctoproject.org/typhoon/#/builders/64/builds/39
https://autobuilder.yoctoproject.org/typhoon/#/builders/45/builds/39

Cheers,

Richard



-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
André Draszik Jan. 21, 2019, 9:14 a.m. | #2
Hi,

On Thu, 2018-11-29 at 09:52 -0800, Khem Raj wrote:
> Complete changelogs are here

> https://git.musl-libc.org/cgit/musl/log/?qt=range&q=c50985d5c8e316c5c464f352e79eeebfed1121a9..39ef612aa193cc6e954ac5a01574300ccd4b7ef9

> 

> Signed-off-by: Khem Raj <raj.khem@gmail.com>

> ---

> V2: move to latest master ( 3 more commits for regressions )

> 

>  meta/recipes-core/musl/musl_git.bb | 2 +-

>  1 file changed, 1 insertion(+), 1 deletion(-)

> 

> diff --git a/meta/recipes-core/musl/musl_git.bb b/meta/recipes-

> core/musl/musl_git.bb

> index 0d8f8eb2a4..b416ec45bf 100644

> --- a/meta/recipes-core/musl/musl_git.bb

> +++ b/meta/recipes-core/musl/musl_git.bb

> @@ -4,7 +4,7 @@

>  require musl.inc

>  inherit linuxloader

>  

> -SRCREV = "c50985d5c8e316c5c464f352e79eeebfed1121a9"

> +SRCREV = "39ef612aa193cc6e954ac5a01574300ccd4b7ef9"


With this, link detection doesn't seem to work correctly anymore, instead
constant messages in the syslog:

    Atheros 8031 ethernet 30be0000.etherne:01: attached PHY driver [Atheros 8031 ethernet] (mii_bus:phy_addr=30be0000.etherne:01, irq=-1)
    IPv6: ADDRCONF(NETDEV_UP): eth1: link is not ready
    fec 30bf0000.ethernet eth1: Link is Up - 1Gbps/Full - flow control rx/tx
    IPv6: ADDRCONF(NETDEV_CHANGE): eth1: link becomes ready
    Atheros 8031 ethernet 30be0000.etherne:01: attached PHY driver [Atheros 8031 ethernet] (mii_bus:phy_addr=30be0000.etherne:01, irq=-1)
    IPv6: ADDRCONF(NETDEV_UP): eth1: link is not ready
    fec 30bf0000.ethernet eth1: Link is Up - 1Gbps/Full - flow control rx/tx
    IPv6: ADDRCONF(NETDEV_CHANGE): eth1: link becomes ready
    Atheros 8031 ethernet 30be0000.etherne:01: attached PHY driver [Atheros 8031 ethernet] (mii_bus:phy_addr=30be0000.etherne:01, irq=-1)

etc.

I've tracked it down to commit d1395c43c019 ("allow freeaddrinfo of
arbitrary sublists of addrinfo list" in musl.

Even latest musl master de7dc1318f49 doesn't work for me with the same
problem. The only way to make it work is to revert d1395c43c019 in musl,
even with latest master.

link detection should be a kernel thing, or not? So how can a user-space
change affect the kernel's link detection?

Any ideas?


Cheers,
Andre'


-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
Khem Raj Jan. 21, 2019, 4:17 p.m. | #3
Hi André

this feature was implemented but does not emulate full
glibc behavior and its possible that app is expecting the glibc
behavior, I have fixed nfsutils app in past see for details

https://patchwork.kernel.org/patch/10707759/

I think you can start with suspecting app, I dont rule out libc bug
but thats less likely

On Mon, Jan 21, 2019 at 1:14 AM André Draszik <git@andred.net> wrote:
>
> Hi,
>
> On Thu, 2018-11-29 at 09:52 -0800, Khem Raj wrote:
> > Complete changelogs are here
> > https://git.musl-libc.org/cgit/musl/log/?qt=range&q=c50985d5c8e316c5c464f352e79eeebfed1121a9..39ef612aa193cc6e954ac5a01574300ccd4b7ef9
> >
> > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > ---
> > V2: move to latest master ( 3 more commits for regressions )
> >
> >  meta/recipes-core/musl/musl_git.bb | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-core/musl/musl_git.bb b/meta/recipes-
> > core/musl/musl_git.bb
> > index 0d8f8eb2a4..b416ec45bf 100644
> > --- a/meta/recipes-core/musl/musl_git.bb
> > +++ b/meta/recipes-core/musl/musl_git.bb
> > @@ -4,7 +4,7 @@
> >  require musl.inc
> >  inherit linuxloader
> >
> > -SRCREV = "c50985d5c8e316c5c464f352e79eeebfed1121a9"
> > +SRCREV = "39ef612aa193cc6e954ac5a01574300ccd4b7ef9"
>
> With this, link detection doesn't seem to work correctly anymore, instead
> constant messages in the syslog:
>
>     Atheros 8031 ethernet 30be0000.etherne:01: attached PHY driver [Atheros 8031 ethernet] (mii_bus:phy_addr=30be0000.etherne:01, irq=-1)
>     IPv6: ADDRCONF(NETDEV_UP): eth1: link is not ready
>     fec 30bf0000.ethernet eth1: Link is Up - 1Gbps/Full - flow control rx/tx
>     IPv6: ADDRCONF(NETDEV_CHANGE): eth1: link becomes ready
>     Atheros 8031 ethernet 30be0000.etherne:01: attached PHY driver [Atheros 8031 ethernet] (mii_bus:phy_addr=30be0000.etherne:01, irq=-1)
>     IPv6: ADDRCONF(NETDEV_UP): eth1: link is not ready
>     fec 30bf0000.ethernet eth1: Link is Up - 1Gbps/Full - flow control rx/tx
>     IPv6: ADDRCONF(NETDEV_CHANGE): eth1: link becomes ready
>     Atheros 8031 ethernet 30be0000.etherne:01: attached PHY driver [Atheros 8031 ethernet] (mii_bus:phy_addr=30be0000.etherne:01, irq=-1)
>
> etc.
>
> I've tracked it down to commit d1395c43c019 ("allow freeaddrinfo of
> arbitrary sublists of addrinfo list" in musl.
>
> Even latest musl master de7dc1318f49 doesn't work for me with the same
> problem. The only way to make it work is to revert d1395c43c019 in musl,
> even with latest master.
>
> link detection should be a kernel thing, or not? So how can a user-space
> change affect the kernel's link detection?
>
> Any ideas?
>
>
> Cheers,
> Andre'
>
>
André Draszik Jan. 21, 2019, 4:52 p.m. | #4
Thanks Khem,

So is it actually user-space that does link state detection? I thought it's
the kernel.

I use connman here, I'll have a look, maybe some memory corruption causing
this so...

Thanks again,
Andre'


On Mon, 2019-01-21 at 08:17 -0800, Khem Raj wrote:
> Hi André
> 
> this feature was implemented but does not emulate full
> glibc behavior and its possible that app is expecting the glibc
> behavior, I have fixed nfsutils app in past see for details
> 
> https://patchwork.kernel.org/patch/10707759/
> 
> I think you can start with suspecting app, I dont rule out libc bug
> but thats less likely
> 
> On Mon, Jan 21, 2019 at 1:14 AM André Draszik <git@andred.net> wrote:
> > Hi,
> > 
> > On Thu, 2018-11-29 at 09:52 -0800, Khem Raj wrote:
> > > Complete changelogs are here
> > > https://git.musl-libc.org/cgit/musl/log/?qt=range&q=c50985d5c8e316c5c464f352e79eeebfed1121a9..39ef612aa193cc6e954ac5a01574300ccd4b7ef9
> > > 
> > > Signed-off-by: Khem Raj <raj.khem@gmail.com>
> > > ---
> > > V2: move to latest master ( 3 more commits for regressions )
> > > 
> > >  meta/recipes-core/musl/musl_git.bb | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/meta/recipes-core/musl/musl_git.bb b/meta/recipes-
> > > core/musl/musl_git.bb
> > > index 0d8f8eb2a4..b416ec45bf 100644
> > > --- a/meta/recipes-core/musl/musl_git.bb
> > > +++ b/meta/recipes-core/musl/musl_git.bb
> > > @@ -4,7 +4,7 @@
> > >  require musl.inc
> > >  inherit linuxloader
> > > 
> > > -SRCREV = "c50985d5c8e316c5c464f352e79eeebfed1121a9"
> > > +SRCREV = "39ef612aa193cc6e954ac5a01574300ccd4b7ef9"
> > 
> > With this, link detection doesn't seem to work correctly anymore,
> > instead
> > constant messages in the syslog:
> > 
> >     Atheros 8031 ethernet 30be0000.etherne:01: attached PHY driver
> > [Atheros 8031 ethernet] (mii_bus:phy_addr=30be0000.etherne:01, irq=-1)
> >     IPv6: ADDRCONF(NETDEV_UP): eth1: link is not ready
> >     fec 30bf0000.ethernet eth1: Link is Up - 1Gbps/Full - flow control
> > rx/tx
> >     IPv6: ADDRCONF(NETDEV_CHANGE): eth1: link becomes ready
> >     Atheros 8031 ethernet 30be0000.etherne:01: attached PHY driver
> > [Atheros 8031 ethernet] (mii_bus:phy_addr=30be0000.etherne:01, irq=-1)
> >     IPv6: ADDRCONF(NETDEV_UP): eth1: link is not ready
> >     fec 30bf0000.ethernet eth1: Link is Up - 1Gbps/Full - flow control
> > rx/tx
> >     IPv6: ADDRCONF(NETDEV_CHANGE): eth1: link becomes ready
> >     Atheros 8031 ethernet 30be0000.etherne:01: attached PHY driver
> > [Atheros 8031 ethernet] (mii_bus:phy_addr=30be0000.etherne:01, irq=-1)
> > 
> > etc.
> > 
> > I've tracked it down to commit d1395c43c019 ("allow freeaddrinfo of
> > arbitrary sublists of addrinfo list" in musl.
> > 
> > Even latest musl master de7dc1318f49 doesn't work for me with the same
> > problem. The only way to make it work is to revert d1395c43c019 in musl,
> > even with latest master.
> > 
> > link detection should be a kernel thing, or not? So how can a user-space
> > change affect the kernel's link detection?
> > 
> > Any ideas?
> > 
> > 
> > Cheers,
> > Andre'
> > 
> >

Patch

diff --git a/meta/recipes-core/musl/musl_git.bb b/meta/recipes-core/musl/musl_git.bb
index 0d8f8eb2a4..b416ec45bf 100644
--- a/meta/recipes-core/musl/musl_git.bb
+++ b/meta/recipes-core/musl/musl_git.bb
@@ -4,7 +4,7 @@ 
 require musl.inc
 inherit linuxloader
 
-SRCREV = "c50985d5c8e316c5c464f352e79eeebfed1121a9"
+SRCREV = "39ef612aa193cc6e954ac5a01574300ccd4b7ef9"
 
 PV = "1.1.20+git${SRCPV}"