search for: device_initcall

...static int __init virtio_ccw_init(void) > { > /* parse no_auto string before we do anything further */ > no_auto_parse(); > + summary_indicators = cio_dma_zalloc(MAX_AIRQ_AREAS); What happens if this fails? > return ccw_driver_register(&virtio_ccw_driver); > } > device_initcall(virtio_ccw_init);

...static int __init virtio_ccw_init(void) > { > /* parse no_auto string before we do anything further */ > no_auto_parse(); > + summary_indicators = cio_dma_zalloc(MAX_AIRQ_AREAS); What happens if this fails? > return ccw_driver_register(&virtio_ccw_driver); > } > device_initcall(virtio_ccw_init);

...#include <linux/pagemap.h> > #include <linux/swap.h> > +#include <linux/virtio_config.h> > #include <asm/facility.h> > #include <asm/sections.h> > #include <asm/uv.h> > @@ -413,3 +414,27 @@ static int __init uv_info_init(void) > } > device_initcall(uv_info_init); > #endif > + > +/* > + * arch_validate_virtio_iommu_platform s/arch_validate_virtio_iommu_platform/arch_validate_virtio_features/ > + * @dev: the VIRTIO device being added > + * > + * Return value: returns -ENODEV if any features of the > + *...

...#include <linux/pagemap.h> > #include <linux/swap.h> > +#include <linux/virtio_config.h> > #include <asm/facility.h> > #include <asm/sections.h> > #include <asm/uv.h> > @@ -413,3 +414,27 @@ static int __init uv_info_init(void) > } > device_initcall(uv_info_init); > #endif > + > +/* > + * arch_validate_virtio_iommu_platform s/arch_validate_virtio_iommu_platform/arch_validate_virtio_features/ > + * @dev: the VIRTIO device being added > + * > + * Return value: returns -ENODEV if any features of the > + *...

..._parse(); > + > + summary_indicators = cio_dma_zalloc(MAX_AIRQ_AREAS); > + if (!summary_indicators) > + return -ENOMEM; > + > return ccw_driver_register(&virtio_ccw_driver); Don't you need to free summary_indicators again if registering the driver fails? > } > device_initcall(virtio_ccw_init);

...#include <linux/pagemap.h> > #include <linux/swap.h> > +#include <linux/virtio_config.h> > #include <asm/facility.h> > #include <asm/sections.h> > #include <asm/uv.h> > @@ -413,3 +414,27 @@ static int __init uv_info_init(void) > } > device_initcall(uv_info_init); > #endif > + > +/* > + * arch_validate_virtio_iommu_platform > + * @dev: the VIRTIO device being added > + * > + * Return value: returns -ENODEV if any features of the > + * device breaks the protected virtualization > + * 0 oth...

Hi all, I changed the patch subject to reflect the content, becoming more general. 1) I removed the ack from Christian and Jason even far as I understand they gave it for the functionality more than for the implementation. @Jason, @Christian, please can I get back your acked-by with these changes? 2) previous patch had another name: [PATCH v3 0/1] s390: virtio: let arch choose to

...t; @@ -1500,6 +1509,7 @@ static int __init virtio_ccw_init(void) > { > /* parse no_auto string before we do anything further */ > no_auto_parse(); > + summary_indicators = cio_dma_zalloc(MAX_AIRQ_AREAS); > return ccw_driver_register(&virtio_ccw_driver); > } > device_initcall(virtio_ccw_init); > -- Pierre Morel Linux/KVM/QEMU in B?blingen - Germany

...t; @@ -1500,6 +1509,7 @@ static int __init virtio_ccw_init(void) > { > /* parse no_auto string before we do anything further */ > no_auto_parse(); > + summary_indicators = cio_dma_zalloc(MAX_AIRQ_AREAS); > return ccw_driver_register(&virtio_ccw_driver); > } > device_initcall(virtio_ccw_init); > -- Pierre Morel Linux/KVM/QEMU in B?blingen - Germany

...adding if (!summary_indicators) virtio_ccw_use_airq = 0; /* fall back to classic */ ? Since it ain't very likely to happen, we could also just fail virtio_ccw_init() with -ENOMEM. Regards, Halil > > > return ccw_driver_register(&virtio_ccw_driver); > > } > > device_initcall(virtio_ccw_init); >

...7 @@ #include <linux/memblock.h> #include <linux/pagemap.h> #include <linux/swap.h> +#include <linux/virtio_config.h> #include <asm/facility.h> #include <asm/sections.h> #include <asm/uv.h> @@ -413,3 +414,27 @@ static int __init uv_info_init(void) } device_initcall(uv_info_init); #endif + +/* + * arch_validate_virtio_iommu_platform + * @dev: the VIRTIO device being added + * + * Return value: returns -ENODEV if any features of the + * device breaks the protected virtualization + * 0 otherwise. + */ +int arch_validate_virtio_featur...

...gt; #include <linux/swap.h> > > +#include <linux/virtio_config.h> > > #include <asm/facility.h> > > #include <asm/sections.h> > > #include <asm/uv.h> > > @@ -413,3 +414,27 @@ static int __init uv_info_init(void) > > } > > device_initcall(uv_info_init); > > #endif > > + > > +/* > > + * arch_validate_virtio_iommu_platform > > s/arch_validate_virtio_iommu_platform/arch_validate_virtio_features/ > > > + * @dev: the VIRTIO device being added > > + * > > + * Return value: returns -ENO...

...> #include <linux/swap.h> >> +#include <linux/virtio_config.h> >> #include <asm/facility.h> >> #include <asm/sections.h> >> #include <asm/uv.h> >> @@ -413,3 +414,27 @@ static int __init uv_info_init(void) >> } >> device_initcall(uv_info_init); >> #endif >> + >> +/* >> + * arch_validate_virtio_iommu_platform > > s/arch_validate_virtio_iommu_platform/arch_validate_virtio_features/ > >> + * @dev: the VIRTIO device being added >> + * >> + * Return value: returns -ENODEV if...

...ysfs_ops hyp_sysfs_ops = { .show = hyp_sysfs_show, .store = hyp_sysfs_store, }; @@ -52,8 +50,7 @@ static int __init hypervisor_subsys_init return -ENODEV; hypervisor_subsys.kset.kobj.ktype = &hyp_sysfs_kobj_type; - return subsystem_register(&hypervisor_subsys); + return 0; } device_initcall(hypervisor_subsys_init); -EXPORT_SYMBOL_GPL(hypervisor_subsys); Index: head-2007-05-31/include/xen/hypervisor_sysfs.h =================================================================== --- head-2007-05-31.orig/include/xen/hypervisor_sysfs.h 2007-06-06 17:04:59.000000000 +0200 +++ head-2007-05-31/i...

...ND_ADAPTER; ccw->flags = CCW_FLAG_SLI; @@ -1500,6 +1509,7 @@ static int __init virtio_ccw_init(void) { /* parse no_auto string before we do anything further */ no_auto_parse(); + summary_indicators = cio_dma_zalloc(MAX_AIRQ_AREAS); return ccw_driver_register(&virtio_ccw_driver); } device_initcall(virtio_ccw_init); -- 2.16.4

...ND_ADAPTER; ccw->flags = CCW_FLAG_SLI; @@ -1501,6 +1508,7 @@ static int __init virtio_ccw_init(void) { /* parse no_auto string before we do anything further */ no_auto_parse(); + summary_indicators = cio_dma_zalloc(MAX_AIRQ_AREAS); return ccw_driver_register(&virtio_ccw_driver); } device_initcall(virtio_ccw_init); -- 2.13.4

...1,6 +1508,11 @@ static int __init virtio_ccw_init(void) { /* parse no_auto string before we do anything further */ no_auto_parse(); + + summary_indicators = cio_dma_zalloc(MAX_AIRQ_AREAS); + if (!summary_indicators) + return -ENOMEM; + return ccw_driver_register(&virtio_ccw_driver); } device_initcall(virtio_ccw_init); -- 2.13.4

...o_parse(); - return ccw_driver_register(&virtio_ccw_driver); + + summary_indicators = cio_dma_zalloc(MAX_AIRQ_AREAS); + if (!summary_indicators) + return -ENOMEM; + rc = ccw_driver_register(&virtio_ccw_driver); + if (rc) + cio_dma_free(summary_indicators, MAX_AIRQ_AREAS); + return rc; } device_initcall(virtio_ccw_init); -- 2.17.1

...o_parse(); - return ccw_driver_register(&virtio_ccw_driver); + + summary_indicators = cio_dma_zalloc(MAX_AIRQ_AREAS); + if (!summary_indicators) + return -ENOMEM; + rc = ccw_driver_register(&virtio_ccw_driver); + if (rc) + cio_dma_free(summary_indicators, MAX_AIRQ_AREAS); + return rc; } device_initcall(virtio_ccw_init); -- 2.17.1

..._DEVICE_MAX, - "hmm_device"); - if (ret) - return ret; - - hmm_device_class = class_create(THIS_MODULE, "hmm_device"); - if (IS_ERR(hmm_device_class)) { - unregister_chrdev_region(hmm_device_devt, HMM_DEVICE_MAX); - return PTR_ERR(hmm_device_class); - } - return 0; -} - -device_initcall(hmm_init); #endif /* CONFIG_DEVICE_PRIVATE || CONFIG_DEVICE_PUBLIC */ -- 2.20.1