Commit 236c64e4 authored by Eric W. Biederman's avatar Eric W. Biederman

gfs2: Remove the QUOTA_USER and QUOTA_GROUP defines

Remove the QUOTA_USER and QUOTA_GRUP defines.  Remove
the last vestigal users of QUOTA_USER and QUOTA_GROUP.

Now that struct kqid is used throughout the gfs2 quota
code the need there is to use QUOTA_USER and QUOTA_GROUP
and the defines are just extraneous and confusing.

Cc: Steven Whitehouse <swhiteho@redhat.com>
Signed-off-by: default avatar"Eric W. Biederman" <ebiederm@xmission.com>
parent 05e0a60d
...@@ -65,9 +65,6 @@ ...@@ -65,9 +65,6 @@
#include "inode.h" #include "inode.h"
#include "util.h" #include "util.h"
#define QUOTA_USER 1
#define QUOTA_GROUP 0
struct gfs2_quota_change_host { struct gfs2_quota_change_host {
u64 qc_change; u64 qc_change;
u32 qc_flags; /* GFS2_QCF_... */ u32 qc_flags; /* GFS2_QCF_... */
...@@ -1476,18 +1473,14 @@ static int gfs2_get_dqblk(struct super_block *sb, struct kqid qid, ...@@ -1476,18 +1473,14 @@ static int gfs2_get_dqblk(struct super_block *sb, struct kqid qid,
struct gfs2_quota_data *qd; struct gfs2_quota_data *qd;
struct gfs2_holder q_gh; struct gfs2_holder q_gh;
int error; int error;
int type;
memset(fdq, 0, sizeof(struct fs_disk_quota)); memset(fdq, 0, sizeof(struct fs_disk_quota));
if (sdp->sd_args.ar_quota == GFS2_QUOTA_OFF) if (sdp->sd_args.ar_quota == GFS2_QUOTA_OFF)
return -ESRCH; /* Crazy XFS error code */ return -ESRCH; /* Crazy XFS error code */
if (qid.type == USRQUOTA) if ((qid.type != USRQUOTA) &&
type = QUOTA_USER; (qid.type != GRPQUOTA))
else if (qid.type == GRPQUOTA)
type = QUOTA_GROUP;
else
return -EINVAL; return -EINVAL;
error = qd_get(sdp, qid, &qd); error = qd_get(sdp, qid, &qd);
...@@ -1499,7 +1492,7 @@ static int gfs2_get_dqblk(struct super_block *sb, struct kqid qid, ...@@ -1499,7 +1492,7 @@ static int gfs2_get_dqblk(struct super_block *sb, struct kqid qid,
qlvb = (struct gfs2_quota_lvb *)qd->qd_gl->gl_lksb.sb_lvbptr; qlvb = (struct gfs2_quota_lvb *)qd->qd_gl->gl_lksb.sb_lvbptr;
fdq->d_version = FS_DQUOT_VERSION; fdq->d_version = FS_DQUOT_VERSION;
fdq->d_flags = (type == QUOTA_USER) ? FS_USER_QUOTA : FS_GROUP_QUOTA; fdq->d_flags = (qid.type == USRQUOTA) ? FS_USER_QUOTA : FS_GROUP_QUOTA;
fdq->d_id = from_kqid_munged(current_user_ns(), qid); fdq->d_id = from_kqid_munged(current_user_ns(), qid);
fdq->d_blk_hardlimit = be64_to_cpu(qlvb->qb_limit) << sdp->sd_fsb2bb_shift; fdq->d_blk_hardlimit = be64_to_cpu(qlvb->qb_limit) << sdp->sd_fsb2bb_shift;
fdq->d_blk_softlimit = be64_to_cpu(qlvb->qb_warn) << sdp->sd_fsb2bb_shift; fdq->d_blk_softlimit = be64_to_cpu(qlvb->qb_warn) << sdp->sd_fsb2bb_shift;
...@@ -1526,21 +1519,13 @@ static int gfs2_set_dqblk(struct super_block *sb, struct kqid qid, ...@@ -1526,21 +1519,13 @@ static int gfs2_set_dqblk(struct super_block *sb, struct kqid qid,
int alloc_required; int alloc_required;
loff_t offset; loff_t offset;
int error; int error;
int type;
if (sdp->sd_args.ar_quota == GFS2_QUOTA_OFF) if (sdp->sd_args.ar_quota == GFS2_QUOTA_OFF)
return -ESRCH; /* Crazy XFS error code */ return -ESRCH; /* Crazy XFS error code */
switch(qid.type) { if ((qid.type != USRQUOTA) &&
case USRQUOTA: (qid.type != GRPQUOTA))
type = QUOTA_USER;
break;
case GRPQUOTA:
type = QUOTA_GROUP;
break;
default:
return -EINVAL; return -EINVAL;
}
if (fdq->d_fieldmask & ~GFS2_FIELDMASK) if (fdq->d_fieldmask & ~GFS2_FIELDMASK)
return -EINVAL; return -EINVAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment