Re: [PATCH 1/2] powerpc/mm/ptdump: Wrap seq_printf() to handle NULL pointers
Russell Currey
ruscur at russell.cc
Wed Apr 24 17:00:33 AEST 2019
whoops, will fix
--
Russell Currey
ruscur at russell.cc
On Wed, Apr 24, 2019, at 4:56 PM, Christophe Leroy wrote:
>
>
> Le 24/04/2019 à 08:39, Russell Currey a écrit :
> > Lovingly borrowed from the arch/arm64 ptdump code.
> >
> > This doesn't seem to be an issue in practice, but is necessary for my
> > upcoming commit.
> >
> > Converts a putc() into a puts().
> >
> > Signed-off-by: Russell Currey <ruscur at russell.cc>
> > ---
> > arch/powerpc/mm/ptdump/ptdump.c | 32 ++++++++++++++++++++++----------
> > 1 file changed, 22 insertions(+), 10 deletions(-)
> >
> > diff --git a/arch/powerpc/mm/ptdump/ptdump.c b/arch/powerpc/mm/ptdump/ptdump.c
> > index 37138428ab55..c50cb7faa334 100644
> > --- a/arch/powerpc/mm/ptdump/ptdump.c
> > +++ b/arch/powerpc/mm/ptdump/ptdump.c
> > @@ -104,6 +104,18 @@ static struct addr_marker address_markers[] = {
> > { -1, NULL },
> > };
> >
> > +#define pt_dump_seq_printf(m, fmt, args...) \
> > +({ \
> > + if (m) \
> > + seq_printf(m, fmt, ##args); \
> > +})
> > +
> > +#define pt_dump_seq_puts(m, fmt) \
> > +({ \
> > + if (m) \
> > + seq_printf(m, fmt); \
>
> Why not use seq_puts() here ?
>
> Christophe
>
> > +})
> > +
> > static void dump_flag_info(struct pg_state *st, const struct flag_info
> > *flag, u64 pte, int num)
> > {
> > @@ -121,19 +133,19 @@ static void dump_flag_info(struct pg_state *st, const struct flag_info
> > val = pte & flag->val;
> > if (flag->shift)
> > val = val >> flag->shift;
> > - seq_printf(st->seq, " %s:%llx", flag->set, val);
> > + pt_dump_seq_printf(st->seq, " %s:%llx", flag->set, val);
> > } else {
> > if ((pte & flag->mask) == flag->val)
> > s = flag->set;
> > else
> > s = flag->clear;
> > if (s)
> > - seq_printf(st->seq, " %s", s);
> > + pt_dump_seq_printf(st->seq, " %s", s);
> > }
> > st->current_flags &= ~flag->mask;
> > }
> > if (st->current_flags != 0)
> > - seq_printf(st->seq, " unknown flags:%llx", st->current_flags);
> > + pt_dump_seq_printf(st->seq, " unknown flags:%llx", st->current_flags);
> > }
> >
> > static void dump_addr(struct pg_state *st, unsigned long addr)
> > @@ -148,12 +160,12 @@ static void dump_addr(struct pg_state *st, unsigned long addr)
> > #define REG "0x%08lx"
> > #endif
> >
> > - seq_printf(st->seq, REG "-" REG " ", st->start_address, addr - 1);
> > + pt_dump_seq_printf(st->seq, REG "-" REG " ", st->start_address, addr - 1);
> > if (st->start_pa == st->last_pa && st->start_address + PAGE_SIZE != addr) {
> > - seq_printf(st->seq, "[" REG "]", st->start_pa);
> > + pt_dump_seq_printf(st->seq, "[" REG "]", st->start_pa);
> > delta = PAGE_SIZE >> 10;
> > } else {
> > - seq_printf(st->seq, " " REG " ", st->start_pa);
> > + pt_dump_seq_printf(st->seq, " " REG " ", st->start_pa);
> > delta = (addr - st->start_address) >> 10;
> > }
> > /* Work out what appropriate unit to use */
> > @@ -161,7 +173,7 @@ static void dump_addr(struct pg_state *st, unsigned long addr)
> > delta >>= 10;
> > unit++;
> > }
> > - seq_printf(st->seq, "%9lu%c", delta, *unit);
> > + pt_dump_seq_printf(st->seq, "%9lu%c", delta, *unit);
> >
> > }
> >
> > @@ -178,7 +190,7 @@ static void note_page(struct pg_state *st, unsigned long addr,
> > st->start_address = addr;
> > st->start_pa = pa;
> > st->last_pa = pa;
> > - seq_printf(st->seq, "---[ %s ]---\n", st->marker->name);
> > + pt_dump_seq_printf(st->seq, "---[ %s ]---\n", st->marker->name);
> > /*
> > * Dump the section of virtual memory when:
> > * - the PTE flags from one entry to the next differs.
> > @@ -202,7 +214,7 @@ static void note_page(struct pg_state *st, unsigned long addr,
> > st->current_flags,
> > pg_level[st->level].num);
> >
> > - seq_putc(st->seq, '\n');
> > + pt_dump_seq_puts(st->seq, "\n");
> > }
> >
> > /*
> > @@ -211,7 +223,7 @@ static void note_page(struct pg_state *st, unsigned long addr,
> > */
> > while (addr >= st->marker[1].start_address) {
> > st->marker++;
> > - seq_printf(st->seq, "---[ %s ]---\n", st->marker->name);
> > + pt_dump_seq_printf(st->seq, "---[ %s ]---\n", st->marker->name);
> > }
> > st->start_address = addr;
> > st->start_pa = pa;
> >
>
More information about the Linuxppc-dev
mailing list