[PATCH v2] powerpc: dts: t1040rdb: fix ports names for Seville Ethernet switch

2022-01-11 Thread Maxim
From: Maxim Kiselev On board rev A, the network interface labels for the switch ports written on the front panel are different than on rev B and later. This patch fixes network interface names for the switch ports according to labels that are written on the front panel of the board rev B. They

Re: kexec for powerpc32

2010-04-01 Thread Maxim Uvarov
Christian Kujau wrote: On Wed, 31 Mar 2010 at 17:23, Anton Vorontsov wrote: Kernel has all needed for kexec, but kexec-tools are broken for powerpc32. http://www.mail-archive.com/linuxppc-dev@lists.ozlabs.org/msg22498.html Oh :-\ I've just asked around, and it seems that

Re: [PATCH] kexec-tools, ppc64: fix build error on ppc64

2010-05-05 Thread Maxim Uvarov
ndex 831b263..03bdb9a 100644 > --- a/kexec/arch/mips/Makefile > +++ b/kexec/arch/mips/Makefile > @@ -14,5 +14,3 @@ dist += kexec/arch/mips/Makefile $(mips_KEXEC_SRCS) > \ > kexec/arch/mips/kexec-mips.h\ >kexec/arch/mips/crashdump-mips.h

[PATCH] Fix kexec on powerpc32

2010-05-11 Thread Maxim Uvarov
files; - combine ppc and ppc64 to powerpc directory (I'm planning to do it. And that why in some places my patch have ifdefs for PPC64); Best regards, Maxim Uvarov. From: Maxim Uvarov Signed-off-by: Maxim Uvarov Signed-off-by: Maxim Uvarov --- kexec/arch/ppc/Makefile

[PATCH] [POWEPC] crashdump: do not fail on null pointer dereferencing

2010-05-11 Thread Maxim Uvarov
Best regards, Maxim Uvarov. From: Maxim Uvarov Signed-off-by: Maxim Uvarov --- arch/powerpc/kernel/crash.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/kernel/crash.c b/arch/powerpc/kernel/crash.c index 6f4613d..341d8af 100644 --- a/arch/powerpc

[PATCH 1/2] Fix kexec on powerpc32

2010-05-11 Thread Maxim Uvarov
This patch is required in case if you are using new toolchains. Best regards, Maxim Uvarov. From: Maxim Uvarov Signed-off-by: Maxim Uvarov --- kexec/arch/ppc/Makefile|2 kexec/arch/ppc/crashdump-powerpc.c | 439 ++ kexec/arch/ppc/crashdump

[PATCH 2/2] powerpc new toolchains fix (crt.S)

2010-05-11 Thread Maxim Uvarov
This patch is required in case if you are using new toolchains. Best regards, Maxim Uvarov. From: Maxim Uvarov Linker does not provide some vital functions when building freestanding applications with a new toolchain, so we have to provide our own CRT. p.s. Without the CRT we won't se

[PATCH] powerpc new toolchains fix (crt.S)

2010-05-11 Thread Maxim Uvarov
Forgot to add purgatory/arch/ppc/crt.S to the patch. Resending... -- This patch is required in case if you are using new toolchains. Best regards, Maxim Uvarov. From: Maxim Uvarov Linker does not provide some vital functions when building freestanding applications with a new toolchain, so we

Re: [PATCH 1/2] Fix kexec on powerpc32

2010-05-11 Thread Maxim Uvarov
2010/5/12 Christian Kujau > Maxim Uvarov wrote on 2010-05-11 10:47 : > > This patch is required in case if you are using new toolchains. > > I'm on Debian/stable (binutils 2.18, gcc 4.3.2), so I applied your patches > to the latest -git checkout: > > First, co

[PATCH v2 1/2] Fix kexec on powerpc32

2010-05-12 Thread Maxim Uvarov
c" echo c > /proc/sysrq-trigger I also think that is is reasonable: - put GAME_CUBE specific code to separate files; - combine ppc and ppc64 to powerpc directory (I'm planning to do it. And that why in some places my patch have ifdefs for PPC64); Best

[PATCH v2 2/2] powerpc new toolchains fix (crt.S)

2010-05-12 Thread Maxim Uvarov
c" echo c > /proc/sysrq-trigger I also think that is is reasonable: - put GAME_CUBE specific code to separate files; - combine ppc and ppc64 to powerpc directory (I'm planning to do it. And that why in some places my patch have ifdefs for PPC64); Best

Re: [PATCH 1/2] Fix kexec on powerpc32

2010-05-12 Thread Maxim Uvarov
2010/5/12 Christian Kujau > On Tue, 11 May 2010 at 21:47, Maxim Uvarov wrote: > > This patch is required in case if you are using new toolchains. > > After kexec-tools compiled, I actually tried it: > > # kexec -l /boot/2.6/zImage --command-line="root=/dev/hda6 ro&quo

Re: [PATCH 1/2] Fix kexec on powerpc32

2010-05-13 Thread Maxim Uvarov
flags (or kernel options?) I can set, to find out why > it hangs here? Or does kexec not understand zImages? > > zImage should not work with kexec. Only vmlinux. > Thanks, > Christian. > -- > BOFH excuse #373: > > Suspicio

Re: [PATCH v3 0/7] Fixup booting with device trees and uImage/elf on ppc32

2010-07-26 Thread Maxim Uvarov
h/ppc/purgatory-ppc.c|5 + > > 11 files changed, 852 insertions(+), 239 deletions(-) > > create mode 100644 doc/mpc85xx.txt > > > > > > > > ___ > > kexec mailing list > > ke...@lists.infradead.org > > http://lists.infradead.org/mailman/listinfo/kexec > -- Best regards, Maxim Uvarov ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH v4 10/12] KVM: x86: never write to memory from kvm_vcpu_check_block()

2023-12-13 Thread Maxim Levitsky
rsion of 'vmx_complete_nested_posted_interrupt' and call it from kvm_guest_apic_has_interrupt(). What do you think? I can prepare a patch for this. Can you share a reproducer or write a new one that can be shared? Best regards, Maxim Levitsky

Re: [PATCH v4 10/12] KVM: x86: never write to memory from kvm_vcpu_check_block()

2023-12-13 Thread Maxim Levitsky
On Wed, 2023-12-13 at 14:31 -0800, Jim Mattson wrote: > On Wed, Dec 13, 2023 at 2:25 PM Maxim Levitsky wrote: > > On Tue, 2023-12-12 at 07:28 -0800, Sean Christopherson wrote: > > > On Sun, Dec 10, 2023, Jim Mattson wrote: > > > > On Thu, Dec 7, 2023 at 8:21 AM Sean

Re: [PATCH] soc: fsl: qe: Add check for platform_driver_register

2024-04-08 Thread Maxim Kochetkov
09.04.2024 06:08, Chen Ni пишет: Return platform_driver_register() in order to transfer the error if it fails. Fixes: be7ecbd240b2 ("soc: fsl: qe: convert QE interrupt controller to platform_device") Signed-off-by: Chen Ni Reviewed-by: Maxim Kochetkov

Re: [PATCH v2 devicetree 1/2] powerpc: dts: t1040: add bindings for Seville Ethernet switch

2020-09-29 Thread Maxim Kochetkov
Reviewed-by: Maxim Kochetkov 29.09.2020 14:32, Vladimir Oltean пишет: Add the description of the embedded L2 switch inside the SoC dtsi file for NXP T1040. Signed-off-by: Vladimir Oltean --- Changes in v2: Make switch node disabled by default. arch/powerpc/boot/dts/fsl/t1040si-post.dtsi

Re: [PATCH v2 devicetree 2/2] powerpc: dts: t1040rdb: add ports for Seville Ethernet switch

2020-09-29 Thread Maxim Kochetkov
Reviewed-by: Maxim Kochetkov 29.09.2020 14:32, Vladimir Oltean пишет: From: Vladimir Oltean Define the network interface names for the switch ports and hook them up to the 2 QSGMII PHYs that are onboard. A conscious decision was taken to go along with the numbers that are written on the

Re: [PATCH v2 devicetree 0/2] Add Seville Ethernet switch to T1040RDB

2020-09-29 Thread Maxim Kochetkov
Reviewed-by: Maxim Kochetkov 29.09.2020 14:32, Vladimir Oltean пишет: Seville is a DSA switch that is embedded inside the T1040 SoC, and supported by the mscc_seville DSA driver inside drivers/net/dsa/ocelot. This series adds this switch to the SoC's dtsi files and to the T1040RDB board

Re: [PATCH] synchronize_irq needs a barrier

2007-10-19 Thread Maxim Levitsky
urn 0; and the interrupt handler: smp_rmb(); if (dev->insuspend) goto out; Am I right? Best regards, Maxim Levitsky ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [PATCH] synchronize_irq needs a barrier

2007-10-19 Thread Maxim Levitsky
On Saturday 20 October 2007 04:25:34 Linus Torvalds wrote: > > On Sat, 20 Oct 2007, Maxim Levitsky wrote: > > > > and the interrupt handler: > > > > smp_rmb(); > > if (dev->insuspend) > > goto out; > > Something like that

Re: [PATCH] synchronize_irq needs a barrier

2007-10-19 Thread Maxim Levitsky
i_dev, 0x40, tmp); pci_enable_wake(pci_dev, PCI_D3hot, 1); pci_enable_wake(pci_dev, PCI_D3cold, 1); /* Power down device*/ pci_set_power_state(pci_dev, pci_choose_state (pci_dev,state)); pci_save_state(pci_dev); I guess, everybody makes mistakes... :-) Other network drivers has a bit more complicated .suspend/.resume routines, but I didn't see a driver waiting for output queue to finish Best regards, Maxim Levitsky ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [PATCH] synchronize_irq needs a barrier

2007-10-19 Thread Maxim Levitsky
> Hi, Thanks a lot. I fix the order of calls in dmfe.c and in saa7134-core.c. I probably need to add this synchronize_irq() logic in dmfe.c too, but I probably do it later, I think I am overestimating this race, since most drivers don't do dev->insuspend checks in IRQ handler. Maybe even just use free_irq() after all Best regards, Maxim Levitsky ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [PATCH] synchronize_irq needs a barrier

2007-10-19 Thread Maxim Levitsky
in case*/ saa_writel(SAA7134_IRQ_REPORT,saa_readl(SAA7134_IRQ_REPORT)); ...... This should be bullet-proof. > > Ben. > > > Best regards, Maxim Levitsky ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

1st version of azfs

2007-12-17 Thread Maxim Shchetynin
Hello, please, have a look at the following patch. This is a first version of a non-buffered filesystem to be used on "ioremapped" devices. Thank you in advance for your comments. Subject: azfs: initial submit of azfs, a non-buffered filesystem From: Maxim Shchetynin <[EMAIL PRO

1st version of azfs

2007-12-17 Thread Maxim Shchetynin
...and here once more the same patch as attachment... (See attached file: linux-2.6.24-rc4-azfs.diff.gz) Mit freundlichen Grüßen / met vriendelijke groeten / avec regards Maxim V. Shchetynin Linux Kernel Entwicklung IBM Deutschland Entwicklung GmbH Linux für Cell, Abteilung

Re: AZFS file system proposal

2008-07-07 Thread Maxim Shchetynin
ed memory that needs to be uncached but should > not be guarded, for performance reasons. > > Maybe we can introduce a new PAGE_IOMEM here that we can use > in all places that need something like this. In spufs we need > the same flags for the local store mappings. > > I woul

azfs: initial submit of azfs, a non-buffered filesystem

2008-07-07 Thread Maxim Shchetynin
AZFS is a file system which keeps all files on memory mapped random access storage. It was designed to work on the axonram device driver for IBM QS2x blade servers, but can operate on any block device that exports a direct_access method. Signed-off-by: Maxim Shchetynin <[EMAIL PROTEC

Re: azfs: initial submit of azfs, a non-buffered filesystem

2008-07-08 Thread Maxim Shchetynin
Am Mon, 7 Jul 2008 21:37:43 +0200 schrieb Uli Luckas <[EMAIL PROTECTED]>: > > AZFS is a file system which keeps all files on memory mapped random > > access storage. > Hi Maxim, > do you mean "memory backed" instead of "memory mapped"? Right, I have c

Re: AZFS file system proposal

2008-07-09 Thread Maxim Shchetynin
ndif ... ... ... -- Mit freundlichen Grüßen / met vriendelijke groeten / avec regards Maxim V. Shchetynin Linux Kernel Entwicklung IBM Deutschland Research & Development GmbH Linux für Cell, Abteilung 3250 Schönaicher Straße 220 71032 Böblingen Vorsitzender des Auf

Re: AZFS file system proposal

2008-07-09 Thread Maxim Shchetynin
et vriendelijke groeten / avec regards Maxim V. Shchetynin Linux Kernel Entwicklung IBM Deutschland Research & Development GmbH Linux für Cell, Abteilung 3250 Schönaicher Straße 220 71032 Böblingen Vorsitzender des Aufsichtsrats: Martin Jetter Geschäftsführung: Herber

Re: [PATCH] kexec memory ranges dynamic allocation

2008-10-15 Thread Maxim Uvarov
2008/10/15 Simon Horman <[EMAIL PROTECTED]> > On Tue, Oct 14, 2008 at 07:11:19PM +0400, Maxim Uvarov wrote: > > Hello all, > > > > As you all know it is not easy to count exact value of memory ranges from > > device tree on powerpc. > > It very depends o

Re: [PATCH] kexec memory ranges dynamic allocation

2008-10-15 Thread Maxim Uvarov
Patch corrected. ( git_kexec_powerpc_v2.patch is attached.) I tested it on ppc64 pasemi electra board. Both kexec -l and kexec -p works. Maxim. 2008/10/15 Simon Horman <[EMAIL PROTECTED]> > On Tue, Oct 14, 2008 at 07:11:19PM +0400, Maxim Uvarov wrote: > > Hello all, > >

Re: [PATCH] kexec memory ranges dynamic allocation

2008-10-30 Thread Maxim Uvarov
t 15, 2008 at 12:46:24PM +0400, Maxim Uvarov wrote: > > Patch corrected. ( git_kexec_powerpc_v2.patch is attached.) > > > > I tested it on ppc64 pasemi electra board. Both kexec -l and kexec -p > works. > > > > Maxim. > > > > > > > > 2008/10

Re: [PATCH] kexec memory ranges dynamic allocation

2008-11-01 Thread Maxim Uvarov
2008/11/1 Simon Horman <[EMAIL PROTECTED]> > On Fri, Oct 31, 2008 at 09:53:23AM +0300, Maxim Uvarov wrote: > > 2008/10/31 Simon Horman <[EMAIL PROTECTED]> > > > > > Hi, > > > > > > Could someone please comment on the satus of this patch? >

Re: [PATCH 1/3] KVM: x86: Don't re-acquire SRCU lock in complete_emulated_io()

2022-04-19 Thread Maxim Levitsky
srcu); > - r = kvm_emulate_instruction(vcpu, EMULTYPE_NO_DECODE); > - srcu_read_unlock(&vcpu->kvm->srcu, vcpu->srcu_idx); > - return r; > + return kvm_emulate_instruction(vcpu, EMULTYPE_NO_DECODE); > } > > static int complete_emulated_pio(struct kvm_vcpu *vcpu) I wonder how this did work Reviewed-by: Maxim Levitsky Best regards, Maxim Levitsky

Re: [PATCH 3/3] KVM: Add helpers to wrap vcpu->srcu_idx and yell if it's abused

2022-04-19 Thread Maxim Levitsky
rcu_read_lock(vcpu); > > if (kvm_apic_accept_events(vcpu) < 0) { > r = 0; > @@ -10625,7 +10623,7 @@ int kvm_arch_vcpu_ioctl_run(struct kvm_vcpu *vcpu) > if (kvm_run->kvm_valid_regs) > store_regs(vcpu); > post_kvm_run_save(vcpu); > - srcu_read_unlock(&kvm->srcu, vcpu->srcu_idx); > + kvm_vcpu_srcu_read_unlock(vcpu); > > kvm_sigset_deactivate(vcpu); > vcpu_put(vcpu); > diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h > index 252ee4a61b58..76fc7233bd6a 100644 > --- a/include/linux/kvm_host.h > +++ b/include/linux/kvm_host.h > @@ -315,7 +315,10 @@ struct kvm_vcpu { > int cpu; > int vcpu_id; /* id given by userspace at creation */ > int vcpu_idx; /* index in kvm->vcpus array */ > - int srcu_idx; > + int srcu_idx; /* Don't use this directly. You've been warned. */ > +#ifdef CONFIG_PROVE_RCU > + int srcu_depth; > +#endif > int mode; > u64 requests; > unsigned long guest_debug; > @@ -841,6 +844,25 @@ static inline void kvm_vm_bugged(struct kvm *kvm) > unlikely(__ret);\ > }) > > +static inline void kvm_vcpu_srcu_read_lock(struct kvm_vcpu *vcpu) > +{ > +#ifdef CONFIG_PROVE_RCU > + WARN_ONCE(vcpu->srcu_depth++, > + "KVM: Illegal vCPU srcu_idx LOCK, depth=%d", vcpu->srcu_depth > - 1); > +#endif > + vcpu->srcu_idx = srcu_read_lock(&vcpu->kvm->srcu); > +} > + > +static inline void kvm_vcpu_srcu_read_unlock(struct kvm_vcpu *vcpu) > +{ > + srcu_read_unlock(&vcpu->kvm->srcu, vcpu->srcu_idx); > + > +#ifdef CONFIG_PROVE_RCU > + WARN_ONCE(--vcpu->srcu_depth, > + "KVM: Illegal vCPU srcu_idx UNLOCK, depth=%d", > vcpu->srcu_depth); > +#endif > +} > + > static inline bool kvm_dirty_log_manual_protect_and_init_set(struct kvm *kvm) > { > return !!(kvm->manual_dirty_log_protect & KVM_DIRTY_LOG_INITIALLY_SET); Looks good to me overall. Note that there are still places that acquire the lock and store the idx into a local variable, for example kvm_xen_vcpu_set_attr and such. I didn't check yet if these should be converted as well. Best regards, Maxim Levitsky

Re: [PATCH 3/3] KVM: Add helpers to wrap vcpu->srcu_idx and yell if it's abused

2022-04-19 Thread Maxim Levitsky
On Tue, 2022-04-19 at 15:45 +, Sean Christopherson wrote: > On Tue, Apr 19, 2022, Maxim Levitsky wrote: > > On Fri, 2022-04-15 at 00:43 +, Sean Christopherson wrote: > > > Add wrappers to acquire/release KVM's SRCU lock when stashing the index > > > in vcpu-

[PATCH] soc: fsl: qe: convert QE interrupt controller to platform_device

2021-07-05 Thread Maxim Kochetkov
devlink=on by default") Fixes: ea718c699055 ("Revert "Revert "driver core: Set fw_devlink=on by default""") Signed-off-by: Maxim Kochetkov --- drivers/soc/fsl/qe/qe_ic.c | 38 +++--- 1 file changed, 23 insertions(+), 15 deletions(

Re: [PATCH] soc: fsl: qe: convert QE interrupt controller to platform_device

2021-07-19 Thread Maxim Kochetkov
15.07.2021 01:29, Li Yang wrote: From the original code, this should be type = "qeic". It is not defined in current binding but probably needed for backward compatibility. I took these strings from this part: np = of_find_compatible_node(NULL, NULL, "fsl,qe-ic"); if (!np) {

[PATCH v2] soc: fsl: qe: convert QE interrupt controller to platform_device

2021-07-19 Thread Maxim Kochetkov
[1] - https://lore.kernel.org/lkml/CAGETcx9PiX==mlxb9po8myyk6u2vhpvwtmsa5nkd-ywh5xh...@mail.gmail.com Fixes: e590474768f1 ("driver core: Set fw_devlink=on by default") Fixes: ea718c699055 ("Revert "Revert "driver core: Set fw_devlink=on by default""") Signed-off-by: Maxim Kochetko

[PATCH v3] soc: fsl: qe: convert QE interrupt controller to platform_device

2021-07-26 Thread Maxim Kochetkov
[1] - https://lore.kernel.org/lkml/CAGETcx9PiX==mlxb9po8myyk6u2vhpvwtmsa5nkd-ywh5xh...@mail.gmail.com Fixes: e590474768f1 ("driver core: Set fw_devlink=on by default") Fixes: ea718c699055 ("Revert "Revert "driver core: Set fw_devlink=on by default""") Signed-off-by: Maxim Kochetko

[PATCH v4] soc: fsl: qe: convert QE interrupt controller to platform_device

2021-08-03 Thread Maxim Kochetkov
[1] - https://lore.kernel.org/lkml/CAGETcx9PiX==mlxb9po8myyk6u2vhpvwtmsa5nkd-ywh5xh...@mail.gmail.com Fixes: e590474768f1 ("driver core: Set fw_devlink=on by default") Fixes: ea718c699055 ("Revert "Revert "driver core: Set fw_devlink=on by default""") Signed-off-by: Maxim Kochet

Re: [PATCH v4] soc: fsl: qe: convert QE interrupt controller to platform_device

2021-08-06 Thread Maxim Kochetkov
03.08.2021 20:51, Saravana Kannan wrote: So lets convert this driver to simple platform_device with probe(). Also use platform_get_ and devm_ family function to get/allocate resources and drop unused .compatible = "qeic". Yes, please! Should I totally drop { .type = "qeic"}, or keep?

[PATCH] soc: fsl: qe: fix static checker warning

2021-08-11 Thread Maxim Kochetkov
o make checkpatch happy. Fixes: be7ecbd240b2 ("soc: fsl: qe: convert QE interrupt controller to platform_device") Signed-off-by: Maxim Kochetkov Reported-by: Dan Carpenter --- drivers/soc/fsl/qe/qe_ic.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/drivers/soc

[PATCH] powerpc: dts: t1040rdb: fix ports names for Seville Ethernet switch

2021-12-30 Thread Maxim Kiselev
Fix network interface names for the switch ports according to labels that are written on the front panel of the board. They start from ETH3 and end at ETH10. Fixes: e69eb0824d8c ("powerpc: dts: t1040rdb: add ports for Seville Ethernet switch") Signed-off-by: Maxim Kiselev Reviewed

Re: [PATCH] powerpc: dts: t1040rdb: fix ports names for Seville Ethernet switch

2021-12-30 Thread Maxim Kiselev
this pic https://habrastorage.org/files/433/d0f/057/433d0f05784f42b6acf9bb777970b1e9.jpg чт, 30 дек. 2021 г. в 16:00, Vladimir Oltean : > On Thu, Dec 30, 2021 at 01:43:28PM +0300, Maxim Kiselev wrote: > > Fix network interface names for the switch ports according to labels > > t

[PATCH] powerpc: dts: t104xrdb: fix phy type for FMAN 4/5

2021-12-30 Thread Maxim Kiselev
T1040RDB has two RTL8211E-VB phys which requires setting of internal delays for correct work. Changing the phy-connection-type property to `rgmii-id` will fix this issue. Signed-off-by: Maxim Kiselev Reviewed-by: Maxim Kochetkov --- arch/powerpc/boot/dts/fsl/t104xrdb.dtsi | 4 ++-- 1 file

[PATCH] powerpc: dts: add device tree for T1040RDB-REV-A

2022-01-11 Thread Maxim Kiselev
On board rev A, the network interface labels for the switch ports written on the front panel are different than on rev B and later. This patch introduces a separate device tree for rev A. The main device tree is supposed to cover rev B and later. Signed-off-by: Maxim Kiselev --- arch/powerpc

[PATCH v3] powerpc: dts: t1040rdb: fix ports names for Seville Ethernet switch

2022-01-21 Thread Maxim Kiselev
end at ETH10. This patch also introduces a separate device tree for rev A. The main device tree is supposed to cover rev B and later. Fixes: e69eb0824d8c ("powerpc: dts: t1040rdb: add ports for Seville Ethernet switch") Signed-off-by: Maxim Kiselev Reviewed-by: Maxim Kochetkov R