everybody knows that interrupt handler should be short as possible. and adding functions like printk for debugging inside an interrupt handler is something that
Yes, it is very bad since printf most probably is not reentrant. What can happen is that the main program calls printf, an interrupt arrives while printf is executing, and then the IRQ handler calls printf again: very bad things may happen (e.g., deadlock, corrupt internal buffers, etc.)