Replace the awkward xlog_write_adv_cnt with an inline helper that makes
it more obvious that it's modifying it's paramters, and replace the use
of an integer type for "ptr" with a real void pointer. Also move
xlog_write_adv_cnt to xfs_log_priv.h as it will be used outside of
xfs_log.c in the delayed logging series.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
kmem_zone_t *xfs_log_ticket_zone;
kmem_zone_t *xfs_log_ticket_zone;
-#define xlog_write_adv_cnt(ptr, len, off, bytes) \
- { (ptr) += (bytes); \
- (len) -= (bytes); \
- (off) += (bytes);}
-
/* Local miscellaneous function prototypes */
STATIC int xlog_commit_record(struct log *log, struct xlog_ticket *ticket,
xlog_in_core_t **, xfs_lsn_t *);
/* Local miscellaneous function prototypes */
STATIC int xlog_commit_record(struct log *log, struct xlog_ticket *ticket,
xlog_in_core_t **, xfs_lsn_t *);
uint flags);
#if defined(DEBUG)
uint flags);
#if defined(DEBUG)
-STATIC void xlog_verify_dest_ptr(xlog_t *log, __psint_t ptr);
+STATIC void xlog_verify_dest_ptr(xlog_t *log, char *ptr);
STATIC void xlog_verify_grant_head(xlog_t *log, int equals);
STATIC void xlog_verify_iclog(xlog_t *log, xlog_in_core_t *iclog,
int count, boolean_t syncing);
STATIC void xlog_verify_grant_head(xlog_t *log, int equals);
STATIC void xlog_verify_iclog(xlog_t *log, xlog_in_core_t *iclog,
int count, boolean_t syncing);
*/
static int
xlog_write_start_rec(
*/
static int
xlog_write_start_rec(
+ struct xlog_op_header *ophdr,
struct xlog_ticket *ticket)
{
struct xlog_ticket *ticket)
{
- struct xlog_op_header *ophdr = (struct xlog_op_header *)ptr;
-
if (!(ticket->t_flags & XLOG_TIC_INITED))
return 0;
if (!(ticket->t_flags & XLOG_TIC_INITED))
return 0;
static xlog_op_header_t *
xlog_write_setup_ophdr(
struct log *log,
static xlog_op_header_t *
xlog_write_setup_ophdr(
struct log *log,
+ struct xlog_op_header *ophdr,
struct xlog_ticket *ticket,
uint flags)
{
struct xlog_ticket *ticket,
uint flags)
{
- struct xlog_op_header *ophdr = (struct xlog_op_header *)ptr;
-
ophdr->oh_tid = cpu_to_be32(ticket->t_tid);
ophdr->oh_clientid = ticket->t_clientid;
ophdr->oh_res2 = 0;
ophdr->oh_tid = cpu_to_be32(ticket->t_tid);
ophdr->oh_clientid = ticket->t_clientid;
ophdr->oh_res2 = 0;
lv = log_vector;
vecp = lv->lv_iovecp;
while (lv && index < lv->lv_niovecs) {
lv = log_vector;
vecp = lv->lv_iovecp;
while (lv && index < lv->lv_niovecs) {
int log_offset;
error = xlog_state_get_iclog_space(log, len, &iclog, ticket,
int log_offset;
error = xlog_state_get_iclog_space(log, len, &iclog, ticket,
return error;
ASSERT(log_offset <= iclog->ic_size - 1);
return error;
ASSERT(log_offset <= iclog->ic_size - 1);
- ptr = (__psint_t)((char *)iclog->ic_datap + log_offset);
+ ptr = iclog->ic_datap + log_offset;
/* start_lsn is the first lsn written to. That's all we need. */
if (!*start_lsn)
/* start_lsn is the first lsn written to. That's all we need. */
if (!*start_lsn)
int copy_off;
ASSERT(reg->i_len % sizeof(__int32_t) == 0);
int copy_off;
ASSERT(reg->i_len % sizeof(__int32_t) == 0);
- ASSERT((__psint_t)ptr % sizeof(__int32_t) == 0);
+ ASSERT((unsigned long)ptr % sizeof(__int32_t) == 0);
start_rec_copy = xlog_write_start_rec(ptr, ticket);
if (start_rec_copy) {
record_cnt++;
start_rec_copy = xlog_write_start_rec(ptr, ticket);
if (start_rec_copy) {
record_cnt++;
- xlog_write_adv_cnt(ptr, len, log_offset,
+ xlog_write_adv_cnt(&ptr, &len, &log_offset,
if (!ophdr)
return XFS_ERROR(EIO);
if (!ophdr)
return XFS_ERROR(EIO);
- xlog_write_adv_cnt(ptr, len, log_offset,
+ xlog_write_adv_cnt(&ptr, &len, &log_offset,
sizeof(struct xlog_op_header));
len += xlog_write_setup_copy(ticket, ophdr,
sizeof(struct xlog_op_header));
len += xlog_write_setup_copy(ticket, ophdr,
/* copy region */
ASSERT(copy_len >= 0);
/* copy region */
ASSERT(copy_len >= 0);
- memcpy((xfs_caddr_t)ptr, reg->i_addr + copy_off,
- copy_len);
- xlog_write_adv_cnt(ptr, len, log_offset, copy_len);
+ memcpy(ptr, reg->i_addr + copy_off, copy_len);
+ xlog_write_adv_cnt(&ptr, &len, &log_offset, copy_len);
copy_len += start_rec_copy + sizeof(xlog_op_header_t);
record_cnt++;
copy_len += start_rec_copy + sizeof(xlog_op_header_t);
record_cnt++;
* part of the log in case we trash the log structure.
*/
void
* part of the log in case we trash the log structure.
*/
void
-xlog_verify_dest_ptr(xlog_t *log,
- __psint_t ptr)
+xlog_verify_dest_ptr(
+ struct log *log,
+ char *ptr)
{
int i;
int good_ptr = 0;
{
int i;
int good_ptr = 0;
- for (i=0; i < log->l_iclog_bufs; i++) {
- if (ptr >= (__psint_t)log->l_iclog_bak[i] &&
- ptr <= (__psint_t)log->l_iclog_bak[i]+log->l_iclog_size)
+ for (i = 0; i < log->l_iclog_bufs; i++) {
+ if (ptr >= log->l_iclog_bak[i] &&
+ ptr <= log->l_iclog_bak[i] + log->l_iclog_size)
xlog_panic("xlog_verify_dest_ptr: invalid ptr");
xlog_panic("xlog_verify_dest_ptr: invalid ptr");
-} /* xlog_verify_dest_ptr */
STATIC void
xlog_verify_grant_head(xlog_t *log, int equals)
STATIC void
xlog_verify_grant_head(xlog_t *log, int equals)
extern kmem_zone_t *xfs_log_ticket_zone;
extern kmem_zone_t *xfs_log_ticket_zone;
+static inline void
+xlog_write_adv_cnt(void **ptr, int *len, int *off, size_t bytes)
+{
+ *ptr += bytes;
+ *len -= bytes;
+ *off += bytes;
+}
+
/*
* Unmount record type is used as a pseudo transaction type for the ticket.
* It's value must be outside the range of XFS_TRANS_* values.
/*
* Unmount record type is used as a pseudo transaction type for the ticket.
* It's value must be outside the range of XFS_TRANS_* values.