On Thu, Sep 07, 2017 at 05:11:45AM +0000, Okamoto, Takayuki wrote: > Hi Dave, > > Thank you for your reply. > > > Your fix looks correct and seems to work. For stylistic reasons, I may > > write it like this instead, but the effect should be the same: > > > > header->max_vl = sve_max_vl; > > if (WARN_ON(!sve_vl_valid(sve_max_vl)) > > header->max_vl = header->vl; > > It is better than my fix. > Please, apply it at next version. I've rebased to v4.13 and pushed a branch to track fixes against v2, here: * http://linux-arm.org/git?p=linux-dm.git;a=shortlog;h=refs/heads/sve/v2%2Bfixes * git://linux-arm.org/linux-dm.git sve/v2+fixes Cheers ---Dave
