[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] traps: coding style fixes




On 11.09.18 13:42, Julien Grall wrote:
@@ -1429,7 +1430,8 @@ static void do_debug_trap(struct cpu_user_regs *regs, unsigned int code)
  {
      uint32_t reg;
      uint32_t domid = current->domain->domain_id;

While you are at fixing coding style: newline here.

-    switch ( code ) {
+    switch ( code )
+    {
      case 0xe0 ... 0xef:
          reg = code - 0xe0;
          printk("DOM%d: R%d = 0x%"PRIregister" at 0x%"PRIvaddr"\n",
@@ -2071,7 +2073,8 @@ void do_trap_guest_sync(struct cpu_user_regs *regs)
        enter_hypervisor_head(regs);
  -    switch (hsr.ec) {
+    switch (hsr.ec)

Xen coding style requires space after ( and before ).

+    {
      case HSR_EC_WFI_WFE:
          /*
           * HCR_EL2.TWI, HCR_EL2.TWE
@@ -2270,7 +2273,8 @@ void leave_hypervisor_tail(void)
      while (1)
      {
          local_irq_disable();
-        if (!softirq_pending(smp_processor_id())) {
+        if (!softirq_pending(smp_processor_id()))

Same here.

Yep.

--

*Andrii Anisov*



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.