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: <ZTsqROr8s18aWwSY@boqun-archlinux> Date: Thu, 26 Oct 2023 20:11:00 -0700 From: Boqun Feng <boqun.feng@...il.com> To: Andrew Lunn <andrew@...n.ch> Cc: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>, FUJITA Tomonori <fujita.tomonori@...il.com>, netdev@...r.kernel.org, rust-for-linux@...r.kernel.org, tmgross@...ch.edu, benno.lossin@...ton.me, wedsonaf@...il.com Subject: Re: [PATCH net-next v7 0/5] Rust abstractions for network PHY drivers On Fri, Oct 27, 2023 at 04:47:17AM +0200, Andrew Lunn wrote: > > I wonder whether that actually helps, if a reviewer takes average four > > days to review a version (wants to give accurate comments and doesn't > > work on this full time), and the developer send a new version every > > three days, there is no possible way for the developer to get the > > reviews. > > > > (Honestly, if people could reach out to a conclusion for anything in > > three days, the world would be a much more peaceful place ;-)) > > May i suggest you subscribe to the netdev list and watch it in action. > I'm sorry, I wasn't questioning about the process of netdev, I respect the hard work behind that. I was simply wondering whether sending out a new version so quickly is a good way to kick reviewers... it's sometimes frustating to see new version post but old comments were not resolved, it rather discourages reviewers.. > It should also be noted, patches don't need reviews to be merged. If > there is no feedback within three days, and it passes the CI tests, it Do the CI tests support Rust now? Does Tomo's patch pass CI? Looks like something we'd like to see (and help). > likely will be merged. Real problems can be fixed up later, if need > be. But this doesn't apply to pure API, right? So if some one post a pure Rust API with no user, but some tests, and the CI passes, the API won't get merged? Even though no review is fine and if API has problems, we can fix it later? Regards, Boqun > > Andrew >
Powered by blists - more mailing lists