On 8/21/24 18:15, Al Viro wrote:
On Wed, Aug 21, 2024 at 12:38:48AM -0600, Shuah Khan wrote:
On 8/13/24 19:03, Al Viro wrote:
On Tue, Aug 13, 2024 at 07:16:00PM +0100, Al Viro wrote:
On Tue, Aug 13, 2024 at 11:00:04AM +0300, Dan Carpenter wrote:
3f4b0acefd818e Al Viro 2024-08-06 3240 if (IS_ERR(*new_fdp)) {
3f4b0acefd818e Al Viro 2024-08-06 3241 *new_fdp = NULL;
3f4b0acefd818e Al Viro 2024-08-06 @3242 return PTR_ERR(new_fdp);
^^^^^^^^^^^^^^^^
err = PTR_ERR(*new_fdp);
*new_fdp = NULL;
return err;
Argh... Obvious braino, but what it shows is that failures of that
thing are not covered by anything in e.g. LTP. Or in-kernel
self-tests, for that matter...
FWIW, this does exercise that codepath, but I would really like to
have kselftest folks to comment on the damn thing - I'm pretty sure
that it's _not_ a good style for those.
Looks good to me. Would you be able to send a patch for this new test?
Umm... Send as in "mail somewhere specific", or as "push into vfs.git",
or...?
If you can send the patch to kselftest mailing test, I can take this through
my tree.
thanks,
-- Shuah