search for: l513

...gt; Why not just allocate event structures separately? > > Is it a win to have them share a cache line for some reason? Users may call vring_new_virtqueue() with preallocated memory to setup the ring, e.g.: https://github.com/torvalds/linux/blob/11da3a7f84f1/drivers/s390/virtio/virtio_ccw.c#L513-L522 https://github.com/torvalds/linux/blob/11da3a7f84f1/drivers/misc/mic/vop/vop_main.c#L306-L307 Below is the corresponding definition in split ring: https://github.com/oasis-tcs/virtio-spec/blob/89dd55f5e606/split-ring.tex#L64-L78 If my understanding is correct, this is just for legacy interf...

...PS D:\> echo $LASTEXITCODE 0 The issue here is in line https://repo.or.cz/syslinux.git/blob/HEAD:/win/syslinux.c#l509 eventhough we fail to update MBR but we still exit with 0. Need to have exit(1) to indicate that operation has failed. https://repo.or.cz/syslinux.git/blob/HEAD:/win/syslinux.c#l513 can have exit(1) aswell. thanks, Pradhap

On Wed, Jul 11, 2018 at 10:27:08AM +0800, Tiwei Bie wrote: > This commit introduces the support for creating packed ring. > All split ring specific functions are added _split suffix. > Some necessary stubs for packed ring are also added. > > Signed-off-by: Tiwei Bie <tiwei.bie at intel.com> I'd rather have a patch just renaming split functions, then add all packed stuff

On Wed, Jul 11, 2018 at 10:27:08AM +0800, Tiwei Bie wrote: > This commit introduces the support for creating packed ring. > All split ring specific functions are added _split suffix. > Some necessary stubs for packed ring are also added. > > Signed-off-by: Tiwei Bie <tiwei.bie at intel.com> I'd rather have a patch just renaming split functions, then add all packed stuff