* [PATCH] cryptodev: update SRCREV
@ 2019-04-15 3:42 kai.kang
2019-04-15 7:38 ` Adrian Bunk
2019-04-24 2:23 ` Kang Kai
0 siblings, 2 replies; 5+ messages in thread
From: kai.kang @ 2019-04-15 3:42 UTC (permalink / raw)
To: openembedded-core
From: Kai Kang <kai.kang@windriver.com>
Update SRCREV of cryptodev which only contains one fix for linux 5.0:
$ git log fd8b15ef1c8398a69a37932ee48c74ab40329a29..f971e0cd4a0ebe59fb2e8e17240399bf6901b09b
commit f971e0cd4a0ebe59fb2e8e17240399bf6901b09b
Author: Derald D. Woods <woods.technical@gmail.com>
Date: Sun Feb 10 13:22:19 2019 -0600
Fix module loading with Linux v5.0-rc5
This commit fixes this module load error:
[...]
[ 29.112091] cryptodev: loading out-of-tree module taints kernel.
[ 29.128906] cryptodev: Unknown symbol crypto_givcipher_type (err -2)
[ 29.188842] cryptodev: Unknown symbol crypto_givcipher_type (err -2)
modprobe: can't load module cryptodev (extra/cryptodev.ko): unknown symbol in module, or unknown parameter
[...]
Upstream Linux support for unused GIVCIPHER, and others, was dropped here:
c79b411eaa72 (crypto: skcipher - remove remnants of internal IV generators)
Signed-off-by: Derald D. Woods <woods.technical@gmail.com>
Signed-off-by: Kai Kang <kai.kang@windriver.com>
---
meta/recipes-kernel/cryptodev/cryptodev.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meta/recipes-kernel/cryptodev/cryptodev.inc b/meta/recipes-kernel/cryptodev/cryptodev.inc
index 252d39d23a..79ddefd435 100644
--- a/meta/recipes-kernel/cryptodev/cryptodev.inc
+++ b/meta/recipes-kernel/cryptodev/cryptodev.inc
@@ -4,7 +4,7 @@ LICENSE = "GPLv2"
LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
SRC_URI = "git://github.com/cryptodev-linux/cryptodev-linux"
-SRCREV = "fd8b15ef1c8398a69a37932ee48c74ab40329a29"
+SRCREV = "f971e0cd4a0ebe59fb2e8e17240399bf6901b09b"
S = "${WORKDIR}/git"
--
2.20.0
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] cryptodev: update SRCREV
2019-04-15 3:42 [PATCH] cryptodev: update SRCREV kai.kang
@ 2019-04-15 7:38 ` Adrian Bunk
2019-04-24 2:23 ` Kang Kai
1 sibling, 0 replies; 5+ messages in thread
From: Adrian Bunk @ 2019-04-15 7:38 UTC (permalink / raw)
To: kai.kang; +Cc: openembedded-core
On Sun, Apr 14, 2019 at 11:42:31PM -0400, kai.kang@windriver.com wrote:
> From: Kai Kang <kai.kang@windriver.com>
>
> Update SRCREV of cryptodev which only contains one fix for linux 5.0:
>...
> --- a/meta/recipes-kernel/cryptodev/cryptodev.inc
> +++ b/meta/recipes-kernel/cryptodev/cryptodev.inc
> @@ -4,7 +4,7 @@ LICENSE = "GPLv2"
> LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
>
> SRC_URI = "git://github.com/cryptodev-linux/cryptodev-linux"
> -SRCREV = "fd8b15ef1c8398a69a37932ee48c74ab40329a29"
> +SRCREV = "f971e0cd4a0ebe59fb2e8e17240399bf6901b09b"
>
> S = "${WORKDIR}/git"
This moves cryptodev from a release to a git snapshot,
but no PV is set to show that.
It is a systematic problem caused by the switch to git hashes for
releases that random git snapshots look exactly the same as releases
without any clean way to see what a commit actually is.
IMHO this needs support for tags, and then checks like (pseudo-code)
if(TAG) (commit(TAG) == SRCREV)
TAG xor (PV contains "+git")
Short-term for the suggested change in cryptodev the
"which only contains one fix" is correct, and this
is not much different from adding the fix as .patch
which also wouldn't change PV.
So no objection from me to applying the suggested patch as-is.
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] cryptodev: update SRCREV
2019-04-15 3:42 [PATCH] cryptodev: update SRCREV kai.kang
2019-04-15 7:38 ` Adrian Bunk
@ 2019-04-24 2:23 ` Kang Kai
2019-04-24 12:38 ` Burton, Ross
1 sibling, 1 reply; 5+ messages in thread
From: Kang Kai @ 2019-04-24 2:23 UTC (permalink / raw)
To: openembedded-core
On 2019/4/15 上午11:42, kai.kang@windriver.com wrote:
> From: Kai Kang <kai.kang@windriver.com>
>
> Update SRCREV of cryptodev which only contains one fix for linux 5.0:
>
> $ git log fd8b15ef1c8398a69a37932ee48c74ab40329a29..f971e0cd4a0ebe59fb2e8e17240399bf6901b09b
> commit f971e0cd4a0ebe59fb2e8e17240399bf6901b09b
> Author: Derald D. Woods <woods.technical@gmail.com>
> Date: Sun Feb 10 13:22:19 2019 -0600
>
> Fix module loading with Linux v5.0-rc5
>
> This commit fixes this module load error:
> [...]
> [ 29.112091] cryptodev: loading out-of-tree module taints kernel.
> [ 29.128906] cryptodev: Unknown symbol crypto_givcipher_type (err -2)
> [ 29.188842] cryptodev: Unknown symbol crypto_givcipher_type (err -2)
> modprobe: can't load module cryptodev (extra/cryptodev.ko): unknown symbol in module, or unknown parameter
> [...]
Without this patch it fails to modprobe module cryptodev. Ping.
Kai
>
> Upstream Linux support for unused GIVCIPHER, and others, was dropped here:
>
> c79b411eaa72 (crypto: skcipher - remove remnants of internal IV generators)
>
> Signed-off-by: Derald D. Woods <woods.technical@gmail.com>
>
> Signed-off-by: Kai Kang <kai.kang@windriver.com>
> ---
> meta/recipes-kernel/cryptodev/cryptodev.inc | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/meta/recipes-kernel/cryptodev/cryptodev.inc b/meta/recipes-kernel/cryptodev/cryptodev.inc
> index 252d39d23a..79ddefd435 100644
> --- a/meta/recipes-kernel/cryptodev/cryptodev.inc
> +++ b/meta/recipes-kernel/cryptodev/cryptodev.inc
> @@ -4,7 +4,7 @@ LICENSE = "GPLv2"
> LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
>
> SRC_URI = "git://github.com/cryptodev-linux/cryptodev-linux"
> -SRCREV = "fd8b15ef1c8398a69a37932ee48c74ab40329a29"
> +SRCREV = "f971e0cd4a0ebe59fb2e8e17240399bf6901b09b"
>
> S = "${WORKDIR}/git"
>
--
Kai Kang
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] cryptodev: update SRCREV
2019-04-24 2:23 ` Kang Kai
@ 2019-04-24 12:38 ` Burton, Ross
2019-04-25 1:36 ` Kang Kai
0 siblings, 1 reply; 5+ messages in thread
From: Burton, Ross @ 2019-04-24 12:38 UTC (permalink / raw)
To: Kang Kai; +Cc: OE-core
I'd prefer this to be applying the patch, which also makes the fix
easier to backport.
Ross
On Wed, 24 Apr 2019 at 03:26, Kang Kai <Kai.Kang@windriver.com> wrote:
>
> On 2019/4/15 上午11:42, kai.kang@windriver.com wrote:
> > From: Kai Kang <kai.kang@windriver.com>
> >
> > Update SRCREV of cryptodev which only contains one fix for linux 5.0:
> >
> > $ git log fd8b15ef1c8398a69a37932ee48c74ab40329a29..f971e0cd4a0ebe59fb2e8e17240399bf6901b09b
> > commit f971e0cd4a0ebe59fb2e8e17240399bf6901b09b
> > Author: Derald D. Woods <woods.technical@gmail.com>
> > Date: Sun Feb 10 13:22:19 2019 -0600
> >
> > Fix module loading with Linux v5.0-rc5
> >
> > This commit fixes this module load error:
> > [...]
> > [ 29.112091] cryptodev: loading out-of-tree module taints kernel.
> > [ 29.128906] cryptodev: Unknown symbol crypto_givcipher_type (err -2)
> > [ 29.188842] cryptodev: Unknown symbol crypto_givcipher_type (err -2)
> > modprobe: can't load module cryptodev (extra/cryptodev.ko): unknown symbol in module, or unknown parameter
> > [...]
>
>
> Without this patch it fails to modprobe module cryptodev. Ping.
>
> Kai
>
>
> >
> > Upstream Linux support for unused GIVCIPHER, and others, was dropped here:
> >
> > c79b411eaa72 (crypto: skcipher - remove remnants of internal IV generators)
> >
> > Signed-off-by: Derald D. Woods <woods.technical@gmail.com>
> >
> > Signed-off-by: Kai Kang <kai.kang@windriver.com>
> > ---
> > meta/recipes-kernel/cryptodev/cryptodev.inc | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/meta/recipes-kernel/cryptodev/cryptodev.inc b/meta/recipes-kernel/cryptodev/cryptodev.inc
> > index 252d39d23a..79ddefd435 100644
> > --- a/meta/recipes-kernel/cryptodev/cryptodev.inc
> > +++ b/meta/recipes-kernel/cryptodev/cryptodev.inc
> > @@ -4,7 +4,7 @@ LICENSE = "GPLv2"
> > LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> >
> > SRC_URI = "git://github.com/cryptodev-linux/cryptodev-linux"
> > -SRCREV = "fd8b15ef1c8398a69a37932ee48c74ab40329a29"
> > +SRCREV = "f971e0cd4a0ebe59fb2e8e17240399bf6901b09b"
> >
> > S = "${WORKDIR}/git"
> >
>
>
> --
> Kai Kang
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] cryptodev: update SRCREV
2019-04-24 12:38 ` Burton, Ross
@ 2019-04-25 1:36 ` Kang Kai
0 siblings, 0 replies; 5+ messages in thread
From: Kang Kai @ 2019-04-25 1:36 UTC (permalink / raw)
To: Burton, Ross; +Cc: OE-core
On 2019/4/24 下午8:38, Burton, Ross wrote:
> I'd prefer this to be applying the patch, which also makes the fix
> easier to backport.
OK. I'll send v2 to apply the the patch.
Regards,
Kai
>
> Ross
>
> On Wed, 24 Apr 2019 at 03:26, Kang Kai <Kai.Kang@windriver.com> wrote:
>> On 2019/4/15 上午11:42, kai.kang@windriver.com wrote:
>>> From: Kai Kang <kai.kang@windriver.com>
>>>
>>> Update SRCREV of cryptodev which only contains one fix for linux 5.0:
>>>
>>> $ git log fd8b15ef1c8398a69a37932ee48c74ab40329a29..f971e0cd4a0ebe59fb2e8e17240399bf6901b09b
>>> commit f971e0cd4a0ebe59fb2e8e17240399bf6901b09b
>>> Author: Derald D. Woods <woods.technical@gmail.com>
>>> Date: Sun Feb 10 13:22:19 2019 -0600
>>>
>>> Fix module loading with Linux v5.0-rc5
>>>
>>> This commit fixes this module load error:
>>> [...]
>>> [ 29.112091] cryptodev: loading out-of-tree module taints kernel.
>>> [ 29.128906] cryptodev: Unknown symbol crypto_givcipher_type (err -2)
>>> [ 29.188842] cryptodev: Unknown symbol crypto_givcipher_type (err -2)
>>> modprobe: can't load module cryptodev (extra/cryptodev.ko): unknown symbol in module, or unknown parameter
>>> [...]
>>
>> Without this patch it fails to modprobe module cryptodev. Ping.
>>
>> Kai
>>
>>
>>> Upstream Linux support for unused GIVCIPHER, and others, was dropped here:
>>>
>>> c79b411eaa72 (crypto: skcipher - remove remnants of internal IV generators)
>>>
>>> Signed-off-by: Derald D. Woods <woods.technical@gmail.com>
>>>
>>> Signed-off-by: Kai Kang <kai.kang@windriver.com>
>>> ---
>>> meta/recipes-kernel/cryptodev/cryptodev.inc | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/meta/recipes-kernel/cryptodev/cryptodev.inc b/meta/recipes-kernel/cryptodev/cryptodev.inc
>>> index 252d39d23a..79ddefd435 100644
>>> --- a/meta/recipes-kernel/cryptodev/cryptodev.inc
>>> +++ b/meta/recipes-kernel/cryptodev/cryptodev.inc
>>> @@ -4,7 +4,7 @@ LICENSE = "GPLv2"
>>> LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
>>>
>>> SRC_URI = "git://github.com/cryptodev-linux/cryptodev-linux"
>>> -SRCREV = "fd8b15ef1c8398a69a37932ee48c74ab40329a29"
>>> +SRCREV = "f971e0cd4a0ebe59fb2e8e17240399bf6901b09b"
>>>
>>> S = "${WORKDIR}/git"
>>>
>>
>> --
>> Kai Kang
>>
>> --
>> _______________________________________________
>> Openembedded-core mailing list
>> Openembedded-core@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-core
--
Kai Kang
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-04-25 1:39 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-04-15 3:42 [PATCH] cryptodev: update SRCREV kai.kang
2019-04-15 7:38 ` Adrian Bunk
2019-04-24 2:23 ` Kang Kai
2019-04-24 12:38 ` Burton, Ross
2019-04-25 1:36 ` Kang Kai
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox