5786 2014-02-04 12:26:32 +0000 module_autoload_xxx variables are not taken into account for sstate signature 2014-12-26 14:41:07 +0000 1 1 7 Build System, Metadata & Runtime OE-Core kernel 1.5 x86 Multiple RESOLVED FIXED Medium normal 1.7 1 ndec13 richard.purdie bluelightning bruce.ashfield dvhart richard.purdie tom.zanussi --- --- Don't know oldest_to_newest 40176 0 ndec13 2014-02-04 12:26:32 +0000 hi, it seems that module_autoload variables are not included in the kernel recipes. Here is what I did: add the following in linux-yocto_3.10.bbappend in my layer module_autoload_snd-intel8x0 = "snd-intel8x0" that does not trigger a rebuild (repackaging), and my change is ignored. Cleaning the sstate and rebuild, obviously, work. thanks 40178 1 bluelightning 2014-02-04 13:41:37 +0000 Just to clarify, the problem is that the module_autoload_* variables don't get included in the task hash for do_package as they ought to - because of the way they are read, bitbake can't determine that a dependency on them exists; thus adding one or changing one of their values and then building the kernel or image doesn't have any effect. Because the name is dynamic I'm not sure how we can fix this. Richard, any suggestions? 40198 2 dvhart 2014-02-05 16:42:55 +0000 As this is more bitbake-core related that kernel specific - I'm reassigning to Saul. 41726 3 richard.purdie 2014-03-25 21:55:40 +0000 I don't know how to fix this for the current codebase. The format of this variable (and module_conf_xxxx) are horrible. What we need is a: KERNEL_MODULE_AUTOLOAD = "<list oof modules>" KERNEL_MODULE_PROBECONF = "<list of modules>" set of variables which replace the module_autoload ones and supplement the module_conf ones. Unfortunately this is an API change, probably too late for this cycle but can make it early in the next one. Once we have the above variables, adding those to sstate is straight forward. 41727 4 1860 richard.purdie 2014-03-25 22:06:37 +0000 Created attachment 1860 Rough proof of concept Rough proof of concept 43948 5 richard.purdie 2014-06-13 16:14:42 +0000 Patch finished and posted for review 44017 6 richard.purdie 2014-06-17 10:32:55 +0000 Resolved by http://git.yoctoproject.org/cgit.cgi/poky/commit/?id=c56c9a2f41eac8f87f13547806edc794b77ba54e 1860 2014-03-25 22:06:37 +0000 2014-03-25 22:06:37 +0000 Rough proof of concept 1 text/plain 2103 richard.purdie RnJvbSBhZDc3MDc0NWM4ZjBhNGY2NDRjZWY4YWI4MWQ0YzIyZGNjNmNiOGUwIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBSaWNoYXJkIFB1cmRpZSA8cmljaGFyZC5wdXJkaWVAbGludXhm b3VuZGF0aW9uLm9yZz4KRGF0ZTogVHVlLCAyNSBNYXIgMjAxNCAyMjowNToxNSArMDAwMApTdWJq ZWN0OiBrZXJuZWwtbW9kdWxlLXNwbGl0OiBBZGQgc3VwcG9ydCBmb3IgS0VSTkVMX01PRFVMRV9B VVRPTE9BRCBhbmQgS0VSTkVMX01PRFVMRV9QUk9CRUNPTkYKClNpZ25lZC1vZmYtYnk6IFJpY2hh cmQgUHVyZGllIDxyaWNoYXJkLnB1cmRpZUBsaW51eGZvdW5kYXRpb24ub3JnPgotLS0KZGlmZiAt LWdpdCBhL21ldGEvY2xhc3Nlcy9rZXJuZWwtbW9kdWxlLXNwbGl0LmJiY2xhc3MgYi9tZXRhL2Ns YXNzZXMva2VybmVsLW1vZHVsZS1zcGxpdC5iYmNsYXNzCmluZGV4IGQ0M2Y3NDMuLjEzNjNmYjIg MTAwNjQ0Ci0tLSBhL21ldGEvY2xhc3Nlcy9rZXJuZWwtbW9kdWxlLXNwbGl0LmJiY2xhc3MKKysr IGIvbWV0YS9jbGFzc2VzL2tlcm5lbC1tb2R1bGUtc3BsaXQuYmJjbGFzcwpAQCAtMTMwLDggKzEz MCwxMSBAQCBweXRob24gc3BsaXRfa2VybmVsX21vZHVsZV9wYWNrYWdlcyAoKSB7CiAKICAgICAg ICAgIyBJZiBhdXRvbG9hZGluZyBpcyByZXF1ZXN0ZWQsIG91dHB1dCAvZXRjL21vZHVsZXMtbG9h ZC5kLzxuYW1lPi5jb25mIGFuZCBhcHBlbmQKICAgICAgICAgIyBhcHByb3ByaWF0ZSBtb2Rwcm9i ZSBjb21tYW5kcyB0byB0aGUgcG9zdGluc3QKKyAgICAgICAgYXV0b2xvYWRsaXN0ID0gKGQuZ2V0 VmFyKCJLRVJORUxfTU9EVUxFX0FVVE9MT0FEIiwgVHJ1ZSkgb3IgIiIpLnNwbGl0KCkKICAgICAg ICAgYXV0b2xvYWQgPSBkLmdldFZhcignbW9kdWxlX2F1dG9sb2FkXyVzJyAlIGJhc2VuYW1lLCBU cnVlKQogICAgICAgICBpZiBhdXRvbG9hZDoKKyAgICAgICAgICAgIGJiLmVycm9yKCJLRVJORUxf TU9EVUxFX0FVVE9MT0FEIGhhcyByZXBsYWNlZCBtb2R1bGVfYXV0b2xvYWRfJXMsIHBsZWFzZSBy ZXBsYWNlIGl0ISIgJSBiYXNlbmFtZSkKKyAgICAgICAgaWYgYmFzZW5hbWUgaW4gYXV0b2xvYWRs aXN0OgogICAgICAgICAgICAgbmFtZSA9ICclcy9ldGMvbW9kdWxlcy1sb2FkLmQvJXMuY29uZicg JSAoZHZhciwgYmFzZW5hbWUpCiAgICAgICAgICAgICBmID0gb3BlbihuYW1lLCAndycpCiAgICAg ICAgICAgICBmb3IgbSBpbiBhdXRvbG9hZC5zcGxpdCgpOgpAQCAtMTQ0LDEyICsxNDcsMTUgQEAg cHl0aG9uIHNwbGl0X2tlcm5lbF9tb2R1bGVfcGFja2FnZXMgKCkgewogICAgICAgICAgICAgZC5z ZXRWYXIoJ3BrZ19wb3N0aW5zdF8lcycgJSBwa2csIHBvc3RpbnN0KQogCiAgICAgICAgICMgV3Jp dGUgb3V0IGFueSBtb2Rjb25mIGZyYWdtZW50CisgICAgICAgIG1vZGNvbmZsaXN0ID0gKGQuZ2V0 VmFyKCJLRVJORUxfTU9EVUxFX1BST0JFQ09ORiIsIFRydWUpIG9yICIiKS5zcGxpdCgpCiAgICAg ICAgIG1vZGNvbmYgPSBkLmdldFZhcignbW9kdWxlX2NvbmZfJXMnICUgYmFzZW5hbWUsIFRydWUp Ci0gICAgICAgIGlmIG1vZGNvbmY6CisgICAgICAgIGlmIG1vZGNvbmYgYW5kIGJhc2VuYW1lIGlu IG1vZGNvbmZsaXN0OgogICAgICAgICAgICAgbmFtZSA9ICclcy9ldGMvbW9kcHJvYmUuZC8lcy5j b25mJyAlIChkdmFyLCBiYXNlbmFtZSkKICAgICAgICAgICAgIGYgPSBvcGVuKG5hbWUsICd3JykK ICAgICAgICAgICAgIGYud3JpdGUoIiVzXG4iICUgbW9kY29uZikKICAgICAgICAgICAgIGYuY2xv c2UoKQorICAgICAgICBlbGlmIG1vZGNvbmY6CisgICAgICAgICAgICBiYi5lcnJvcigiUGxlYXNl IGVuc3VyZSBtb2R1bGUgJXMgaXMgbGlzdGVkIGluIEtFUk5FTF9NT0RVTEVfUFJPQkVDT05GIHNp bmNlIG1vZHVsZV9jb25mXyVzIGlzIHNldCIgJSAoYmFzZW5hbWUsIGJhc2VuYW1lKSkKIAogICAg ICAgICBmaWxlcyA9IGQuZ2V0VmFyKCdGSUxFU18lcycgJSBwa2csIFRydWUpCiAgICAgICAgIGZp bGVzID0gIiVzIC9ldGMvbW9kdWxlcy1sb2FkLmQvJXMuY29uZiAvZXRjL21vZHByb2JlLmQvJXMu Y29uZiIgJSAoZmlsZXMsIGJhc2VuYW1lLCBiYXNlbmFtZSkKLS0KY2dpdCB2MC45LjEK