[lustre-devel] [PATCH 21/24] lustre: move remainder of linux-tracefile.c to tracefile.c
James Simmons
jsimmons at infradead.org
Sun Jun 24 13:29:05 PDT 2018
> On Thu, Jun 21 2018, James Simmons wrote:
>
> >> It's good to keep related code together.
> >>
> >> Signed-off-by: NeilBrown <neilb at suse.com>
> >
> > Nak. For some reason this corrupts my dmesg output. Its really strange and
> > I haven't figured it out yet.
>
> This patch cannot possibly be the cause of any such behavioural change.
> It purely moves code from one file to another, it doesn't change the
> code at all.
In a way you are correct. What this patch does is expose how badly messed
up cfs_print_to_console() is. Originally it was a printk() call but some
newbie changed it incorrectly into a pr_info() for all cases. This is
totally incorrect but it landed anyways :-( For some bizarre reason it
works even with it being completely wrong. I did fix this patch up
and redid the later patches due the changes from the fix up. Is it okay
it I push the newer version of these patches with my changes?
More information about the lustre-devel
mailing list