On Wed, Oct 10, 2018 at 08:02:10AM -0700, Darrick J. Wong wrote:
> On Wed, Oct 10, 2018 at 04:58:53PM +0200, Carlos Maiolino wrote:
> > > > uint32_t xs_rmap_2[__XBTS_MAX];
> > > > -#define XFSSTAT_END_REFCOUNT (XFSSTAT_END_RMAP_V2 + __XBTS_MAX)
> > > > uint32_t xs_refcbt_2[__XBTS_MAX];
> > > > -#define XFSSTAT_END_XQMSTAT (XFSSTAT_END_REFCOUNT + 6)
> > > > uint32_t xs_qm_dqreclaims;
> > > > uint32_t xs_qm_dqreclaim_misses;
> > > > uint32_t xs_qm_dquot_dups;
> > > > uint32_t xs_qm_dqcachemisses;
> > > > uint32_t xs_qm_dqcachehits;
> > > > uint32_t xs_qm_dqwants;
> > > > -#define XFSSTAT_END_QM (XFSSTAT_END_XQMSTAT+2)
> > > > uint32_t xs_qm_dquot;
> > > > uint32_t xs_qm_dquot_unused;
> > > > /* Extra precision counters */
> > > > @@ -163,10 +139,12 @@ struct __xfsstats {
> > > > uint64_t xs_read_bytes;
> > > > };
> > > >
> > > > +#define xfsstats_offset(f) (offsetof(struct __xfsstats, f)/sizeof(uint32_t))
> > >
> > > Goes past 80 columns, but otherwise looks ok,
> > >
> > > Reviewed-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx>
> >
> > Ops, sorry, I traded readability+tabs X 80 columns, I think changing the tabs
> > for spaces is enough to fix it.
> >
> > Do you want me to send it again?
>
> <shrug> If Dave elects to fix it on the way in that's fine with me.
>
> Though I guess so long as I'm being pedantic about things that 'f' ought
> to be parentheses-wrapped too, e.g.
>
> #define xfsstats_offset(f) (offsetof(struct __xfsstats, (f)) / sizeof(uint32_t))
I'll clean it up on merge.
Thanks,
Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
