pseries: Clean up hcall_dprintf() debugging messages
The pseries machine code has a number of debug messages for debugging PAPR hypercalls, dependent on DEBUG_SPAPR_HCALLS. This patch cleans these messages up a bit, by adding __func__ to the hcall_dprintf() macro and simplifying up a number of the individual messages accordingly. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Andreas Färber <afaerber@suse.de>
This commit is contained in:
		
							parent
							
								
									52d631dcc7
								
							
						
					
					
						commit
						d9599c9205
					
				| 
						 | 
				
			
			@ -272,7 +272,7 @@ extern sPAPREnvironment *spapr;
 | 
			
		|||
 | 
			
		||||
#ifdef DEBUG_SPAPR_HCALLS
 | 
			
		||||
#define hcall_dprintf(fmt, ...) \
 | 
			
		||||
    do { fprintf(stderr, fmt, ## __VA_ARGS__); } while (0)
 | 
			
		||||
    do { fprintf(stderr, "%s: " fmt, __func__, ## __VA_ARGS__); } while (0)
 | 
			
		||||
#else
 | 
			
		||||
#define hcall_dprintf(fmt, ...) \
 | 
			
		||||
    do { } while (0)
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -279,21 +279,19 @@ static target_ulong h_register_logical_lan(CPUPPCState *env,
 | 
			
		|||
 | 
			
		||||
    if (check_bd(dev, VLAN_VALID_BD(buf_list, SPAPR_VIO_TCE_PAGE_SIZE),
 | 
			
		||||
                 SPAPR_VIO_TCE_PAGE_SIZE) < 0) {
 | 
			
		||||
        hcall_dprintf("Bad buf_list 0x" TARGET_FMT_lx " for "
 | 
			
		||||
                      "H_REGISTER_LOGICAL_LAN\n", buf_list);
 | 
			
		||||
        hcall_dprintf("Bad buf_list 0x" TARGET_FMT_lx "\n", buf_list);
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    filter_list_bd = VLAN_VALID_BD(filter_list, SPAPR_VIO_TCE_PAGE_SIZE);
 | 
			
		||||
    if (check_bd(dev, filter_list_bd, SPAPR_VIO_TCE_PAGE_SIZE) < 0) {
 | 
			
		||||
        hcall_dprintf("Bad filter_list 0x" TARGET_FMT_lx " for "
 | 
			
		||||
                      "H_REGISTER_LOGICAL_LAN\n", filter_list);
 | 
			
		||||
        hcall_dprintf("Bad filter_list 0x" TARGET_FMT_lx "\n", filter_list);
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if (!(rec_queue & VLAN_BD_VALID)
 | 
			
		||||
        || (check_bd(dev, rec_queue, VLAN_RQ_ALIGNMENT) < 0)) {
 | 
			
		||||
        hcall_dprintf("Bad receive queue for H_REGISTER_LOGICAL_LAN\n");
 | 
			
		||||
        hcall_dprintf("Bad receive queue\n");
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -358,13 +356,13 @@ static target_ulong h_add_logical_lan_buffer(CPUPPCState *env,
 | 
			
		|||
            ", 0x" TARGET_FMT_lx ")\n", reg, buf);
 | 
			
		||||
 | 
			
		||||
    if (!sdev) {
 | 
			
		||||
        hcall_dprintf("Wrong device in h_add_logical_lan_buffer\n");
 | 
			
		||||
        hcall_dprintf("Bad device\n");
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    if ((check_bd(dev, buf, 4) < 0)
 | 
			
		||||
        || (VLAN_BD_LEN(buf) < 16)) {
 | 
			
		||||
        hcall_dprintf("Bad buffer enqueued in h_add_logical_lan_buffer\n");
 | 
			
		||||
        hcall_dprintf("Bad buffer enqueued\n");
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -204,8 +204,7 @@ static target_ulong h_put_tce(CPUPPCState *env, sPAPREnvironment *spapr,
 | 
			
		|||
    VIOsPAPR_RTCE *rtce;
 | 
			
		||||
 | 
			
		||||
    if (!dev) {
 | 
			
		||||
        hcall_dprintf("spapr_vio_put_tce on non-existent LIOBN "
 | 
			
		||||
                      TARGET_FMT_lx "\n", liobn);
 | 
			
		||||
        hcall_dprintf("LIOBN 0x" TARGET_FMT_lx " does not exist\n", liobn);
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -217,8 +216,7 @@ static target_ulong h_put_tce(CPUPPCState *env, sPAPREnvironment *spapr,
 | 
			
		|||
#endif
 | 
			
		||||
 | 
			
		||||
    if (ioba >= dev->rtce_window_size) {
 | 
			
		||||
        hcall_dprintf("spapr_vio_put_tce on out-of-boards IOBA 0x"
 | 
			
		||||
                      TARGET_FMT_lx "\n", ioba);
 | 
			
		||||
        hcall_dprintf("Out-of-bounds IOBA 0x" TARGET_FMT_lx "\n", ioba);
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -414,22 +412,20 @@ static target_ulong h_reg_crq(CPUPPCState *env, sPAPREnvironment *spapr,
 | 
			
		|||
    VIOsPAPRDevice *dev = spapr_vio_find_by_reg(spapr->vio_bus, reg);
 | 
			
		||||
 | 
			
		||||
    if (!dev) {
 | 
			
		||||
        hcall_dprintf("h_reg_crq on non-existent unit 0x"
 | 
			
		||||
                      TARGET_FMT_lx "\n", reg);
 | 
			
		||||
        hcall_dprintf("Unit 0x" TARGET_FMT_lx " does not exist\n", reg);
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    /* We can't grok a queue size bigger than 256M for now */
 | 
			
		||||
    if (queue_len < 0x1000 || queue_len > 0x10000000) {
 | 
			
		||||
        hcall_dprintf("h_reg_crq, queue size too small or too big (0x%llx)\n",
 | 
			
		||||
                      (unsigned long long)queue_len);
 | 
			
		||||
        hcall_dprintf("Queue size too small or too big (0x" TARGET_FMT_lx
 | 
			
		||||
                      ")\n", queue_len);
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
    /* Check queue alignment */
 | 
			
		||||
    if (queue_addr & 0xfff) {
 | 
			
		||||
        hcall_dprintf("h_reg_crq, queue not aligned (0x%llx)\n",
 | 
			
		||||
                      (unsigned long long)queue_addr);
 | 
			
		||||
        hcall_dprintf("Queue not aligned (0x" TARGET_FMT_lx ")\n", queue_addr);
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -460,8 +456,7 @@ static target_ulong h_free_crq(CPUPPCState *env, sPAPREnvironment *spapr,
 | 
			
		|||
    VIOsPAPRDevice *dev = spapr_vio_find_by_reg(spapr->vio_bus, reg);
 | 
			
		||||
 | 
			
		||||
    if (!dev) {
 | 
			
		||||
        hcall_dprintf("h_free_crq on non-existent unit 0x"
 | 
			
		||||
                      TARGET_FMT_lx "\n", reg);
 | 
			
		||||
        hcall_dprintf("Unit 0x" TARGET_FMT_lx " does not exist\n", reg);
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -484,8 +479,7 @@ static target_ulong h_send_crq(CPUPPCState *env, sPAPREnvironment *spapr,
 | 
			
		|||
    uint64_t crq_mangle[2];
 | 
			
		||||
 | 
			
		||||
    if (!dev) {
 | 
			
		||||
        hcall_dprintf("h_send_crq on non-existent unit 0x"
 | 
			
		||||
                      TARGET_FMT_lx "\n", reg);
 | 
			
		||||
        hcall_dprintf("Unit 0x" TARGET_FMT_lx " does not exist\n", reg);
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
    crq_mangle[0] = cpu_to_be64(msg_hi);
 | 
			
		||||
| 
						 | 
				
			
			@ -505,8 +499,7 @@ static target_ulong h_enable_crq(CPUPPCState *env, sPAPREnvironment *spapr,
 | 
			
		|||
    VIOsPAPRDevice *dev = spapr_vio_find_by_reg(spapr->vio_bus, reg);
 | 
			
		||||
 | 
			
		||||
    if (!dev) {
 | 
			
		||||
        hcall_dprintf("h_enable_crq on non-existent unit 0x"
 | 
			
		||||
                      TARGET_FMT_lx "\n", reg);
 | 
			
		||||
        hcall_dprintf("Unit 0x" TARGET_FMT_lx " does not exist\n", reg);
 | 
			
		||||
        return H_PARAMETER;
 | 
			
		||||
    }
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue