On Fri, Nov 10, 2017 at 3:38 AM, Andy Lutomirski <luto@xxxxxxxxxx> wrote: > On Thu, Nov 9, 2017 at 8:14 AM, Djalal Harouni <tixxdz@xxxxxxxxx> wrote: >> This patch introduces the new 'pids' mount option, as it was discussed >> and suggested by Andy Lutomirski [1]. >> >> * If 'pids=' is passed without 'newinstance' then it has no effect. > > Would it be safer this were an error instead? Hm, I tend to say that you are right, but I also keep your comment when you said that "newinstance" should be the default later and users won't have to explicitly pass it. What you think ? -- tixxdz