From: sandeen@sandeen.net
To: xfs@oss.sgi.com
Cc: hch@infradead.org
Subject: [patch 00/11] RFC: compat ioctl fixes/cleanups/additions
Date: Tue, 18 Nov 2008 22:44:01 -0600 [thread overview]
Message-ID: <20081119044401.573365619@sandeen.net> (raw)
This should properly hook up all ioctls which need compat
handlers on intel or ppc; it passes the auto qa group on intel,
and does ok on ppc (there are still issues with xfsdump itself
if the ppc page size is large (64k in my case))
This might be a little overboard; I'm half tempted to remove the
handlers that only intel boxes need; would anyone be running 32-bit
apps calling xfs ioctls on x86_64 or ia64? Seems unlikely; and
removing them would cut down the size of this patchset:
linux-2.6/xfs_ioctl.c | 6
linux-2.6/xfs_ioctl32.c | 651 +++++++++++++++++++++++++++++++++---------------
linux-2.6/xfs_ioctl32.h | 209 +++++++++++++++
xfs_fs.h | 6
xfs_itable.c | 45 ++-
xfs_itable.h | 14 +
6 files changed, 716 insertions(+), 215 deletions(-)
If the sgi guys can toss this into qa, I'd be much obliged. I
hope to have some xfsdump fixes at some point so we can actually
see most things pass (if your ppc box has < 64k pages, it might
work fine for you already...)
Thanks,
-Eric
--
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
next reply other threads:[~2008-11-19 5:16 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-19 4:44 sandeen [this message]
2008-11-19 4:44 ` [patch 01/11] Move compat ioctl structs & numbers into xfs_ioctl32.h sandeen
[not found] ` <20081119145941.GA13050@infradead.org>
2008-11-19 15:24 ` Eric Sandeen
2008-11-20 1:54 ` Eric Sandeen
2008-11-19 4:44 ` [patch 02/11] Fix the compat XFS_IOC_FSGEOMETRY_V1 ioctl sandeen
2008-11-19 4:44 ` [patch 03/11] Add compat handlers for data & rt growfs ioctls sandeen
2008-11-19 4:44 ` [patch 04/11] Add compat handlers for swapext ioctl sandeen
2008-11-19 4:44 ` [patch 05/11] Make the bulkstat_one compat ioctl handling more sane sandeen
2008-11-19 4:44 ` [patch 06/11] Fix xfs_bulkstat_one size checks & error handling sandeen
2008-11-19 4:44 ` [patch 07/11] Fix compat XFS_IOC_FSBULKSTAT_SINGLE ioctl sandeen
2008-11-19 4:44 ` [patch 08/11] Hook up compat XFS_IOC_ATTRLIST_BY_HANDLE ioctl handler sandeen
2008-11-19 4:44 ` [patch 09/11] Hook up compat XFS_IOC_ATTRMULTI_BY_HANDLE " sandeen
[not found] ` <20081119151628.GI13050@infradead.org>
2008-11-19 15:30 ` Eric Sandeen
2008-11-19 4:44 ` [patch 10/11] Hook up compat XFS_IOC_FSSETDM_BY_HANDLE " sandeen
2008-11-19 4:44 ` [patch 11/11] Reorder xfs_ioctl32.c for some tidiness sandeen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20081119044401.573365619@sandeen.net \
--to=sandeen@sandeen.net \
--cc=hch@infradead.org \
--cc=xfs@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox