On Mon, Dec 26, 2016 at 05:26:19PM -0500, David Miller wrote: > From: Anatoly Pugachev <matorola@xxxxxxxxx> > Date: Sun, 25 Dec 2016 20:56:08 +0300 > > > Disabling kernel config option > > CRYPTO_MANAGER_DISABLE_TESTS > > i.e. enable run-time self tests, makes kernel unbootable: > > > > tested with git kernels v4.9-8648-g5cc60aeedf31 and v4.9-12259-g7c0f6ba682b9 > > I think the testing code for the new synchronous compression module is > putting kernel image pointers into scatterlists, which in turn we > attempt to transform to and from page structs. > > That doesn't work. > > It's coming from the test input buffers: > > static int test_acomp(struct crypto_acomp *tfm, struct comp_testvec *ctemplate, > struct comp_testvec *dtemplate, int ctcount, int dtcount) > { > ... > sg_init_one(&src, ctemplate[i].input, ilen); > > These have to be copied into kmalloc() buffers or similar, just like > the skchiper tests do. > > The crash on sparc64 shows that we try to dereference a page struct at > a bogus vmemmap address for a page that doesn't exist. > > I hacked up the following and this makes the crashes go away: Thanks Dave. I've just applied the patch https://patchwork.kernel.org/patch/9483763/ which should fix this. Cheers, -- Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -- To unsubscribe from this list: send the line "unsubscribe linux-crypto" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html