RE: Can't understand /proc/interrupts output for GICv3 case

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

 



Hi, Greg KH,
Thanks for the advices.
Some questions inline.

> -----Original Message-----
> From: 'Greg KH' <greg@xxxxxxxxx>
> Sent: Monday, April 11, 2022 11:26 PM
> To: Chan Kim <ckim@xxxxxxxxxx>
> Cc: 'qemu-discuss' <qemu-discuss@xxxxxxxxxx>;
> kernelnewbies@xxxxxxxxxxxxxxxxx
> Subject: Re: Can't understand /proc/interrupts output for GICv3 case
> 
> On Mon, Apr 11, 2022 at 11:15:57PM +0900, Chan Kim wrote:
> >
> > > > > What bus type is your driver written for?
> > > > >
> > > > That sounds very logical. In my case I added it to system bus.
> > >
> > > What exactly do you mean by "system bus"?
> > >
> > I meant 'sysbus' in qemu code that I showed in the qemu code.
> > And I think it's the CPU bus.
> >
> > >
> > > Where is your kernel code?
> > >
> > This is the init function of my char driver. I thought if the struct
> > cdev contains struct device, maybe I could use the struct device's
> > of_node to call of_irq_get but it doesn't.
> > And I remember I've seen the cdev in usually contained in the driver
> > data of platform driver(?). Can I implement platform driver in kernel
> module form?
> > Below is the char driver init code. Currently it's request_irq(6, ...
> > ) but I want to know out the number 6 using program. If you have any
> > advice, please tell me.
> >
> > static int __init chr_driver_init(void) {
> > 	int ret;
> > 	/* Allocating Major number */
> > 	if ((alloc_chrdev_region(&dev, 0, 1, "axpu_Dev")) < 0) {
> > 		printk(KERN_INFO"Cannot allocate the major number..\n");
> > 		return -1;
> > 	}
> >
> > 	printk(KERN_INFO"Major = %d Minor = %d..\n",MAJOR(dev),MINOR(dev));
> >
> > 	/* creating cdev structure */
> > 	cdev_init(&axpu_cdev, &fops);
> > 	axpu_cdev.owner = THIS_MODULE;
> >
> > 	/* Adding character device to the system */
> > 	if ((cdev_add(&axpu_cdev,dev,1)) < 0) {
> > 		printk(KERN_INFO "Cannot add the device to the
system...\n");
> > 		goto r_class;
> > 	}
> >
> > 	/* creating struct class */
> > 	if ((dev_class = class_create(THIS_MODULE, "axpu_class")) == NULL)
> {
> > 		printk(KERN_INFO "cannot create the struct class...\n");
> > 		goto r_class;
> > 	}
> >
> > 	/* for interrupt test !! */
> > 	/*  for vanilla work-around.. already made by mkdev */
> > 	if ((device_create(dev_class, NULL, dev, NULL, "axpu_device")) ==
> > NULL) {
> > 		printk(KERN_INFO "cannot create the device ..\n");
> > 		goto r_device;
> > 	}
> > 	else { printk(KERN_INFO "axpu_device created..\n"); }
> > 	/**/
> 
> You can replace all of the above code by just using the miscdevice
> interface instead.  Please use that, it ensures that you do everything
> properly and simplifies it all.
>
> >
> > 	vaddr = ioremap(AXPU_BASE, 0x80000);
> 
> Wait, where are you picking those random values from?

Yes, it now looks weird to me. I have passed the register address
information in the device tree and the kernel already knows my device's
address range. Then, how should I get this virtual io address in this
driver? I need it to access some registers. How can I ask the system bus?
And my driver is a kernel module because I want to use it in ubuntu-20.04 on
a virtual machine, I want it to be a kernel module that I can insmod or
rmmod.(actually my job is to provide this virtual machine to some folks).
And I cannot build the ubuntu image even if I change it to a platform device
driver and add it in the kernel tree.

> That's not how drivers work in the kernel.  You have to have a bus pass
> you a device.  Please register your driver with the proper bus type and go
> from there.  The probe callback should be the place you set things up
> based on the device and resources given to you by the bus.
> 
> You can't just randomly pick memory locations and think things will work.
> 
> > 	if(!vaddr)
> > 	{
> > 		printk(KERN_INFO"Failed to map the address.\n");
> > 		release_mem_region(AXPU_BASE,AXPU_SIZE);
> > 		return 1;
> > 	}
> > 	printk("----- AXPU_BASE mapped at vaddr = %px\n", vaddr);
> >
> > 	ret = request_irq(6, axpu_irq_handler, IRQF_SHARED, "axpu_irq",
> > &axpu_cdev);
> 
> Same for that, just picking 6 will not work, sorry.
> 

Yes, that was my original question. How can I get my irq number (I know it's
hwirq 47) and I peeked into kernel that irq 6 was assigned for the irq_desc.
So I changed my driver to request irq 6 for my device and I found at least
it works for now, all the register access and interrupts. I know this is not
the solution and I'm curious how I should get the irq number of io virtual
address in this situation.

> Perhaps take a look at the book, Linux Device Drivers, 3rd edition.
> It's free online and should help you out a lot.
> 
> > 	printk("request_irq returned %d\n", ret); // -EINVAL
> > 	printk(KERN_INFO "Device driver inserted ..done properly..\n");
> > 	return 0;
> >
> > r_device :
> > 	class_destroy(dev_class);
> >
> > r_class :
> > 	unregister_chrdev_region(dev,1);
> > 	return -1;
> 
> One final comment, don't make up error values like this, use real ERROR
> codes.
> 
> thanks,
> 
> greg k-h

Yes, I've read the book sometimes (not the whole part) but if I read it now,
I'll be able to more understand it. It's a bit outdated though. Why don't
you update your book? :)

Thank you again for the advices.
Best regards,

Chan Kim





_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies



[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]

  Powered by Linux