一个copy_to_user无法实现功能的问题，跪求解决！

yv9200

我们之前开发的是在ubuntu开发的驱动以及应用程序，架构X86 32,应用程序32位，内核版本为2.6.26.3，开发完可以正常使用。但目前工作需用把它移植到mips64的中标麒麟的系统上，遇到了和驱动开发有关的问题，之前是ioctl无法调用成功，后来把.unlocked_ioctl改成compat_ioctl后，ioctl可以正常工作，新的问题是通过ioctl找到的信息无法通过copy_to_user传到用户空间，导致程序无法工作，我在网上找了一些文章说是32位的应用程序和64位的操作系统有冲突，说是需要把copy_to_user第一个参数从原来的32位改成64位就好了 ，我把那个指针的类型从unsigned long 改成unsigned long long ，仍然不能给用户空间返回正确的值。求大神解答！

Tinnal

回复 1# yv9200


  这个好弄呀，内核里头已经有大量的compat_ioctl的参考实例。多看看别人各种情况下是怎么写的就可以了。

Tinnal

下面是driver/scsi/ch.c的一个样例，这样的例子还有很多。自己慢慢学一下就行。要注意的只是：第一个参数的内容会按32位程序的布局来安排，因此内核在使用时，要注意转换，其它就没什么了。

1. 
   
2. static long ch_ioctl_compat(struct file * file,
   
3.                             unsigned int cmd, unsigned long arg)
   
4. {
   
5.         scsi_changer *ch = file->private_data;
   
6.        
   
7.         switch (cmd) {
   
8.         case CHIOGPARAMS:
   
9.         case CHIOGVPARAMS:
   
10.         case CHIOPOSITION:
    
11.         case CHIOMOVE:
    
12.         case CHIOEXCHANGE:
    
13.         case CHIOGELEM:
    
14.         case CHIOINITELEM:
    
15.         case CHIOSVOLTAG:
    
16.                 /* compatible */
    
17.                 return ch_ioctl(NULL /* inode, unused */,
    
18.                                 file, cmd, arg);
    
19.         case CHIOGSTATUS32:
    
20.         {
    
21.                 struct changer_element_status32 ces32;
    
22.                 unsigned char __user *data;
    
23.                
    
24.                 if (copy_from_user(&ces32, (void __user *)arg, sizeof (ces32)))
    
25.                         return -EFAULT;
    
26.                 if (ces32.ces_type < 0 || ces32.ces_type >= CH_TYPES)
    
27.                         return -EINVAL;
    
28. 
    
29.                 data = compat_ptr(ces32.ces_data);
    
30.                 return ch_gstatus(ch, ces32.ces_type, data);
    
31.         }
    
32.         default:
    
33.                 // return scsi_ioctl_compat(ch->device, cmd, (void*)arg);
    
34.                 return -ENOIOCTLCMD;
    
35. 
    
36.         }
    
37. }
    
38. #endif

复制代码

yv9200

这是我的ioctl里面的一段代码 标注为红色的那个debug可以打印出数值，但传到应用程序，那个结构体打印出的都是默认的值，我发现copy_to_user里的第一个参数已经被转化为64位。
case VDRIVER_SCAN_CARDS:
                {
                        vdriver_debug("\n");
                        vdriver_debug("VDRIVER_SCAN_CARDS............\n");
                        vd_scan_cards = kmalloc(sizeof(VD_PCI_SCAN_CARDS), GFP_KERNEL);
                        if(copy_from_user(vd_scan_cards, (VD_PCI_SCAN_CARDS *)arg, sizeof(VD_PCI_SCAN_CARDS)))
                        {
                                vdriver_error("VDRIVER_SCAN_CARDS comand error !\n");
                                return -EFAULT;
                        }
                        vendorid = vd_scan_cards->searchId.dwVendorId;
                        deviceid = vd_scan_cards->searchId.dwDeviceId;
                        if(deviceid == 0)
                        {
                                deviceid = PCI_ANY_ID;
                        }
                        if(vendorid == 0)
                        {
                                vendorid = PCI_ANY_ID;
                        }
                        vdriver_debug("The usr params vendorid is %x,deviceid is %x\n",vendorid, deviceid);
                        do
                        {
                                pci_dev = pci_get_device(vendorid, deviceid, pci_dev);
                                vdriver_debug("vdriver scan cards the pci_dev is %x\n",(unsigned int)pci_dev);
                                if(pci_dev != NULL)
                                {
                                        vd_scan_cards->cardId[card_count].dwVendorId = pci_dev->vendor;
                                        vd_scan_cards->cardId[card_count].dwDeviceId = pci_dev->device;
                                        vdriver_debug("vdriver scan cards VendorId is %x,DeviceId is %x\n",pci_dev->vendor,pci_dev->device);
                                        busnum = (pci_dev->bus->number);
                                        devnum = ((pci_dev->devfn)>>3) & 0x1f;
                                        funnum = (pci_dev->devfn) & 0x7;
                                        vd_scan_cards->cardSlot[card_count].dwBus = busnum;
                                        vd_scan_cards->cardSlot[card_count].dwSlot = devnum;
                                        vd_scan_cards->cardSlot[card_count].dwFunction = funnum;
                                        vdriver_debug("vdriver scan cards dwBus is %d,dwSlot is %d,dwFunction is %d\n",(unsigned int)vd_scan_cards->cardSlot[card_count].dwBus,
                                                                 (unsigned int)vd_scan_cards->cardSlot[card_count].dwSlot,(unsigned int)vd_scan_cards->cardSlot[card_count].dwFunction);
                                        card_ir = pci_dev->irq;
                                        vdriver_debug("vdriver scan cards interrupt num is %x\n",card_ir);
                                        card_count ++;
                                }
                        }
                        while(pci_dev != 0);
                        vd_scan_cards->dwCards = card_count;
                        vdriver_debug("Total scan the card number is %d...........\n",(unsigned int)vd_scan_cards->dwCards);
                        if (copy_to_user((VD_PCI_SCAN_CARDS *)arg, vd_scan_cards, sizeof(VD_PCI_SCAN_CARDS)))
                                return -EFAULT;
                        kfree(vd_scan_cards);
                        break;
                }回复 3# Tinnal


   

playmud

如果你是uint32_t那么用打印用%u 如果是int32_t 那么用%d 如果是uint64_t 那么用%lu 如果是int64_t那么用%l打印。如果还有问题，检查是不是结构体字节对齐的问题。
你的问题不是copy_to_user和copy_from_user的问题。
此外vd_scan_cards 你需要判断是否分配成功， 另外 return 之前 你需要释放掉。

yv9200

多谢大神的解答，问题得以解决，是结构体字节对齐的问题。回复 5# playmud