* Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > > + if (n < NR_REGPARMS) { > > + switch (n) { > > + case 0: return regs->ax; > > + case 1: return regs->dx; > > + case 2: return regs->cx; > > + } > > Normal kernel style would be > > switch (n) { > case 0: > return regs->ax; > case 1: > return regs->dx; > case 2: > return regs->cx; > } the original single-line shortcut is acceptable too. Ingo -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html