[RFC] strace patches for new ioctls

[RFC] strace patches for new ioctls

[Jeff Mahoney]

[-- Attachment #1.1: Type: text/plain, Size: 536 bytes --]

Hi all -

Strace 4.12 was tagged for release today and it supports decoding of
btrfs ioctls.  I'd like to propose a requirement that future ioctl
additions come with a patch to strace as well so we don't get out of sync.

I wrote the decoding to help with some issues some of our developers
were seeing while writing various tools.  Rather than describing their
workload or showing us the code, an strace in verbose mode shows us
pretty much exactly what they're doing.

Opinions?

-Jeff

-- 
Jeff Mahoney
SUSE Labs


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 881 bytes --]

Re: [RFC] strace patches for new ioctls

[Chris Mason]

On Tue, May 31, 2016 at 04:41:55PM -0400, Jeff Mahoney wrote:
> Hi all -
> 
> Strace 4.12 was tagged for release today and it supports decoding of
> btrfs ioctls.  I'd like to propose a requirement that future ioctl
> additions come with a patch to strace as well so we don't get out of sync.
> 
> I wrote the decoding to help with some issues some of our developers
> were seeing while writing various tools.  Rather than describing their
> workload or showing us the code, an strace in verbose mode shows us
> pretty much exactly what they're doing.
> 
> Opinions?

Maybe add a link to the wiki of your last set of patches to use as a
template?  It's a great idea.

-chris

Re: [RFC] strace patches for new ioctls

[David Sterba]

On Tue, May 31, 2016 at 04:51:03PM -0400, Chris Mason wrote:
> On Tue, May 31, 2016 at 04:41:55PM -0400, Jeff Mahoney wrote:
> > Hi all -
> > 
> > Strace 4.12 was tagged for release today and it supports decoding of
> > btrfs ioctls.  I'd like to propose a requirement that future ioctl
> > additions come with a patch to strace as well so we don't get out of sync.
> > 
> > I wrote the decoding to help with some issues some of our developers
> > were seeing while writing various tools.  Rather than describing their
> > workload or showing us the code, an strace in verbose mode shows us
> > pretty much exactly what they're doing.
> > 
> > Opinions?
> 
> Maybe add a link to the wiki of your last set of patches to use as a
> template?  It's a great idea.

[ I've replied to that but the mail got lost somewhere ]

There's a new wiki page to track the notes, checklists and hints for
such things.

https://btrfs.wiki.kernel.org/index.php/Development_notes