On Sat, 2017-08-05 at 07:49 +0200, Borislav Petkov wrote: > On Fri, Aug 04, 2017 at 09:48:23PM +0000, Kani, Toshimitsu wrote: > > Not sure if anyone cares, but I thought it should return with > > -ENODEV when this modules found no target, and -EBUSY when it found > > a target but it's busy. Hence, this ordering. > > You can still return -EBUSY. Just do the owner check first because if > we're going to return due to ghes_edac being loaded already, we might > just as well skip the other checks as they're a pointless waste of > cycles and energy. Got it. Will move up the check. Thanks, -Toshi ��.n��������+%������w��{.n�����{�����ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f