[PATCH 3/3] Add btrfs ioctl support.
Jeff Mahoney
jeffm at suse.com
Thu Mar 31 13:49:38 UTC 2016
On 3/31/16 12:34 AM, Jeff Mahoney wrote:
> +#include "xlat/btrfs_dev_replace_args.h"
This file was renamed and doesn't exist in a clean build, so I'll at
least have to clean that up.
I'm also working up test cases for it. One hiccup is that I've used the
<name> <value> functionality in the xlat/*.in files to fill in #defines
that aren't exported in the version of btrfs.h we're building with.
I'd like to just include the xlat/btrfs_*.h files in my test program so
we don't need to define them multiple times. The problem there is that
the .h files also define the xlat structures, which I don't actually
need, and then cause build warnings because they're unused. Do you have
a preference on how I handle this? I see 3 options:
1/ Define the macros in multiple places
2/ Teach xlat/gen.sh to produce two .h files. One contains the xlat
structures, the other contains the defines.
3/ Teach xlat/gen.sh to wrap the structure definition in a "#ifndef
XLAT_DEFINES_ONLY" and allow test programs to define that.
My preference is #3 since it means no code change outside of the
xlat/gen.sh script.
Thanks,
-Jeff
--
Jeff Mahoney
SUSE Labs
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 881 bytes
Desc: OpenPGP digital signature
URL: <http://lists.strace.io/pipermail/strace-devel/attachments/20160331/16a33bfd/attachment.bin>
More information about the Strace-devel
mailing list