On 31.08.2017 17:32, Stephen Hemminger wrote: > On Thu, 31 Aug 2017 12:22:29 +0200 > Markus Theil <markus.th...@tu-ilmenau.de> wrote: > >> +/* >> + * It masks the msi on/off of generating MSI messages. >> + */ >> +static void >> +igbuio_msi_mask_irq(struct pci_dev *pdev, struct msi_desc *desc, int32_t >> state) >> +{ >> + u32 mask_bits = desc->masked; >> + u32 offset = desc->irq - pdev->irq; >> + u32 mask = 1 << offset; >> + u32 flag = !!state << offset; >> + >> + if (!desc->msi_attrib.maskbit) >> + return; >> + >> + mask_bits &= ~mask; >> + mask_bits |= flag; >> + >> + if (mask_bits != desc->masked) { >> + pci_write_config_dword(pdev, desc->mask_pos, mask_bits); >> + desc->masked = mask_bits; >> + } >> +} >> + > Why not use the existing kernel API pci_msi_mask_irq()? Thanks for the advice, I have considered it in v5.
- [dpdk-dev] [PATCH] igb_uio: MSI IRQ mode, irq enable/dis... Markus Theil
- [dpdk-dev] [PATCH v2 1/2] igb_uio: MSI IRQ mode, ir... Markus Theil
- [dpdk-dev] [PATCH v2 2/2] igb_uio: conform to c... Markus Theil
- [dpdk-dev] [PATCH v3] igb_uio: MSI IRQ mode... Markus Theil
- Re: [dpdk-dev] [PATCH v3] igb_uio: MSI ... Ferruh Yigit
- Re: [dpdk-dev] [PATCH v3] igb_uio:... Markus Theil
- [dpdk-dev] [PATCH v4 1/3] igb_uio: refa... Markus Theil
- [dpdk-dev] [PATCH v4 2/3] igb_uio:... Markus Theil
- [dpdk-dev] [PATCH v4 3/3] igb_uio:... Markus Theil
- Re: [dpdk-dev] [PATCH v4 3/3] ... Stephen Hemminger
- Re: [dpdk-dev] [PATCH v4 3... Markus Theil
- [dpdk-dev] [PATCH v5 1/5] igb_... Markus Theil
- [dpdk-dev] [PATCH v5 3/5] ... Markus Theil
- [dpdk-dev] [PATCH v5 4/5] ... Markus Theil
- Re: [dpdk-dev] [PATCH v5 4... Stephen Hemminger
- Re: [dpdk-dev] [PATCH v5 4... Burakov, Anatoly
- Re: [dpdk-dev] [PATCH v5 4... Ferruh Yigit
- Re: [dpdk-dev] [PATCH v5 4... Markus Theil
- Re: [dpdk-dev] [PATCH v5 4... Ferruh Yigit
- [dpdk-dev] [PATCH v5 2/5] ... Markus Theil
- [dpdk-dev] [PATCH v5 5/5] ... Markus Theil