kernelnewbies.kernelnewbies.org archive mirror
 help / color / mirror / Atom feed
* irq_desc not found for my interrupt number during request_irq (radix tree of irq_desc has only 64 entries..)
@ 2022-04-06  1:45 Chan Kim
  0 siblings, 0 replies; 2+ messages in thread
From: Chan Kim @ 2022-04-06  1:45 UTC (permalink / raw)
  To: 'qemu-discuss', kernelnewbies


[-- Attachment #1.1: Type: text/plain, Size: 2975 bytes --]

Hello all, 

I'm doing an linux driver and application test on a qemu arm64 virtual
machine.
My virtual machine is slightly modified version of arm 'virt' machine and it
has our device model included.
When I do request_irq for INTID 208 (SPI 176) in linux on my arm64 virtual
machine, it returns -EINVAL.
I poked into that function and found the desc is returning NULL in function
request_threaded_irq.

	int request_threaded_irq(unsigned int irq, irq_handler_t handler,
	             irq_handler_t thread_fn, unsigned long irqflags,
	             const char *devname, void *dev_id)
	{
	    struct irqaction *action;
	    struct irq_desc *desc;
	    int retval;
	...
	desc = irq_to_desc(irq);
	    if (!desc) 
	        return -EINVAL;
	...
	}

and irq_to_desc is supposed to return an irq_desc from the irq_radix_tree as
shown below.

	struct irq_desc *irq_to_desc(unsigned int irq)
	{
	    return radix_tree_lookup(&irq_desc_tree, irq);
	}

	void *radix_tree_lookup(const struct radix_tree_root *root, unsigned
long index)
	{
	    return __radix_tree_lookup(root, index, NULL, NULL);
	}

	void *__radix_tree_lookup(const struct radix_tree_root *root,
	              unsigned long index, struct radix_tree_node **nodep,
	              void __rcu ***slotp)
	{
	    struct radix_tree_node *node, *parent;
	    unsigned long maxindex;
	    void __rcu **slot;

	 restart:
	    parent = NULL;
	    slot = (void __rcu **)&root->xa_head;
	    radix_tree_load_root(root, &node, &maxindex);
	    if (index > maxindex)
	        return NULL;
	...
	}
	
and I found my request index is 208 but the maxindex of the radix tree is
63. 
So it looks like the radix tree of irq_desc has been set to have maximum 64
irq_descs. 
Where and how can I set this radix tree size?
I looked at qemu's hw/arm/ab21q-build-acpi.c for MADT or DSDT table parts
but couldn't find what to add.
I have my devices address range and irq added in the DSDT and MADT.

static void
build_dsdt(GArray *table_data, BIOSLinker *linker, Ab21qMachineState *vms)
{
...
acpi_dsdt_add_axpu(scope, &memmap[AB21Q_AXPU],
                       (irqmap[AB21Q_AXPU] + ARM_SPI_BASE));
...
}
static void acpi_dsdt_add_axpu(Aml *scope, const MemMapEntry *axpu_memmap,
                                           uint32_t irq)
{
    Aml *dev = aml_device("AXPU");
    aml_append(dev, aml_name_decl("_HID", aml_string("AXPU0011")));
    aml_append(dev, aml_name_decl("_UID", aml_int(0)));

    Aml *crs = aml_resource_template();
    aml_append(crs, aml_memory32_fixed(axpu_memmap->base,
                                       axpu_memmap->size, AML_READ_WRITE));
    aml_append(crs,
               aml_interrupt(AML_CONSUMER, AML_LEVEL, AML_ACTIVE_HIGH,
                             AML_EXCLUSIVE, &irq, 1));
    aml_append(dev, aml_name_decl("_CRS", crs));

    aml_append(scope, dev);
}

I'm not sure if I can just use _HID value as "AXPU0011". 
If you see anything wrong and have a suggestion, please tell me.

Thank you!
Chan Kim


[-- Attachment #1.2: Type: text/html, Size: 16406 bytes --]

[-- Attachment #2: Type: text/plain, Size: 170 bytes --]

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: irq_desc not found for my interrupt number during request_irq (radix tree of irq_desc has only 64 entries..)
@ 2022-04-06  2:50 Chan Kim
  0 siblings, 0 replies; 2+ messages in thread
From: Chan Kim @ 2022-04-06  2:50 UTC (permalink / raw)
  To: 'qemu-discuss', kernelnewbies


[-- Attachment #1.1: Type: text/plain, Size: 4234 bytes --]

This is the boot message of the virtual machine related to IRQ.
[    0.000000] NR_IRQS: 64, nr_irqs: 64, preallocated irqs: 0
[    0.000000] GICv3: 224 SPIs implemented
[    0.000000] GICv3: 0 Extended SPIs implemented
[    0.000000] GICv3: Distributor has no Range Selector support
[    0.000000] GICv3: 16 PPIs implemented
[    0.000000] GICv3: no VLPI support, no direct LPI support
[    0.000000] GICv3: CPU0: found redistributor 0 region
0:0x00000000080a0000
Somehow the NR_IRQS is 64.
So I tried using SPI 15 which was not assigned. (32 internal IRQs + 15 < 64)
and now request_irq returns 0.
So this irq_desc problem was solved(?) for now. (I don't know why this is so
small when CONFIG_SPARSE_IRQS=y).
And still the handler is not called even when I set qemu_set_irq.
		_____________________________________________
		From: Chan Kim <ckim@etri.re.kr> 
		Sent: Wednesday, April 6, 2022 10:46 AM
		To: 'qemu-discuss' <qemu-discuss@nongnu.org>;
kernelnewbies@kernelnewbies.org
		Subject: irq_desc not found for my interrupt number during
request_irq (radix tree of irq_desc has only 64 entries..)
		
		
		Hello all, 

		I'm doing an linux driver and application test on a qemu
arm64 virtual machine.
		My virtual machine is slightly modified version of arm
'virt' machine and it has our device model included.
		When I do request_irq for INTID 208 (SPI 176) in linux on my
arm64 virtual machine, it returns -EINVAL.
		I poked into that function and found the desc is returning
NULL in function request_threaded_irq.

			int request_threaded_irq(unsigned int irq,
irq_handler_t handler,
			             irq_handler_t thread_fn, unsigned long
irqflags,
			             const char *devname, void *dev_id)
			{
			    struct irqaction *action;
			    struct irq_desc *desc;
			    int retval;
			...
			desc = irq_to_desc(irq);
			    if (!desc) 
			        return -EINVAL;
			...
			}
		
		and irq_to_desc is supposed to return an irq_desc from the
irq_radix_tree as shown below.

			struct irq_desc *irq_to_desc(unsigned int irq)
			{
			    return radix_tree_lookup(&irq_desc_tree, irq);
			}

			void *radix_tree_lookup(const struct radix_tree_root
*root, unsigned long index)
			{
			    return __radix_tree_lookup(root, index, NULL,
NULL);
			}

			void *__radix_tree_lookup(const struct
radix_tree_root *root,
			              unsigned long index, struct
radix_tree_node **nodep,
			              void __rcu ***slotp)
			{
			    struct radix_tree_node *node, *parent;
			    unsigned long maxindex;
			    void __rcu **slot;

			 restart:
			    parent = NULL;
			    slot = (void __rcu **)&root->xa_head;
			    radix_tree_load_root(root, &node, &maxindex);
			    if (index > maxindex)
			        return NULL;
			...
			}

		and I found my request index is 208 but the maxindex of the
radix tree is 63. 
		So it looks like the radix tree of irq_desc has been set to
have maximum 64 irq_descs. 
		Where and how can I set this radix tree size?
		I looked at qemu's hw/arm/ab21q-build-acpi.c for MADT or
DSDT table parts but couldn't find what to add.
		I have my devices address range and irq added in the DSDT
and MADT.

		static void
		build_dsdt(GArray *table_data, BIOSLinker *linker,
Ab21qMachineState *vms)
		{
		...
		acpi_dsdt_add_axpu(scope, &memmap[AB21Q_AXPU],
		                       (irqmap[AB21Q_AXPU] + ARM_SPI_BASE));
		...
		}
		static void acpi_dsdt_add_axpu(Aml *scope, const MemMapEntry
*axpu_memmap,
		                                           uint32_t irq)
		{
		    Aml *dev = aml_device("AXPU");
		    aml_append(dev, aml_name_decl("_HID",
aml_string("AXPU0011")));
		    aml_append(dev, aml_name_decl("_UID", aml_int(0)));

		    Aml *crs = aml_resource_template();
		    aml_append(crs, aml_memory32_fixed(axpu_memmap->base,
		                                       axpu_memmap->size,
AML_READ_WRITE));
		    aml_append(crs,
		               aml_interrupt(AML_CONSUMER, AML_LEVEL,
AML_ACTIVE_HIGH,
		                             AML_EXCLUSIVE, &irq, 1));
		    aml_append(dev, aml_name_decl("_CRS", crs));

		    aml_append(scope, dev);
		}

		I'm not sure if I can just use _HID value as "AXPU0011". 
		If you see anything wrong and have a suggestion, please tell
me.

		Thank you!
		Chan Kim
		

[-- Attachment #1.2: Type: text/html, Size: 17300 bytes --]

[-- Attachment #2: Type: text/plain, Size: 170 bytes --]

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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-04-06  2:51 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-06  1:45 irq_desc not found for my interrupt number during request_irq (radix tree of irq_desc has only 64 entries..) Chan Kim
2022-04-06  2:50 Chan Kim

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).