Messages in this thread | | | From | Logan Gunthorpe <> | Date | Thu, 11 Jun 2020 17:14:52 -0600 | Subject | Re: [PATCH v13 8/9] nvmet-passthru: Add enable/disable helpers |
| |
On 2020-06-11 5:05 p.m., Chaitanya Kulkarni wrote: > On 5/14/20 10:23 AM, Logan Gunthorpe wrote: >> + if (subsys->nr_namespaces) { >> + pr_info("cannot enable both passthru and regular namespaces for a single subsystem"); > > Let's try and keep the error message witin 80 char per line or split the > message into two pr_info() calls,how about this ? > > > pr_info("cannot enable passthru & regular namespaces\n")
Honestly, I think that is too brief. The error message is only 74 chars and there's a long standing exception for long lines in the kernel for printks. Even Linus has recently suggested that keeping to the 80 char limit is not recommended when it harms readability (though I don't generally agree with this 100%)[1].
Logan
[1] https://lwn.net/ml/linux-kernel/CAHk-=wjR0H3+2ba0UUWwoYzYBH0GX9yTf5dj2MZyo0xvyzvJnA@mail.gmail.com/
| |