lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20220302103313.3bacd10b@gandalf.local.home> Date: Wed, 2 Mar 2022 10:33:13 -0500 From: Steven Rostedt <rostedt@...dmis.org> To: Greg KH <gregkh@...uxfoundation.org> Cc: Jithu Joseph <jithu.joseph@...el.com>, hdegoede@...hat.com, markgross@...nel.org, tglx@...utronix.de, mingo@...hat.com, bp@...en8.de, dave.hansen@...ux.intel.com, x86@...nel.org, hpa@...or.com, corbet@....net, andriy.shevchenko@...ux.intel.com, ashok.raj@...el.com, tony.luck@...el.com, linux-kernel@...r.kernel.org, linux-doc@...r.kernel.org, platform-driver-x86@...r.kernel.org, patches@...ts.linux.dev, ravi.v.shankar@...el.com Subject: Re: [RFC 00/10] Introduce In Field Scan driver On Tue, 1 Mar 2022 21:10:20 +0100 Greg KH <gregkh@...uxfoundation.org> wrote: > "RFC" means you are not comfortable submitting the changes yet, so you > don't need my review at this point in time. Become confident in your > changes before asking for others to review the code please. I guess you and I have a different understanding of RFC (Request for Comments). As to me, comments are a form of review. In other words, RFC to me means the review is "does this design look like it will work", and we should be reviewing the design and overview of the patches. Not the nitty gritty details (like missed error handling, unless the design will prevent it). Although, you could add those comments in a review. When I post RFCs, it's not that I'm not comfortable submitting the change, it's because I want to know if what I'm doing makes sense, and I might be missing something that will make this effort in vain. What ever happen to the "Submit early, submit often" mantra? -- Steve
Powered by blists - more mailing lists