[Cluster-devel] [PATCH 2/3] mkfs: Remove unneeded open/close fd test from are_you_sure()

Bob Peterson rpeterso at redhat.com
Tue Sep 27 16:37:32 UTC 2011

----- Original Message -----
| are_you_sure() function should not make an open/close
| test on the underlying device. The pourpose of this
| function is just to ask a yes/no question to the user.
| If we need a function to do this test, we should create
| a new function with its specific pourpose.
| This patch also fix a call to are_you_sure() function and
| move the are_you_sure() function to the begining of the
| file, allowing it to be used in another part of the code
| Signed-off-by: Carlos Maiolino <cmaiolino at redhat.com>


This all looks good and makes sense.  I am concerned
that removing the open/close might change the behavior
when errors occur: For example, if you try to mkfs.gfs2
for a device you don't have permission to.  Or for a
device that doesn't exist.  Please test those two cases
to make sure nothing weird happens.


Bob Peterson
Red Hat File Systems

More information about the Cluster-devel mailing list